Pull string constants into separate .cc for split dll build

Requires https://codereview.chromium.org/15310002/ before landing.

TBR=brettw@chromium.org
R=cpu@chromium.org
BUG=237249

Review URL: https://chromiumcodereview.appspot.com/15300017

git-svn-id: http://src.chromium.org/svn/trunk/src/build@201264 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

install-build-deps: Install libudev1 for Ubuntu 13.04

Install libudev1 Ubuntu 13.04, keep libudev0 for other Ubuntu releases.
Add 13.04 (racing) codename

BUG=240963
TEST=Tested manually

Review URL: https://chromiumcodereview.appspot.com/14718016

git-svn-id: http://src.chromium.org/svn/trunk/src/build@201259 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Disable intreactive autocomplete and remove command line flag for Android

Disabling this feature on beta and stable channels for now. This change will
be reverted on trunk after it is cherrypicked to the branch.

BUG=239478

Review URL: https://chromiumcodereview.appspot.com/15491005

git-svn-id: http://src.chromium.org/svn/trunk/src/build@201209 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

split ui/base/clipboard constants out for split_link

This is a follow on of work started in https://codereview.chromium.org/15310002/

TBR=ben@chromium.org
R=cpu@chromium.org
BUG=237249

Review URL: https://chromiumcodereview.appspot.com/15271009

git-svn-id: http://src.chromium.org/svn/trunk/src/build@201205 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Fix WebKit Android buildbot script.

Was trying to run a command with a int as an arg.

BUG=239515
TBR=dpranke

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@201147 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

cycle bots

BUG=
TBR=jamesr

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@201146 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

fix location for kMediaStreamSource constants in split link

Requires https://codereview.chromium.org/15310002/ before landing.

R=cpu@chromium.org
BUG=237249

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@201127 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Identify data exports during split_link

Generates proper .def files that include "DATA" so that non-functions are
correctly exported.

However, for split_link because the "other" side won't import them properly
(__declspec(dllimport) is required) split_link notes them, and errors out.

This also adds support for linking individual objs out of libs so that some
_constants.cc and _switches.cc files can easily be linked into both DLLs.

R=cpu@chromium.org
BUG=237249

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@201119 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

GTTF: Convert WebContentsImplBrowserTest to use EmbeddedTestServer.

This eliminates a whole class of problems with SpawnedTestServer
failing to start.

BUG=96594, 230200
R=avi@chromium.org, satorux@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@201099 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

[Android] Fix proguard

Gyp supports only very limited ways of changing behavior based upon
CONFIGURATION_NAME. Particularly, it does not support the way that was
supposed to enable/disable use of proguard.

Instead of trying to switch behavior in gyp, instead pass
CONFIGURATION_NAME, proguard_enabled, and
proguard_enabled_dex_input_path to dex.py and switch the behavior
there.

This also extracts the dex actions into build/android/dex_action.gypi
and uses that for the actions in both build/java.gypi and
build/java_apk.gypi.

BUG=240837

Review URL: https://chromiumcodereview.appspot.com/15231006

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200958 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Linux: remove unmaintained SELinux code.

See https://groups.google.com/a/chromium.org/d/msg/chromium-dev/YpU7t2XaGRg/V9-diQZQQDwJ
for discussion.

BUG=none
R=agl@chromium.org, jln@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200838 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Android WebView: tidy up library dependencies.

The comment in android_webview.gyp about static library dependencies not
being handled correctly is incorrect, and the workaround not necessary:
the problem was just that I wasn't using link_settings correctly. Remove
the workaround and instead add proper dependencies in the places that
actually use cpufeatures and cutils (ashmem). This doesn't change the
generated makefiles but makes more logical sense.

BUG=
R=digit@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200806 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

[android_webview] Fix shell launch script again.

In r195135 I accidentally reverted the fix that was landed in
r192348. This is reverting the accidental revert.

BUG=None
NOTRY=True
R=benm@chromium.org

