Граф коммитов

10026 Коммитов

Автор SHA1 Сообщение Дата
mostynb 61e093bf8a gn: leave PKG_CONFIG_PATH untouched when not using a sysroot
This makes cross-compilation possible when not using a sysroot.

Review URL: https://codereview.chromium.org/1543483002

Cr-Original-Commit-Position: refs/heads/master@{#367906}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: c61271cf022bfb412db3e53ce8bbc5d8c2c1a873
2016-01-06 21:09:01 +00:00
mark 3b6fac037b Update Crashpad to 54048cfd78af0a54cf4a2d2ecfb83881f5c54590
6d829e9af7d7 Create WorkerThread, an abstraction to perform some work on
             an interval
5e244711888b Fix build break after 6d829e9af7d7
18a294130552 Create the PruneCrashReportsThread in the handler
c46a676624cf Use std::move() in PruneCrashReportThread
6d2d31d2d137 Use base/macros.h instead of base/basictypes.h
b7e4564ff219 Update mini_chromium to a43fee120b10
54048cfd78af Don’t suppress_wildcard crashpad.gyp:All on platforms where
             it works

Review URL: https://codereview.chromium.org/1568553002

Cr-Original-Commit-Position: refs/heads/master@{#367899}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: b25d2d936da7620ff44ce4a2952f38402efad685
2016-01-06 20:43:49 +00:00
agrieve ff5dda029e GN(android): Simplify android default libs logic
* Always use full path to libgcc.a
* Specify libs c, dl, m only once in either default_libs or runtime_library

Should be a no-op.

BUG=570406

Review URL: https://codereview.chromium.org/1539353002

Cr-Original-Commit-Position: refs/heads/master@{#367877}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 8d8895cc227d9247c1172f235603c23d0be15ab4
2016-01-06 19:29:01 +00:00
thestig 570e4eda7b Put more files under the enable_service_discovery build flag.
TBR=thakis@chromium.org

Review URL: https://codereview.chromium.org/1554563002

Cr-Original-Commit-Position: refs/heads/master@{#367872}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 09de2b88725e565520cc5506593cc6f58251710a
2016-01-06 19:05:01 +00:00
thakis e3c9151144 clang: Enable -Wfor-loop-analysis.
Every time I try this warning, it finds a few true positives
and has no false negatives.  It looks like an excellent
warning.

I did three builds of target "gfx" in both Release and Debug
with and without this warning enabled, doing local (non-goma)
component builds (running -t clean between each build). This
builds about 1.6k build steps each time.

Warning on:
Release: 0m26.850s 0m27.227s 0m27.159s
Debug: 0m24.811s 0m25.044s 0m25.498s

Warning off:
Release: 0m26.834s 0m27.242s 0m26.952s
Debug: 0m24.879s 0m25.064s 0m25.673s

See e.g. the comments in
https://codereview.chromium.org/1269343005/diff/240001/chrome/renderer/spellchecker/spellcheck_worditerator_unittest.cc
https://codereview.chromium.org/196433002/diff/60001/media/cast/transport/pacing/paced_sender_unittest.cc
for examples.

BUG=none
TBR=imcheng

Review URL: https://codereview.chromium.org/1273313003

Cr-Original-Commit-Position: refs/heads/master@{#367855}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 29ec253a32cc66d0855d8e6f17aafa00b1996a80
2016-01-06 18:14:48 +00:00
jbudorick f012cc226f [Android] Add APK under test and additional APK support for the old mode.
BUG=

Review URL: https://codereview.chromium.org/1564583003

Cr-Original-Commit-Position: refs/heads/master@{#367834}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 248e31ab21460ae0e3ddce0cdb6f83e1234a9636
2016-01-06 16:29:09 +00:00
agrieve 830b7e8f8a Move use_webview_internal_framework to android_webview/BUILD.gn
BUG=574108

Review URL: https://codereview.chromium.org/1551393002

Cr-Original-Commit-Position: refs/heads/master@{#367830}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 2aceeb64bdf3d8504ad07d6a3a194cd736b4db3e
2016-01-06 16:04:32 +00:00
jbudorick 54cca8b2e1 [Android] Revise devil handling of host platforms.
This is NOPRESUBMIT because I've locked down devil for the move to catapult
via presubmit. That's the only presubmit failure.

BUG=574568
NOPRESUBMIT=true

Review URL: https://codereview.chromium.org/1565763002

Cr-Original-Commit-Position: refs/heads/master@{#367826}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: da062ad1f4876168012e3834f2f3eb187c6a8f45
2016-01-06 15:47:10 +00:00
hanxi 53c49a6049 Allow placeholders when the native_libs is empty.
Review URL: https://codereview.chromium.org/1563823002

Cr-Original-Commit-Position: refs/heads/master@{#367825}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 536d81764191b86be7a61978ad35694b0f0e78e2
2016-01-06 15:37:02 +00:00
agrieve 88be91117b Fix up proguard output filtering
Removes build noise when building release on Android.

BUG=573257

Review URL: https://codereview.chromium.org/1565433002

Cr-Original-Commit-Position: refs/heads/master@{#367762}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: a8815d201980b613bf76d8f8977c718623e74d03
2016-01-06 03:05:07 +00:00
agrieve ce84be6503 GN(android) Don't enable --gc-sections by default in debug mode.
The flag can cover over legitimate link errors. GYP does not enable it for debug builds.

BUG=159847

Review URL: https://codereview.chromium.org/1541163002

Cr-Original-Commit-Position: refs/heads/master@{#367756}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: c869f0e3f9117634a7980514732d6d743a522213
2016-01-06 02:49:10 +00:00
agrieve 10c2d06ae3 Ensure EMMA classes are put into the main classes.dex
This fixes the "class [com.vladium.util.IntSet] not found" error when
running with coverage on a pre-L device.

BUG=571633

Review URL: https://codereview.chromium.org/1557533003

Cr-Original-Commit-Position: refs/heads/master@{#367748}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 23ea0f5c54db0b62d3bc0c23a8bcb2560734e852
2016-01-06 02:00:01 +00:00
kkimlabs 9a351a810f Fix x64 android_tools target compile failure.
Few functions were unused on x86 causing unused function errors.

BUG=574494

Review URL: https://codereview.chromium.org/1564443002

Cr-Original-Commit-Position: refs/heads/master@{#367657}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 80090bd83b13c23c0f1c75d16e5f878da719b30a
2016-01-05 21:58:32 +00:00
agrieve 0f01c0eb40 GN: Set arm_float_abi when target_cpu = "arm64"
Fixes gn gen failure:

  Generating files...
  ERROR at //third_party/ffmpeg/BUILD.gn:199:11: Undefined identifier
  if (arm_float_abi == "hard") {
      ^------------

BUG=568883

Review URL: https://codereview.chromium.org/1551093002

Cr-Original-Commit-Position: refs/heads/master@{#367615}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 03301146b8d58e6f2e82b60f7df511be646f59a9
2016-01-05 19:46:22 +00:00
wangxianzhu 612d28a13a Revert unfinished work of "full-frame-measurement"
I wanted to measure full-frame performance using the existing
performance tests, but used a method requiring internal API of
content_shell. I didn't finish the remaining work of setting up
reference build of content_shell etc. Also slimiming paint will
change how compositing update, paint invalidation and paint work.

This CL reverts the unfinished work of "full-frame-measurement" by:
- Renaming PerfTestRunner.forceLayoutOrFullFrame() to
  PerfTestRunner.forceLayout();
- Removing full-frame-measurement benchmarks requiring content_shell;
- Removing content_shell from perf bot build.

Future full-frame-measurement tests should use requestAnimationFrame().
(See third_party/WebKit/PerformanceTests/Paint/large-table*.html
created by jchaffraix@ for example.)

BUG=426599
R=jchaffraix@chromium.org,nednguyen@google.com
TBR=wangxianzhu@chromium.org (build/all.gyp)
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:linux_perf_bisect;tryserver.chromium.perf:mac_10_10_perf_bisect;tryserver.chromium.perf:win_perf_bisect

Review URL: https://codereview.chromium.org/1557953002

Cr-Original-Commit-Position: refs/heads/master@{#367562}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 5cba1e2af6c1e29afbe088413bfbf6f94d6eb0f3
2016-01-05 17:42:29 +00:00
pasko 931f7f3363 Build cygprofile_unittests as part of All on Android.
There seems to be no existing bot that wants to run the binary built this way.

The target is currently only enabled for 'instrumented' builds, and this change
will make it build in the 'uninstrumented' configuration. With less
instrumentation it is expected to build just fine.

BUG=573326

Review URL: https://codereview.chromium.org/1552293002

Cr-Original-Commit-Position: refs/heads/master@{#367500}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: cf3368493c77dea49a6a76467417ec32416318d4
2016-01-05 09:12:05 +00:00
agrieve 8b76323816 Port cronet_package GYP target to GN
BUG=462737

Review URL: https://codereview.chromium.org/1553623003

Cr-Original-Commit-Position: refs/heads/master@{#367480}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: e23386b38a4086b1c92d7437f25197c57bcbf297
2016-01-05 04:49:42 +00:00
agrieve 65058fb755 GN Don't enable linker version script for components
It breaks native JNI exporting. Change aligns with GYP behaviour.

BUG=532905

Review URL: https://codereview.chromium.org/1548473003

Cr-Original-Commit-Position: refs/heads/master@{#367470}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 3f53232116444520bec48390d8812f18aa08606f
2016-01-05 03:24:01 +00:00
jbudorick 77bb33f835 [Android] Make devil/ read-only for the move to catapult.
BUG=476719

Review URL: https://codereview.chromium.org/1556983002

Cr-Original-Commit-Position: refs/heads/master@{#367430}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: b45e6de2eb286ae0a245a2d56fd851a3b20a83f4
2016-01-05 00:42:04 +00:00
Nico Weber 6a77d8960e ios: -Werror for third-party
BUG=573779
R=mark@chromium.org

Review URL: https://codereview.chromium.org/1551253002 .

Cr-Original-Commit-Position: refs/heads/master@{#367402}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: cbd8cfb57078da8fd29b509d714d282d42b8a14c
2016-01-04 23:20:30 +00:00
thestig ef03b25620 Remove obsolete win_pdf_metafile_for_printing GN flag.
Review URL: https://codereview.chromium.org/1552673002

Cr-Original-Commit-Position: refs/heads/master@{#367364}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: f5180a8f93deaaeb1032bf9a89a7f00bdc4c31d7
2016-01-04 21:08:10 +00:00
halliwell f3b30876cc Disable Blink assertions on Chromecast device eng builds
Chromecast device eng builds are release builds with dcheck_always_on
to take advantage of DCHECKS.  By default this also enables Blink
assertions, which add ~20% to our binary size.  Recently the eng OTA
package has become too large to push to the device, so space savings
are required, and the Blink assertions haven't provided a lot of value
historically.  They are still enabled for buildbots, desktop builds and
Android TV.

BUG=internal b/26142354

Review URL: https://codereview.chromium.org/1535633002

Cr-Original-Commit-Position: refs/heads/master@{#367363}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 25419b09eee2221bde1ac0320e29ca32026a1f3e
2016-01-04 21:06:44 +00:00
thakis 7d4f0b3508 linux: Build third-party code with -Wunused-result.
Part of an effort to make warning flags consistent across platforms.

BUG=574077

Review URL: https://codereview.chromium.org/1559903002

Cr-Original-Commit-Position: refs/heads/master@{#367355}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 09874b50f59f11ddd1268d6da87989ae09f07bfb
2016-01-04 20:12:38 +00:00
thakis 0cc85a3736 Enable -Wformat in linux, android, chromeos, cast builds.
No behavior change.

BUG=573780

Review URL: https://codereview.chromium.org/1551313002

Cr-Original-Commit-Position: refs/heads/master@{#367353}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 932e1e5939963b5b67019f428e8f209246069a44
2016-01-04 20:09:08 +00:00
thakis 612a8ef640 Roll NSS by one revision
Picks up https://codereview.chromium.org/1558903002/ and syncs the warning
flags in NSS's BUILD.gn with the gyp file.

BUG=573779

Review URL: https://codereview.chromium.org/1553193002

Cr-Original-Commit-Position: refs/heads/master@{#367337}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: be09deaee48ac4e4dc274c04da6ecddc032bd2f5
2016-01-04 19:07:10 +00:00
thakis 5d353bbce6 Remove small amounts of repitition in build/common.gypi
No intended behavior change.

BUG=none

Review URL: https://codereview.chromium.org/1556953002

Cr-Original-Commit-Position: refs/heads/master@{#367336}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: ab5bf4828ec6e3f1f51667e5adff9ff606614ff7
2016-01-04 19:01:03 +00:00
krasin d1b458451f Reduce memory requirement to linux LTO builds to speed up bots.
The new requirements won't be enough for the official bots,
but the old requirements made all the bots to be extremely
slow (5-7 hours), and we don't run official bots as of now,
because that was too slow for CFI launch.

Reverting to the settings good enough for development, and keep in mind,
that we need to do something about the memory consumption for LTO builds
before we could attempt to launch again.

BUG=570227,464797

Review URL: https://codereview.chromium.org/1553093004

Cr-Original-Commit-Position: refs/heads/master@{#367323}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 61e70ccb5957620f3329f07ef79a76b81c96044b
2016-01-04 17:49:33 +00:00
daniimms 65e8d38de4 Fix proto_java_library comment example
proto_path is a string not an array.

BUG=NONE

Review URL: https://codereview.chromium.org/1552073002

Cr-Original-Commit-Position: refs/heads/master@{#367319}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 24ad7029705b271d0a12107231a1db5f4d5abb6d
2016-01-04 16:58:21 +00:00
agrieve e9e500e4b9 Remove lib.target from search paths in adb_gdb and symbol.py
lib.target is used when building with make, which we don't do anymore.

BUG=573347

Review URL: https://codereview.chromium.org/1551993003

Cr-Original-Commit-Position: refs/heads/master@{#367315}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 1dea8dee5f372a03416fcc26faf15f2b0cb716c6
2016-01-04 16:40:37 +00:00
thakis 870911572a Simplify xcode_settings block in common.gypi a bit.
- Mac builds are always 64-bit nowaydays
- Collapse two adjacent OS==ios conditions

No intended behavior change.

BUG=none

Review URL: https://codereview.chromium.org/1557823002

Cr-Original-Commit-Position: refs/heads/master@{#367314}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: ea6bdacc04396d48bd80fc40cf490ad238d88e4d
2016-01-04 16:39:19 +00:00
raphael.kubo.da.costa 99eb573309 Add more packages to the Wheezy sysroot.
x11proto-xinerama-dev is needed by libxinerama-dev, libxdmcp-dev is used
by xcb.

With this change (and after regenerating the sysroot), it is possible to
run gn and configure the build without having a single pkg-config in the
host system (tested on amd64).

R=sbc@chromium.org,dpranke@chromium.org

Review URL: https://codereview.chromium.org/1539393002

Cr-Original-Commit-Position: refs/heads/master@{#367302}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 34a5998d2a36de0149470d23b7f98df863e61d21
2016-01-04 11:05:37 +00:00
thakis e6523fafd5 Linux/CrOS/Cast/Android (and Mac with gn): Build 3rd-party code with -Wall.
Lands the remaining -Wall bits of https://codereview.chromium.org/1551753002/
(reviewed there).

Windows, Mac, and iOS already build 3rd-party code with -Wall.  (Mac and iOS
only in the gyp build, not in gn yet.)  This makes the other platforms match
and also adds -Wall to no_chromium_code in gn builds.

BUG=573250
TBR=thestig@chromium.org

Review URL: https://codereview.chromium.org/1555893002

Cr-Original-Commit-Position: refs/heads/master@{#367275}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 550440b4763042d462c7b0fc09dfba1408be8a8c
2016-01-02 03:12:16 +00:00
geofflang d98282fd8b Reland of mac and ios: Build third-party code with -Wall. (patchset #1 id:1 of https://codereview.chromium.org/1556843002/ )
Reason for revert:
thakis@ plans to address the failing bots.

Original issue's description:
> Revert of mac and ios: Build third-party code with -Wall. (patchset #2 id:20001 of https://codereview.chromium.org/1555843002/ )
>
> Reason for revert:
> Fails compilation on Mac GPU bots.
>
> Original issue's description:
> > mac and ios: Build third-party code with -Wall.
> >
> > This lands the mac and ios build/common.gypi bits of
> > https://codereview.chromium.org/1551753002/ (reviewed there)
> >
> > BUG=573250
> > R=thestig@chromium.org
> > TBR=thestig@chromium.org
> >
> > Committed: https://crrev.com/9830789346abd3d8211deff1ebe7a7f5753ba3fc
> > Cr-Commit-Position: refs/heads/master@{#367255}
> >
> > Committed: https://crrev.com/de5c041e67fa5abee83359b9346ca4896f1723ca
> > Cr-Commit-Position: refs/heads/master@{#367263}
>
> TBR=thestig@chromium.org,thakis@chromium.org
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=573250
>
> Committed: https://crrev.com/6f0ae110927c19c75dac785887d56dc61c90abc5
> Cr-Commit-Position: refs/heads/master@{#367267}

TBR=thestig@chromium.org,thakis@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=573250

Review URL: https://codereview.chromium.org/1554923002

Cr-Original-Commit-Position: refs/heads/master@{#367269}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 59f419c5b24f891279c78b19bc6963c6762d69ef
2016-01-01 23:16:11 +00:00
geofflang f510870c6a Revert of mac and ios: Build third-party code with -Wall. (patchset #2 id:20001 of https://codereview.chromium.org/1555843002/ )
Reason for revert:
Fails compilation on Mac GPU bots.

Original issue's description:
> mac and ios: Build third-party code with -Wall.
>
> This lands the mac and ios build/common.gypi bits of
> https://codereview.chromium.org/1551753002/ (reviewed there)
>
> BUG=573250
> R=thestig@chromium.org
> TBR=thestig@chromium.org
>
> Committed: https://crrev.com/9830789346abd3d8211deff1ebe7a7f5753ba3fc
> Cr-Commit-Position: refs/heads/master@{#367255}
>
> Committed: https://crrev.com/de5c041e67fa5abee83359b9346ca4896f1723ca
> Cr-Commit-Position: refs/heads/master@{#367263}

TBR=thestig@chromium.org,thakis@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=573250

Review URL: https://codereview.chromium.org/1556843002

Cr-Original-Commit-Position: refs/heads/master@{#367267}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 6f0ae110927c19c75dac785887d56dc61c90abc5
2016-01-01 23:05:27 +00:00
Nico Weber 11a85fedfd mac and ios: Build third-party code with -Wall.
This lands the mac and ios build/common.gypi bits of
https://codereview.chromium.org/1551753002/ (reviewed there)

BUG=573250
R=thestig@chromium.org
TBR=thestig@chromium.org

Committed: https://crrev.com/9830789346abd3d8211deff1ebe7a7f5753ba3fc
Cr-Commit-Position: refs/heads/master@{#367255}

Review URL: https://codereview.chromium.org/1555843002 .

Cr-Original-Commit-Position: refs/heads/master@{#367263}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: de5c041e67fa5abee83359b9346ca4896f1723ca
2016-01-01 18:56:37 +00:00
sigbjornf b1270af350 Revert of mac and ios: Build third-party code with -Wall. (patchset #1 id:1 of https://codereview.chromium.org/1555843002/ )
Reason for revert:
ios_Device builder isn't quite ready for -Wall,

 http://build.chromium.org/p/chromium.mac/builders/iOS_Device/builds/33368

breaking compilation.

Original issue's description:
> mac and ios: Build third-party code with -Wall.
>
> This lands the mac and ios build/common.gypi bits of
> https://codereview.chromium.org/1551753002/ (reviewed there)
>
> BUG=573250
> R=thestig@chromium.org
> TBR=thestig@chromium.org
>
> Committed: https://crrev.com/9830789346abd3d8211deff1ebe7a7f5753ba3fc
> Cr-Commit-Position: refs/heads/master@{#367255}

TBR=thestig@chromium.org,thakis@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=573250

Review URL: https://codereview.chromium.org/1553943002

Cr-Original-Commit-Position: refs/heads/master@{#367258}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 02b240dbc9219e8ae755dd99ded199ac7b4bf77f
2016-01-01 06:18:59 +00:00
Nico Weber 36331d364d Linux/CrOS/Cast/Android (and Mac with gn): Build chromium_code with -Wextra.
Lands the -Wextra bits of https://codereview.chromium.org/1551753002/
(reviewed there).

Windows, Mac, and iOS already build chromium_code with -Wextra.  (Mac and iOS
only in the gyp build, not in gn yet.)  This makes the other platforms match
and also adds -Wextra to chromium_code in gn builds.

BUG=573250,572776
R=thestig@chromium.org
TBR=thestig@chromium.org

Review URL: https://codereview.chromium.org/1552863004 .

Cr-Original-Commit-Position: refs/heads/master@{#367256}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: fc84336d83b1e8feb752d7170bdba3f44ff069a4
2016-01-01 03:57:06 +00:00
Nico Weber 82760824b5 mac and ios: Build third-party code with -Wall.
This lands the mac and ios build/common.gypi bits of
https://codereview.chromium.org/1551753002/ (reviewed there)

BUG=573250
R=thestig@chromium.org
TBR=thestig@chromium.org

Review URL: https://codereview.chromium.org/1555843002 .

Cr-Original-Commit-Position: refs/heads/master@{#367255}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 9830789346abd3d8211deff1ebe7a7f5753ba3fc
2016-01-01 03:55:30 +00:00
lukasza 21baabbe3f Rename [Web]PageSerializer[Test|Client|Impl] to ...FrameSerializer...
After refactoring needed to support OOPIFs in save-page-as feature, the
html serializers in blink serialize frames, not pages and therefore
their old names (WebPageSerializer, PageSerializer, etc) do not apply
anymore.  Therefore this CL renames PageSerializer to FrameSerializer
where applicable.

Note that this rename not only covers PageSerializer class, but also
WebPageSerializer, WebPageSerializerClient, PageSerializerTest, test
data files, etc.

Note that FrameSerializer is more correct than HtmlSerializer, because
- FrameSerializer also serializes subresources (i.e. images)
- Even though WebFrameSerializerImpl deals only with HTML documents,
  for now it might make sense to leave it named as WebFrameSerializerImpl:
  - it still takes a Frame as input
  - it implements part of the public WebFrameSerializer class
    (and therefore it is desirable to have a similar name here as well)

BUG=538766,526786

Review URL: https://codereview.chromium.org/1541463002

Cr-Original-Commit-Position: refs/heads/master@{#367227}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 902fcc5c3732cf387828f7e0c10a810e5bf85714
2015-12-31 04:46:34 +00:00
michaelbai 4425b169e9 port alternative_locale_resource to GN.
Suppport to override existing locale resources in dep graph for
android_apk targets.

- Used is_locale_resource in build_config file to indicate that
  android_resources is locale_resource

- Removed all locale_resource in deps graph if
  alternative_locale_resource_dep is specified in android_apk target.

This feature was implemented in gyp.

BUG=565532

Review URL: https://codereview.chromium.org/1554533003

Cr-Original-Commit-Position: refs/heads/master@{#367182}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 37d60adb473ac4e3038bc73c31c564bc0096e80f
2015-12-30 20:48:39 +00:00
michaelbai be311bb2c7 Port android_aapt_path, app_as_shared_lib and alternative_sdk to GN
alternative_sdk_jar is used in proguard

BUG=565532

Review URL: https://codereview.chromium.org/1551863002

Cr-Original-Commit-Position: refs/heads/master@{#367172}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: fd78509dac4927f0406a0947a18c5b15fb0862ee
2015-12-30 19:28:33 +00:00
michaelbai 3abb735179 Move webview_framework_jar to config.gni
As the counterpart of android_sdk_jar, move webview_framework_jar
to config.gni, so it is easy to be overriden in downstream.

BUG=565532

Review URL: https://codereview.chromium.org/1554693002

Cr-Original-Commit-Position: refs/heads/master@{#367167}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: d66326c14bb70a1d61d75b096a9a344bc9a53c1a
2015-12-30 18:59:43 +00:00
Nico Weber e563b8e923 Enable ui_base_unittests to run on swarming on gyp-based bots
Follow-up to https://codereview.chromium.org/1547153002,
to unbreak the isolate_tests step on the ClangToTMacAsan builder.

BUG=98637,503241,572240
TBR=maruel@chromium.org

Review URL: https://codereview.chromium.org/1548173002 .

Cr-Original-Commit-Position: refs/heads/master@{#366900}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 9cc09a13cb2f11cf05cf065b1acd26c1e384d5f0
2015-12-26 15:46:54 +00:00
liushouqun d86e7b07c5 Force locale to C to allow recognizing output from subprocesses in adb_gdb
adb_gdb uses console output for matching running process. In systems with locale
other than EN, the matching will be failed, set the locale to C to fix this.

BUG=

Review URL: https://codereview.chromium.org/1541243003

Cr-Original-Commit-Position: refs/heads/master@{#366864}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 353749a23e10d0a4ff4d4529b9859e31c891c788
2015-12-25 15:16:37 +00:00
thakis f78b30a576 win: Stop disabling compiler warning 4481.
The warning predates C++11. override is no longer a non-standard extension.

BUG=none

Review URL: https://codereview.chromium.org/1543043006

Cr-Original-Commit-Position: refs/heads/master@{#366832}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 67b3d7f0c42f2e5af855feb67de574ffae58a9a8
2015-12-24 18:38:59 +00:00
agrieve 925211e312 GN(android): Add the dependant test apk to the javac classpath
The work-around is to use an intermediate java_library() that both the
test apk and apk-under-test depend on. This change removes the need for
the work-around.

BUG=571678,532905

Review URL: https://codereview.chromium.org/1549543002

Cr-Original-Commit-Position: refs/heads/master@{#366751}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: c23d3b97765e04a997e6602e1b23c11c8721d2a2
2015-12-23 16:25:04 +00:00
agrieve 679b014eb6 GN: Remove asset_location parameter in favour of android_assets()
BUG=547162

Review URL: https://codereview.chromium.org/1533363002

Cr-Original-Commit-Position: refs/heads/master@{#366747}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: e37dbc81379a6018ee85e260daa95da750fabfa1
2015-12-23 15:22:58 +00:00
jam df41a20d76 Suppress erronous TSAN errors in new EDK.
This also addresses a harmless read of uninitialized 2-bytes to quiet the Valgrind error. Also it adds a lock around the usage of initialized_ in RawChannel, even though it's not needed since it's just an optimization to avoid thread hops.

BUG=561803,571735
TBR=glider, thakis

Review URL: https://codereview.chromium.org/1549703002

Cr-Original-Commit-Position: refs/heads/master@{#366727}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 9269e9ad8cb31a90e0ecfa0b24a597776d962d5d
2015-12-23 07:23:19 +00:00
agrieve 4352616b9f Fix build when emma_coverage changes from false->true
by explicitly clearing out hardlinked output file.

BUG=571642

Review URL: https://codereview.chromium.org/1542173002

Cr-Original-Commit-Position: refs/heads/master@{#366675}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 89572b9c9defce673e0ec094b14a0bc9bbe9b8db
2015-12-22 22:47:00 +00:00
agrieve 1aa46d8e9f GN: Set --coverage-dir in bin/run_* script when emma_coverage = true
BUG=

Review URL: https://codereview.chromium.org/1544803003

Cr-Original-Commit-Position: refs/heads/master@{#366674}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 9dff782b5cef8d48a5a89b352a0576aa1074ba83
2015-12-22 22:41:29 +00:00