Review URL: https://chromiumcodereview.appspot.com/15093014

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200776 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

[android_webview] Add a command line setter tool.

This adds a command line setter tool very similar to the ones already
present for content shell and testshell.

BUG=None
TESTS=None
NOTRY=true
R=benm@chromium.org

Review URL: https://chromiumcodereview.appspot.com/15002006

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200775 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Disabling failing/crashing webkit_unit_tests.

It seems like a set of WebKit tests are failing on Android bots and I had to disable them as sheriff. I was not sure who to assign the issue to but trchen@ has done modifications in this are and might be able to reassign to a better resource.

TBR=trchen@chromium.org
BUG=241730

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200774 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Do not use $ANDROID_TOOLCHAIN set by envsetup.sh.

This reverts the changes from commit 6394facfcd6150a69be418212f1caff2f6639540
and instead improves the adb_gdb script to properly auto-detect the right
location of the toolchain binaries.

This means that on a Linux 64-bit systems, it will first try to find the
64-bit binaries, and if not available, will fallback to 32-bit ones.

Tested with the default toolchain from third_party/android_tools/ndk/
(64-bit) and with --ndk-dir=/path/to/android-ndk-r8d (32-bit).

Also remove "gdb_apk" and "gdb_content_shell" which were obsoleted by
"adb_gdb" and "adb_gdb_content_shell", respectively.

BUG=none
R=yfriedman@chromium.org,navabi@chromium.org,wangxianzhu@chromium.org

Review URL: https://chromiumcodereview.appspot.com/14910010

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200547 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Isolate check_internal.py to the .gypi files that need it.

This works around the bug we've had where on some Windows machines, one of the invocations of check_internal.py mysteriously causes a file to open up in Visual Studio.  We have no idea why, but this should avoid the problem.  See details in bug.

BUG=241208

Review URL: https://chromiumcodereview.appspot.com/12330155

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200514 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Use a single Python Forwarder instance for each base test runner instance.

This is achieved by having BaseTestRunner instantiate a single Forwarder
instance and inject it into SpawningServer.

This makes the test server spawner instance use an already configured Forwarder
instance for every single unit test rather than instantiating Forwarder every
time (which also meant unnecessarily running 'adb forward' for every single
unit test).

Note that this CL also makes Forwarder thread-safe. This is due to the fact
that the test server spawner does its work on a worker thread and not to the
fact that a single Forwarder instance can be shared accross multiple shards as
one could think.

BUG=229685,239014
R=bulach@chromium.org, craigdh@chromium.org, frankf@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200500 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

[Android] Pass -p to mkdir commands

These commands should not fail if the parent directory does not exist.
Pass -p so that all the parent directories are also made.

Review URL: https://chromiumcodereview.appspot.com/14795012

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200456 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

[Android] Only always trigger stripping with component build

With the component build we need to always trigger stripping because we
can't specify the inputs correctly at gyp time. This is not required
for the non-component build.

Also, slightly refactor strip_native_libraries.gypi and its users to
make it more clear what variables are being used by the .gypi.

Review URL: https://chromiumcodereview.appspot.com/14844009

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200422 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Move webkit/compositor_bindings into webkit/renderer

The compositor bindings for WebKit should only be used in a render process or in
webkit_support for DumpRenderTree or webkit_unit_tests. It's never used in a
browser process. This moves the bindings code under webkit/renderer/ so we can
enforce this via DEPS rules.

The path is unfortunately a bit awkward. Once we drop DumpRenderTreeSupport and
figure out a way to deal with webkit_unit_tests, we can move the bindings into
content/renderer/gpu/ since they'll only be used by the content embedding.

BUG=239107
TBR=piman@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200350 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Change gpu blacklists from json resource to const string.

Since we no longer do realtime blacklist update, so the need to keep it as a independent json file is no longer there.

BUG=
TEST=content_unitests
R=apatrick@chromium.org, piman@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200315 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Make it possible to run WebGL conformance tests on Android

This patch implements the following changes to make it possible to run
the WebGL conformance test suite on Android:

    1. Copy WebGL test data files to the device.

    2. When running the conformance test, turn on WebGL with the
    --enable-webgl command line flag.

    3. Avoid validating the PCI vendor and device ids for the GPU because
    Android has no such concept.

With these changes, you can run the WebGL conformance test by first
building content_browsertests_apk, removing the WebGL exclusion from
build/android/pylib/gtest/filter/content_browsertests_disabled and then executing:

    $ build/android/run_browser_tests.py -f WebGLConformanceTest.*

BUG=138226

Review URL: https://chromiumcodereview.appspot.com/14944010

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200298 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Generate, merge, and embed manifest when doing split_link

(Requires re-running tools\win\split_link\install_split_link.py)

R=cpu@chromium.org
BUG=237249

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200265 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Add device port unmapping support to forwarder2.

There used not to be any way to unmap a previously forwarded port. Since
the host and device forwarder processes are kept alive during a whole test
suite, both the host and device ended up with an unnecessarily high amount of
unused open sockets. That is source of flakiness (e.g. too many opened file
descriptors).

This adds an UnmapDevicePort() method to the Python Forwarder wrapper class and
modifies the native forwarder2 to accept a negative port on the command line.
When a negative port is provided, the host forwarder tries to unmap that port
(that was mapped previously).

BUG=229685,239014
R=bulach@chromium.org, digit@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200257 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Fix flaky test: Enable RenderWidgetHostViewBrowserTest.CopyFromBackingStore.

Added a spin-wait to cause the test to wait for the browser to create the BackingStore via its normal mechanism.

...and then I went ahead and refactored and cleaned up all the code.  I did not add any significant additional testing.  We will want to do this in a follow-up change.  However, I did get rid of the #ifdef OS_XXX pragmas, so the tests will attempt to run on all platforms, not just Mac and Win, and use the CanXXX() methods to determine which features are supported for testing.

BUG=224351

Review URL: https://chromiumcodereview.appspot.com/14495016

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200226 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Don't tell people they can import their own roots on iOS.

It's not possible at this time.

BUG=240640

Review URL: https://chromiumcodereview.appspot.com/15137010

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200143 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Enable Clang warnings for implementation files on all platforms.

This fixes a few remaining issues for Android builds

BUG=115047

Review URL: https://chromiumcodereview.appspot.com/14876034

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200125 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Remove bzip2 code since it is no longer being used.

BUG=237998
TBR=tommi

Review URL: https://chromiumcodereview.appspot.com/14863012

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200116 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

fix line endings on split_link scripts

(otherwise, presubmit whines about copyright now matching because it
contains \r)

I'm not 100% sure this will fix the problem because something will
do line ending changes, but... we will see.

TBR=cpu@chromium.org
BUG=none

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200079 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

fix license

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200067 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Build chromedriver2 library on the chromedriver waterfall.

It is needed by the python integration tests
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/14657024

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200058 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

split_link tool, config, and scripts for windows build

First pass at split linker. Splits inputs based on json spec, uses .def files
to export symbols in the other half, relinks to generate an import lib, and
finally links again for the real link, using the generated def file and import
lib. And then, repeats those steps until success (not all unresolved externals
are reported by the linker on the first pass). It's necessary to use the
linker to report externals rather than cracking lib/objs because when doing
LTCG, the object files are in an undocumented compiler-internal format and the
exports can't be gathered (and probably aren't fully determined yet).

Currently this approach only handles chrome.dll, not other large targets that
are monolithic exes (rather than mostly in a DLL with an EXE loader).

Integrated with build system by a linker shim. Original link.exe is saved, and
replaced by split_link binary. If "/splitlink" is found on the command line,
then run our script that does the iterations/exporting magic. Otherwise,
fallback to the original linker.

When GYP_DEFINES includes chrome_split_dll=1, the split linker is invoked.

chrome.exe has not yet been modified to know how to load split binaries, so
the build flag will not yet be directly useful for those not working on this
problem.

Release, non-LTCG, non-split:
05/09/2013 04:57 PM 57,447,936 chrome.dll

Release, non-LTCG, split:
05/10/2013 12:47 PM 39,567,872 chrome0.dll
05/10/2013 12:48 PM 19,274,240 chrome1.dll

Release, partial-LTCG (same as current settings), split:
05/10/2013 03:56 PM 25,934,336 chrome0.dll
05/10/2013 04:13 PM 16,347,648 chrome1.dll

It should be possible to get higher optimization levels for chrome1.dll (or
perhaps both parts), but that can happen in subsequent changes after more
testing.

TBR=cpu@chromium.org, maruel@chromium.org
BUG=237249

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200049 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Stop build on failed device status check, apk install and add CheckInstall step.

BUG=230970,236035
R=ilevy@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@200020 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Fix heapchecker shared_library build.

BUG=112389
TBR=rnk@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199965 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Speedup net_unittests on Android by 30%.

The Chrome test server spawner (running on the host) was sleeping way too much.
Note that ideally we should move away from polling but this still sounds like a
reasonable short term improvement.

R=bulach@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199949 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Move ChromeDriver targets out of chromium_builder_qa to their own target.

The ChromeDriver waterfall doesn't need to build PyAuto.
BUG=none

Review URL: https://chromiumcodereview.appspot.com/15149002

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199937 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Remove ENABLE_MESSAGE_CENTER

Next step on re-factoring Notifications. This will bring the MessageCenter classes, most important message_center::Notification, into build on all systems, including those where the MessageCenter is not yet appearign in UI. This will allow to start using this Notification class in client code and remove the old Notification class defined in chrome/browser/notificaitons/notification.h

That will allow the clients that already use Rich Notifications to use richer data type support, for example supply an image bits for a Notification (as in Snapshot notifications).

This also removes a lot of compile-time @ifdefs and replaces them with checking a runtime flag which we already have anyways.

On Android and iOS, the MessageCenter is not compiled in, for the size concerns and uncertain story for notifications in general - the existing ENABLE_NOTIFICATIONS define is used for that.

BUG=174164

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=199625

Review URL: https://chromiumcodereview.appspot.com/15025002

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199920 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

[Android] Only reinstall test apk if needed

When running tests, we should only be reinstalling the test apk if it
is different than the one currently installed. So, before reinstalling,
query the package manager for the path to the installed apk, and then
only reinstall if the md5sum of the installed apk differs from the one
we are about to install.

NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/14601004

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199835 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Support target/host architecture with ninja iOS builds

When iOS builds support ninja and enable GYP_CROSSCOMPILE=1, ninja
will build breakpad, protoc and iossim using host arch instead
of the sub-out-ninja workaround.

TBR=mark@chromium.org
BUG=236517

Review URL: https://chromiumcodereview.appspot.com/14597007

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199827 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Workaround ADB port allocation issue.

BUG=239014
R=bulach@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199711 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Revert 199625 "Remove ENABLE_MESSAGE_CENTER"

Caused a static initializer on Linux:
# message_center_constants.cc __i686.get_pc_thunk.cx

(Not on Mac though. The SkGetColor() macro has a comment about it not adding
a static initializer on release builds, so maybe it's the float-int-multiplication?
I don't have a linux box around to check, sorry.)

> Remove ENABLE_MESSAGE_CENTER
>
> Next step on re-factoring Notifications. This will bring the MessageCenter classes, most important message_center::Notification, into build on all systems, including those where the MessageCenter is not yet appearign in UI. This will allow to start using this Notification class in client code and remove the old Notification class defined in chrome/browser/notificaitons/notification.h
>
> That will allow the clients that already use Rich Notifications to use richer data type support, for example supply an image bits for a Notification (as in Snapshot notifications).
>
> This also removes a lot of compile-time @ifdefs and replaces them with checking a runtime flag which we already have anyways.
>
> On Android and iOS, the MessageCenter is not compiled in, for the size concerns and uncertain story for notifications in general - the existing ENABLE_NOTIFICATIONS define is used for that.
>
> BUG=174164
>
> Review URL: https://chromiumcodereview.appspot.com/15025002

TBR=dimich@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199639 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Remove ENABLE_MESSAGE_CENTER

Next step on re-factoring Notifications. This will bring the MessageCenter classes, most important message_center::Notification, into build on all systems, including those where the MessageCenter is not yet appearign in UI. This will allow to start using this Notification class in client code and remove the old Notification class defined in chrome/browser/notificaitons/notification.h

That will allow the clients that already use Rich Notifications to use richer data type support, for example supply an image bits for a Notification (as in Snapshot notifications).

This also removes a lot of compile-time @ifdefs and replaces them with checking a runtime flag which we already have anyways.

On Android and iOS, the MessageCenter is not compiled in, for the size concerns and uncertain story for notifications in general - the existing ENABLE_NOTIFICATIONS define is used for that.

BUG=174164

Review URL: https://chromiumcodereview.appspot.com/15025002

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199625 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Teach landmines script that Linux now uses ninja by default.

Mostly for documentation purposes. It'll also uselessly cause a clobber.

BUG=239257
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/14940023

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199622 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Move all_webkit inside OS!="ios" block in all.gyp

TBR=abarth@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199620 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Add a target all_webkit that depends on the old all_webkit and content_shell

BUG=111316
R=abarth@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199616 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

cycle bots

BUG=
TBR=

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199605 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Let gyp create ninja files by default on linux.

If a generator is set explicitly (via GYP_GENERATORS or via -f or via
chromium.gyp_env), it will have precedence over the new default.

If you've used make until now, run `ninja -C out/Debug` instead of `make` to
build.

If you're using goma, go/ma has documentation on how to use goma with ninja.
(It's the same as make, except that CC / CXX are now picked up at `gclient sync`
/ `gclient runhooks` / `build/gyp_chromium` time instead of at build time.)

If you can't use ninja for some reason, `export GYP_GENERATORS=make` and sync
again. Please also send me (thakis@chromium.org) an email explaining why ninja
does not work for you.

This will also switch all bots that don't explicitly set a build tool (including
the public bots). compile.py will use ninja instead or make based on if
build.ninja or Makefile are newer, so they should build the right thing
automatically. And since built products end up in the same place, packaging
should do the right thing too.

BUG=239257
R=mark@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199603 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Enable Clang warnings in implementation files for OS X

BUG=115047
R=thakis

Review URL: https://chromiumcodereview.appspot.com/14890022

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199568 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Use $ANDROID_TOOLCHAIN set by envsetup.sh in adb_gdb and gdb_apk

After recent upgradation to third_party/android_tools/ndk, there is no
linux-x86 toolchain directory any more, but linux-x86_64 instead. This
causes failure of adb_gdb and gdb_apk which hard-coded linux-x86
in the scripts. As envsetup.sh already set correct $ANDROID_TOOLCHAIN,
now use it in the scripts instead of the hardcoded one.

BUG=none
NOTRY=true (presubmit passed)

Review URL: https://chromiumcodereview.appspot.com/15028013

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199567 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

[Android] Auto-generate only necessary v14 resources.

V14 Resource auto-generation was introduced by crbug.com/235118.
However, some resources doesn't use v17 attributes, so we don't
need to generate for all resources.

BUG=239742

Review URL: https://chromiumcodereview.appspot.com/14812015

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199564 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

[MIPS] Fix correct toolchain_dir for MIPS/Android builds

After recent change in android_tools to update ndk, the scripts can reuse
android_host_arch variable for MIPS too. This is important, as the release
now contains 64-bit host toolchain set.

BUG=130022
TEST=build content_shell_apk

Review URL: https://chromiumcodereview.appspot.com/15085019

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199492 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

[Android] Warning if Left/Right attributes are used in layout and style xmls.

Since crbug.com/235118, Left/Right attributes are deprecated in favor of Start/End.
So warn if it contains Left/Right.

BUG=239606

Review URL: https://chromiumcodereview.appspot.com/14969022

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199452 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

install-build-deps: Non-english locale + Ubuntu 13.04

Added support for installing dependencies on
non-english Ubuntu 13.04

BUG=239505
TEST=Tested manually

Review URL: https://chromiumcodereview.appspot.com/15085010

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199448 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Small cleanup in build/linux/install-arm-sysroot.py and fix a typo in DEPS.

NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/14647004

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199337 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Update documentation in wrapper script to match the changed behavior in install-debian.wheezy.sysroot.py

NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/15001024

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199321 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Fix static libraries will not be copied to product out directory issue on x86 Android

BUG=None
TEST=Build Chrome for arm and x86 Android, all static libraries are copied to out directory

Review URL: https://chromiumcodereview.appspot.com/14993005

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199267 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

[Android] With gyp_managed_install no device is only a warning

Before running any action that interacts with the device, check that
one is actually attached. If not, print a big warning and skip the action.

BUG=235521
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/13877025

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199263 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

[android] Fix the adb_gdb script.

The ndk under third_party was updated to not contain the 32-bit toolchains
but the script continues to reference those.

BUG=
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/14858018

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199234 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Android: fixes option to exclude specific annotated Android instrumentation tests.

Broken at https://chromiumcodereview.appspot.com/14704006/
Wrong number of parameters in run_uiautomator_tests.py

BUG=169875, 239410
NOTRY=True
TBR=sivachandra@chromium.org,craigdh@chromium.org

Review URL: https://chromiumcodereview.appspot.com/15001004

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199201 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Android: fixes option to exclude specific annotated Android instrumentation tests.

Broken at https://chromiumcodereview.appspot.com/14704006/
If the excluded option is empty, it wasn't running any tests.

BUG=169875,239410
NOTRY=True
TBR=sivachandra@chromium.org,craigdh@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199178 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Enable clang and host_os when generating ninja-ios.

Default clang=1 and host_os==mac when GENERATOR==ninja and OS==ios.

BUG=236517

Review URL: https://chromiumcodereview.appspot.com/14740022

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199171 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Add option to exclude specific annotated Android instrumentation tests.

BUG=169875

Review URL: https://chromiumcodereview.appspot.com/14704006

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199170 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Android: disables WebRTCInternalsBrowserTest.WithRealPeerConnectionCall.

This test was already disabled but the name has changed in r198976.

BUG=181069
NOTRY=True
TBR=nileshagrawal@chromium.org

Review URL: https://chromiumcodereview.appspot.com/15030004

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199165 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

cycle bots

BUG=

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199104 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Update outdated comment.

I made isfinite() c++11-compliant a while ago.

BUG=none
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/15075004

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199067 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Fix and simplify unbundled v8.gyp:

- use shim_headers.gypi instead of expanding it inline
- enable host and target toolsets unconditionally, they're needed
  as deps (and make no difference anyway)

BUG=226860
R=rsleevi@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199063 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Remove inside_chromium_build gyp variable

In the days of olde, we supported compiling from inside a chromium checkout and
inside a WebKit checkout with some chromium libraries. In the latter configuration,
used mainly by the build.webkit.org bots, the paths to the WebKit headers relative
to chromium gyp files was different. This added a lot of complexity and some extra
scripts to fix up paths and includes. Blink only supports building within a chromium
checkout so all of this complexity can be removed.

Review URL: https://chromiumcodereview.appspot.com/14823005

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199057 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Miscellaneous fixes for system libraries files:

- fix icu namespace ambiguity
- fix undeclared variable in libvpx.gyp file
- add gyp target for chromium zip code in the third_party directory

BUG=226860
R=rsleevi@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@199050 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

[Android] Auto-generate v14 style xml resources from the exisitng v17 style resources.

In addition to the previous v14 layout xml auto-generation patch,
(https://codereview.chromium.org/14476011/#ps69001)
this patch adds style xml resource auto-generation.

BUG=235118

Review URL: https://chromiumcodereview.appspot.com/14948008

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198947 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Delete test_shell.

R=jochen@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198933 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Alert on zero online devices when last devices file is missing or empty.

BUG=238535

Review URL: https://chromiumcodereview.appspot.com/14652029

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198931 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Add chrome_split_dll gyp variable, and duplicate chrome_main_dll target (reland)

Currently contains chrome_browser_dll target. Removes some paths from browser->webcore
via content that are there due to single process mode, and removes dependency of content
on content_renderer.

Current remaining paths from chrome_browser_dll to webcore: http://i.imgur.com/7WWRvpA.jpg

Reland of https://chromiumcodereview.appspot.com/14682014

TBR=cpu@chromium.org
BUG=237249

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198928 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Revert 198920 "clang: Turn on C++11 on CrOs too."

Linux Chromium OS ASAN Builder is still not on Precise, apparently.

> clang: Turn on C++11 on CrOs too.
>
> This was blocked on the main waterfall linux_chromeos_clang bot migrating
> to precise, which has happened now.
>
> I believe all bots that build chromeos with clang are on precise now. If any
> turns red, this CL should be reverted.
>
> BUG=none
> R=ajwong@chromium.org
>
> Review URL: https://codereview.chromium.org/14992005

TBR=thakis@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198924 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

clang: Turn on C++11 on CrOs too.

This was blocked on the main waterfall linux_chromeos_clang bot migrating
to precise, which has happened now.

I believe all bots that build chromeos with clang are on precise now. If any
turns red, this CL should be reverted.

BUG=none
R=ajwong@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198920 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Fix unit test runner to exit with error when no devices are attached.

BUG=238474

Review URL: https://chromiumcodereview.appspot.com/14960003

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198914 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Adds cc_perftests to chromium_builder_perf target.

BUG=163720

Review URL: https://chromiumcodereview.appspot.com/14846018

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198911 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

[Android] Remove "Chrome" and "View" personalities

The content layer code is now by default operates in the "Chrome" mode.
All WebView customizations are applied in the android_webview layer.

BUG=b/8296421
R=benm@chromium.org, bulach@chromium.org, jochen@chromium.org, joth@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198877 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

[Android] Move zoom controls management into WebView

Chrome for Android doesn't use zoom controls, so it makes sense
to move all the related code into android_webview.

However, as visibility of zoom controls is controlled from the
internals of ContentView*, we are providing hooks through a new
delegate interface.

BUG=b/8296421
R=benm@chromium.org, joth@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198862 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Convert build/linux/install-debian.wheezy.sysroot.wrapper.sh to python for Windows compatibility.

BUG=224487
TBR=dbeam

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198859 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Use a sysroot for Chrome Linux official builds. (try 2)

BUG=224487
R=mmoss@google.com

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198856 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Revert 198834 "Add chrome_split_dll gyp variable, and duplicate ..."

> Add chrome_split_dll gyp variable, and duplicate chrome_main_dll target
>
> Currently contains chrome_browser_dll target. Removes some paths from browser->webcore
> via content that are there due to single process mode, and removes dependency of content
> on content_renderer.
>
> Current remaining paths from chrome_browser_dll to webcore: http://i.imgur.com/7WWRvpA.jpg
>
> R=cpu@chromium.org
> BUG=237249
>
> Review URL: https://chromiumcodereview.appspot.com/14682014

TBR=scottmg@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198837 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Add chrome_split_dll gyp variable, and duplicate chrome_main_dll target

Currently contains chrome_browser_dll target. Removes some paths from browser->webcore
via content that are there due to single process mode, and removes dependency of content
on content_renderer.

Current remaining paths from chrome_browser_dll to webcore: http://i.imgur.com/7WWRvpA.jpg

R=cpu@chromium.org
BUG=237249

Review URL: https://chromiumcodereview.appspot.com/14682014

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198834 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Linux: Change dump_app_syms script to pass -r to dump_syms. This option was added in Breakpad r1163.

BUG=230995
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/14188058

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198800 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Support GYP_CROSSCOMPILE variable in gyp_helper.

Support setting GYP_CROSSCOMPILE from gyp_env files.

BUG=None

Review URL: https://chromiumcodereview.appspot.com/14652028

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198797 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Turn on ENABLE_PLUGIN_INSTALLATION flag for Aura

BUG=235672
R=cpu@chromium.org
TEST=none

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198780 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Revert 194601 "Android: Enable content_browsertests by default."

> Android: Enable content_browsertests by default.
>
> BUG=138275
>
> Review URL: https://codereview.chromium.org/13445005

TBR=nileshagrawal@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198770 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

[Telemetry] Escape URLs in "Pages:" list.

They must match the trace names printed by PrintPerfResult, which are also
escaped with the same method.

This patch allows the buildbot log processor to work with the updated page
cyclers.

BUG=None
TEST=tools/perf/run_multipage_benchmarks --browser=system page_cycler tools/perf/page_sets/intl_es_fr_pt-BR.json --pageset-repeat=2
NOTRY=True

Review URL: https://chromiumcodereview.appspot.com/14780004

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198765 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Remove use_system_ffmpeg-related logic from media and other mainline gyp files.

BUG=226860
R=fischman@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198746 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Revert "Make IsHostPortUsed() handle ports bound by foreign UIDs."

This is a speculative revert to see if it fixes the flakiness on the Android
testers.

TBR=bulach@chromium.org,shashishekhar@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198735 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Make IsHostPortUsed() handle ports bound by foreign UIDs.

lsof -np is not able (permission denied when running as non-root to be exact)
to retrieve file descriptor information for processes running under a different
UID. This can make IsHostPortUsed() return false when the provided port is
actually used.
Trying to connect to a port rather than binding it should not introduce any
side-effect on the system that could affect next runs as the TODO suggested. It
is also always sane not to depend on an external process (lsof) output format.

BUG=229685
R=bulach@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198683 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Disable IndexedDBBrowserTestWithMissingSSTFile.DestroyTest on Android.

TBR=dgrogan@chromium.org
BUG=233118

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198681 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Add asan=1 gyp defines to asan bot

Fixes asan bot and uses slave_props for extra_gyp_defines.

BUG=160952
TBR=iannucci@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198674 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Switch Asan bot to builder-tester

src/ side of the change.

The data is it transferring is large (2GB)
and it is not used enough to merit the increased
throughput from a split configuration.

BUG=160952
TBR=iannucci@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198673 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Enable Clang warnings in .cc files for Linux+[Aura/ChromeOS]

BUG=115047
R=darin

Review URL: https://chromiumcodereview.appspot.com/14890005

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198649 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Revert 198637 "Use a sysroot for Chrome Linux official builds."

> Use a sysroot for Chrome Linux official builds.
>
> BUG=224487
> R=mmoss@google.com
>
> Review URL: https://codereview.chromium.org/12938016

TBR=thestig@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198643 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Use a sysroot for Chrome Linux official builds.

BUG=224487
R=mmoss@google.com

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198637 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Fix infinite loop caused by ElideText()

BUG=238405
R=ben@chromium.org, mmoss@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198545 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Add content_shell to builder targets that also build DRT

BUG=111316
R=mkwst@chromium.org

Review URL: https://chromiumcodereview.appspot.com/14670012

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198433 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Fix subscripting support.

 Subscripting needs libarc_lite to works. To force compilation with this
library, the linker must have the -f-obj-arc parameter.

R=stuartmorgan@chromium.org

Review URL: https://chromiumcodereview.appspot.com/14850014

git-svn-id: http://src.chromium.org/svn/trunk/src/build@198428 4ff67af0-8c30-449e-8e8b-ad334ec8d88c