chromium/src/build.git
3 hours agoFix gs url in logcat dump, and set content type to text/plain master
luqui@chromium.org [Thu, 24 Apr 2014 03:39:52 +0000 (03:39 +0000)]
Fix gs url in logcat dump, and set content type to text/plain

BUG=362787
TBR=navabi@chromium.org

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

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

3 hours ago[Android] Restore old exception types alongside new ones.
jbudorick@chromium.org [Thu, 24 Apr 2014 03:10:23 +0000 (03:10 +0000)]
[Android] Restore old exception types alongside new ones.

BUG=366305

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

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

6 hours agoAdd telemetry tests to Android tester.
navabi@google.com [Thu, 24 Apr 2014 00:55:22 +0000 (00:55 +0000)]
Add telemetry tests to Android tester.

BUG=337829

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

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

7 hours agoFix build/android/enable_asserts.py.
ppi@chromium.org [Wed, 23 Apr 2014 23:28:00 +0000 (23:28 +0000)]
Fix build/android/enable_asserts.py.

The script seems to have been broken in
https://codereview.chromium.org/221823011 (missing device serial when
creating DeviceUtils).

This fixes the script by iterating over all connected devices.

BUG=267773

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

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

8 hours agoFixed host_gcc_version for Android.
fdegans@chromium.org [Wed, 23 Apr 2014 22:14:01 +0000 (22:14 +0000)]
Fixed host_gcc_version for Android.

BUG=360082

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

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

12 hours agoozone: Always remove glib
spang@chromium.org [Wed, 23 Apr 2014 18:28:51 +0000 (18:28 +0000)]
ozone: Always remove glib

TBR=sadrul
TEST=build with chromeos=1 use_ozone=1
BUG=none
NOTRY=true

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

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

13 hours ago[Android] Remove global --gc-sections ldflag from Debug targets.
primiano@chromium.org [Wed, 23 Apr 2014 17:39:35 +0000 (17:39 +0000)]
[Android] Remove global --gc-sections ldflag from Debug targets.

After some major cleanup, all but very few Android targets (i.e.
chrome_shell_apk and chrome_unittests_apk) have stopped depending
on the linker GC section to build succesfully.
This change removes the global --gc-section arg from all Debug targets
in order to prevent future regressions.

BUG=159847
TBR=thakis@chromium.org

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

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

15 hours agoNewly created profiles should have the grey silhouette as avatar.
mlerman@chromium.org [Wed, 23 Apr 2014 15:34:26 +0000 (15:34 +0000)]
Newly created profiles should have the grey silhouette as avatar.
The name should still be automatically generated.

Also, all the avatar icons are becoming squares (31x38 -> 38x38). New resources will be committed and necessary code changes have been made.

BUG=337986
TEST=When creating a new profile from anywhere except the settings. Also, verify avatar icons are still rendered correctly, whether GAIA, the placeholder or a cartoon icon.
page, the automatically assigned avatar should the grey
placeholder (silhouette).

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

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

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

16 hours agoWebRTC: Remove neteq_unittests test.
kjellander@chromium.org [Wed, 23 Apr 2014 14:03:25 +0000 (14:03 +0000)]
WebRTC: Remove neteq_unittests test.

These tests are being removed in favor for
the test for NetEQ v4 that are a part of the
modules_unittests binary.

BUG=webrtc:2996
NOTRY=True

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

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

20 hours agoUpdate WebRtc/libjingle revision from r5831 to r5963.
perkj@chromium.org [Wed, 23 Apr 2014 10:40:20 +0000 (10:40 +0000)]
Update WebRtc/libjingle revision from r5831 to r5963.

This also temporaryly disable all WebRtcBrowserTest on Android.

TBR Ronghua for README file change.
R=henrika@chromium.org, phoglund@chromium.org
TBR=ronghuawu
BUG= 362852

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

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

22 hours agoLinux: Detect host g++ version as well as target g++ version.
thestig@chromium.org [Wed, 23 Apr 2014 08:20:16 +0000 (08:20 +0000)]
Linux: Detect host g++ version as well as target g++ version.

BUG=360082

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

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

23 hours agoRevert of SDK building script now uses package_version to extract. (https://coderevie...
hidehiko@chromium.org [Wed, 23 Apr 2014 07:05:22 +0000 (07:05 +0000)]
Revert of SDK building script now uses package_version to extract. (https://codereview.chromium.org/235603004/)

Reason for revert:
This update breaks PPAPI*NonSfi tests on linux-32bits bot. See also crbug.com/365817

Original issue's description:
> NaCl: Update revision in DEPS, r13018 -> r13062.
>
> Chromium NaCl scripts now uses package_version to extract.
>
> Now that we have split up the toolchain into multiple tars, we should
> use the package versioning script to manage which tars to extract.
> Unfortunately, currently the SDK uses different directory names compared
> to what the NaCl toolchain directories normally use, so we also must map
> and move the toolchain directories to the SDK ones.
>
> TEST= trybots
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=265303

TBR=noelallen@chromium.org,bradnelson@chromium.org,jvoung@chromium.org,mseaborn@chromium.org,dyen@chromium.org
NOTREECHECKS=true
NOTRY=true

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

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

26 hours agoFixing a race condition in landmines.py
smut@google.com [Wed, 23 Apr 2014 04:19:23 +0000 (04:19 +0000)]
Fixing a race condition in landmines.py

Landmines.py had some logic where it would first check if a directory existed before creating it, but this was sometimes causing an exception wherein the file was reported as existing during the os.mkdirs call, even though it was not reported as existing during the os.path.exists call.

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

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

31 hours agoUpload logcat dump to gs instead of stdout
luqui@chromium.org [Tue, 22 Apr 2014 23:04:13 +0000 (23:04 +0000)]
Upload logcat dump to gs instead of stdout

logcat dumps were flooding master4 with gigabytes of logs per day, causing
CQ outages.

BUG=362787

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

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

32 hours agoRemove dependency on --gc-sections for most Android debug targets.
primiano@chromium.org [Tue, 22 Apr 2014 22:08:20 +0000 (22:08 +0000)]
Remove dependency on --gc-sections for most Android debug targets.

Android targets shouldn't rely (at least on Debug) on the linker GC.
This CL refactors gyp files to fix target dependencies
(some of them wrong for components builds due to the .so
split).
The only targets left, at this point, are chrome_shell
and content_browsertests (will take more work).

BUG=159847

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

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

33 hours agoDon't duplicate test server arguments in Android test setup.
davidben@chromium.org [Tue, 22 Apr 2014 21:03:20 +0000 (21:03 +0000)]
Don't duplicate test server arguments in Android test setup.

Most arguments can be passed as-is and the format is self-desribing. Avoid
having to update yet another file to plumb in a new argument.

This fixes --enable-npn, --ssl-key-exchange, and a handful of other SSL
arguments that were since added.

BUG=354132,365733

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

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

36 hours agoWhitespace change!
agable@chromium.org [Tue, 22 Apr 2014 18:29:17 +0000 (18:29 +0000)]
Whitespace change!

For git tools tutorial tech talk.

TBR=iannucci@chromium.org

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

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

36 hours agoNaCl: Update revision in DEPS, r13018 -> r13062.
dyen@chromium.org [Tue, 22 Apr 2014 18:28:04 +0000 (18:28 +0000)]
NaCl: Update revision in DEPS, r13018 -> r13062.

Chromium NaCl scripts now uses package_version to extract.

Now that we have split up the toolchain into multiple tars, we should
use the package versioning script to manage which tars to extract.
Unfortunately, currently the SDK uses different directory names compared
to what the NaCl toolchain directories normally use, so we also must map
and move the toolchain directories to the SDK ones.

TEST= trybots

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

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

46 hours agoCleanup: Simplify enable_plugin gyp flag and remove an obsolete TODO.
thestig@chromium.org [Tue, 22 Apr 2014 08:29:14 +0000 (08:29 +0000)]
Cleanup: Simplify enable_plugin gyp flag and remove an obsolete TODO.

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

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

2 days agoGet sample_app running on Android again
davemoore@chromium.org [Mon, 21 Apr 2014 21:12:08 +0000 (21:12 +0000)]
Get sample_app running on Android again
Second try after first broke build
https://codereview.chromium.org/240003006/

TBR=viettrungluu
BUG=

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

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

2 days agoMake GN build work on Mac again.
brettw@chromium.org [Mon, 21 Apr 2014 20:54:51 +0000 (20:54 +0000)]
Make GN build work on Mac again.

This adds extension detection for lower-case ".s" assembly files (used in NSPR) in addition to the existing handling of uppercase ".S" ones.

Minor fixes for build files for Mac for dependencies and files.

Proper handling of USE_NSS and addition of USE_OPENSSL_CERTS that match the GYP build.

R=rsleevi@chromium.org

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

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

2 days agoadd build targets for layout test with Dr.Memory - add content_shell_crash_service...
zhaoqin@google.com [Mon, 21 Apr 2014 19:43:11 +0000 (19:43 +0000)]
add build targets for layout test with Dr.Memory - add content_shell_crash_service and layout_test_helper

R=bruening@chromium.org
NOTRY=TRUE
BUG=NONE

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

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

3 days agoAdded "load stale" IDS grit variables to IOS whitelist.
rdsmith@chromium.org [Mon, 21 Apr 2014 03:21:19 +0000 (03:21 +0000)]
Added "load stale" IDS grit variables to IOS whitelist.

Needed for http://codereview.chromium.org/207553008 (r264696) to
build on IOS.

BUG=329621
R=rohitrao@chromium.org

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

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

5 days agodisable more stacktrace usage in debug uclibc builds
mostynb@opera.com [Fri, 18 Apr 2014 23:40:16 +0000 (23:40 +0000)]
disable more stacktrace usage in debug uclibc builds

Here are a couple more instances of OutputToStream / Print that are
no longer available in uClibc builds.

BUG=361130

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

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

5 days agoFinalize apk takes zipalign path as argument instead of hardcoding it
yusufo@chromium.org [Fri, 18 Apr 2014 20:22:43 +0000 (20:22 +0000)]
Finalize apk takes zipalign path as argument instead of hardcoding it

zipalign tool can be in sdk/tools or sdk/build-tools. This change looks
for the right place and uses that instead of having it hardcoded inside
android_sdk_root

BUG=364655

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

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

5 days agoRemove desktop auto-login since we don't plan on shipping it.
rohitrao@chromium.org [Fri, 18 Apr 2014 18:21:25 +0000 (18:21 +0000)]
Remove desktop auto-login since we don't plan on shipping it.

Removes the --enable-autologin switch and associated settings UI.  Autologin is
always enabled for Android builds.  Moves the Android-specific AutoLoginPrompter
class into an android/ directory.

iOS also uses the AutoLoginInfoBarDelegate, so that class is left where it is.

BUG=347214

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

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

6 days agoAdd generate_jni template
cjhopman@chromium.org [Fri, 18 Apr 2014 00:50:10 +0000 (00:50 +0000)]
Add generate_jni template

This adds a template for using the jni_generator. Creates the
base_jni_headers target and updates the base target to depend on it and
actually use the sources that required it.

Adds //third_party/android_tools:cpu_features required by one of the
base files that also required jni.

Disable a bunch of third_party targets for Android builds.

BUG=359249

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

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

6 days ago[Android] Move screenshot implementation back into AndroidCommands.
jbudorick@chromium.org [Thu, 17 Apr 2014 23:46:31 +0000 (23:46 +0000)]
[Android] Move screenshot implementation back into AndroidCommands.

BUG=267773

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

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

6 days agoRemove unused input_shlib_path
cjhopman@chromium.org [Thu, 17 Apr 2014 23:16:50 +0000 (23:16 +0000)]
Remove unused input_shlib_path

This path is now only inferred from test_suite_name.
TBR=darin
NOTRY=true

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

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

6 days agoWork on Mac GN build.
brettw@chromium.org [Thu, 17 Apr 2014 20:33:19 +0000 (20:33 +0000)]
Work on Mac GN build.

This fixes a lot of minor mistakes (mostly missing/extra files and flags) for the Mac GN build.

I separated out some clang flags into a config for extra clang warnings. Several of the third party libraries needed to remove this.

Removes the use_nss flag and uses !use_openssl. This is a result of discussion with rsleevi.

Removes extra duplicate net build file from secondary tree.

ui/gesture_events seems to be getting compiled in GN with more strict warnings than in GYP. Rather than fix this, I fixed the warning in the gesture recognizer unit test. It was returning a const copy (the const is pointless when you're copying).

This also removes a bunch of old GYP integration stuff that was left in the GN build.

R=scottmg@chromium.org

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

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

6 days agoIgnore libpng warnings when running aapt crunch.
newt@chromium.org [Thu, 17 Apr 2014 20:28:26 +0000 (20:28 +0000)]
Ignore libpng warnings when running aapt crunch.

process_resources.py currently considers aapt to have failed if it
outputs anything to stderr. Change this to ignore lines printed to
stderr that contain "libpng warning", which is a known non-error
condition.

BUG=364355

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

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

6 days agoRemove all uses of toolkit_use_gtk in the gyp files.
erg@chromium.org [Thu, 17 Apr 2014 19:20:23 +0000 (19:20 +0000)]
Remove all uses of toolkit_use_gtk in the gyp files.

BUG=297026
R=ben@chromium.org, brettw@chromium.org

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

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

6 days agoRevert 264559 "Get sample_app running on Android again"
davemoore@chromium.org [Thu, 17 Apr 2014 18:19:07 +0000 (18:19 +0000)]
Revert 264559 "Get sample_app running on Android again"

> Get sample_app running on Android again
>
> R=viettrungluu@chromium.org, viettrungluu
> BUG=
>
> Review URL: https://codereview.chromium.org/240003006

TBR=davemoore@chromium.org

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

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

6 days agoGet sample_app running on Android again
davemoore@chromium.org [Thu, 17 Apr 2014 17:26:37 +0000 (17:26 +0000)]
Get sample_app running on Android again

R=viettrungluu@chromium.org, viettrungluu
BUG=

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

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

6 days agoRevert accidental dartium code push
jamescook@chromium.org [Thu, 17 Apr 2014 16:50:56 +0000 (16:50 +0000)]
Revert accidental dartium code push

This reverts r264517 to r264538 from terry@google.com

BUG=none
TEST=none
NOTRY=true
NOTREECHECKS=true
TBR=terry@google.com

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

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

6 days agoAdd blink_tests to the dartium_builder target
terry@google.com [Thu, 17 Apr 2014 15:55:25 +0000 (15:55 +0000)]
Add blink_tests to the dartium_builder target

tbr=vsm@google.com

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

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

6 days agoThis fixes debugging chrome apps from the dart editor.
terry@google.com [Thu, 17 Apr 2014 15:55:21 +0000 (15:55 +0000)]
This fixes debugging chrome apps from the dart editor.

R=devoncarew@google.com

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

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

6 days agoUpdate dartium_builder target in gyp file.
terry@google.com [Thu, 17 Apr 2014 15:55:18 +0000 (15:55 +0000)]
Update dartium_builder target in gyp file.

TBR=jacobr@google.com

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

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

6 days agomerged 1908
terry@google.com [Thu, 17 Apr 2014 15:55:16 +0000 (15:55 +0000)]
merged 1908

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

6 days agoBranching for 1847 @251904
terry@google.com [Thu, 17 Apr 2014 15:55:13 +0000 (15:55 +0000)]
Branching for 1847 @251904

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

6 days agoAdd blink_tests to the dartium_builder target
terry@google.com [Thu, 17 Apr 2014 15:55:06 +0000 (15:55 +0000)]
Add blink_tests to the dartium_builder target

tbr=vsm@google.com

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

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

6 days agoThis fixes debugging chrome apps from the dart editor.
terry@google.com [Thu, 17 Apr 2014 15:55:01 +0000 (15:55 +0000)]
This fixes debugging chrome apps from the dart editor.

R=devoncarew@google.com

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

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

6 days agoUpdate dartium_builder target in gyp file.
terry@google.com [Thu, 17 Apr 2014 15:54:50 +0000 (15:54 +0000)]
Update dartium_builder target in gyp file.

TBR=jacobr@google.com

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

Conflicts:
build/all.gyp

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

7 days agoDelete unused python_ver gyp variable after PyAuto removal.
thestig@chromium.org [Thu, 17 Apr 2014 06:05:59 +0000 (06:05 +0000)]
Delete unused python_ver gyp variable after PyAuto removal.

BUG=224072

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

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

7 days agogyp performance: don't invoke python to check dir existance
bratell@opera.com [Wed, 16 Apr 2014 21:52:49 +0000 (21:52 +0000)]
gyp performance: don't invoke python to check dir existance

We spend a few tenths of a second every gyp invocation starting
the dir_exists.py program. Seems a bit unnecessary.

BUG=362075
R=scottmg@chromium.org

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

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

7 days agoRename IDS ids to be more consistent in ui_strings.grd.
tnagel@chromium.org [Wed, 16 Apr 2014 21:39:27 +0000 (21:39 +0000)]
Rename IDS ids to be more consistent in ui_strings.grd.

This fixes the awkward sorting in grit_whitelist.txt and had been suggested by
Bartosz some time ago.

BUG=none

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

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

7 days agowin: When use_goma=1, favor fastbuild=1 over win_z7=1.
thakis@chromium.org [Wed, 16 Apr 2014 21:11:46 +0000 (21:11 +0000)]
win: When use_goma=1, favor fastbuild=1 over win_z7=1.

BUG=332697
NOTRY=true

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

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

7 days agogyp performance: Add a DoMain version of build/compiler_version
bratell@opera.com [Wed, 16 Apr 2014 20:59:15 +0000 (20:59 +0000)]
gyp performance: Add a DoMain version of build/compiler_version

No need to start a full python interpreter for compiler_version.py. This saves about 0.6 CPU seconds.

BUG=362075
R=scottmg@chromium.org

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

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

7 days agoFill out the rest of the net targets in the GN build except Android.
brettw@chromium.org [Wed, 16 Apr 2014 20:34:31 +0000 (20:34 +0000)]
Fill out the rest of the net targets in the GN build except Android.

This also fixes some missing base files and libresolv naming.

R=jar@chromium.org

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

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

7 days agoMove instrumented-libraries-related settings out of common.gypi.
earthdok@chromium.org [Wed, 16 Apr 2014 19:23:26 +0000 (19:23 +0000)]
Move instrumented-libraries-related settings out of common.gypi.

Instead, apply them as direct_dependent_settings to dependents of the
instrumented_libraries target. This is functionally equivalent, but reduces code
duplication, improves locality and also allows us to disable the use of
instrumented libraries on a target-by-target basis.

BUG=313751
TBR=glider@chromium.org
TEST=MSan
NOTRY=true

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

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

7 days agoozone: evdev: Add libgestures bindings for touchpad & mouse support
spang@chromium.org [Wed, 16 Apr 2014 19:04:30 +0000 (19:04 +0000)]
ozone: evdev: Add libgestures bindings for touchpad & mouse support

The gestures library converts kernel input events into gestures. This
currently happens inside X11 in the xf86-input-cmt driver, but without
X11 we have to do use it from EventFactoryEvdev

BUG=351168

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

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

7 days agoMake NSS work in GN build on Windows.
brettw@chromium.org [Wed, 16 Apr 2014 18:29:50 +0000 (18:29 +0000)]
Make NSS work in GN build on Windows.

This also adds sqlite which is a dependency.

This required some build config changes since nss needs to opt-out of certain default settings.

BUG=
R=scottmg@chromium.org

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

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

7 days agoSplit linux_use_gold_binary into linux_use_bundled_gold and linux_use_bundled_binutils.
sbc@chromium.org [Wed, 16 Apr 2014 09:50:23 +0000 (09:50 +0000)]
Split linux_use_gold_binary into linux_use_bundled_gold and linux_use_bundled_binutils.

linux_use_bundled_binutils adds the binutils to cflags.
linux_use_bundled_gold adds it to the ldflags.  The former
does not work when cross compiling to ARM as the other
binutils do not currently support ARM.  This was causing the
ARM cross compile (for which we do want use the bundled
gold) to break.

I think the word "bundled" works better than "binary" since
it distinguishes from the system gold (which is also a binary).

BUG=363661
TEST=linux_arm_cross_compile trybot

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

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

7 days agowhitespace change to trigger a new build cycle
jochen@chromium.org [Wed, 16 Apr 2014 08:38:59 +0000 (08:38 +0000)]
whitespace change to trigger a new build cycle

BUG=none
TBR=phajdan.jr@chromium.org

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

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

8 days agoenvsetup: Stop adding third_party/android_tools/sdk/tools to PATH.
thakis@chromium.org [Wed, 16 Apr 2014 03:03:48 +0000 (03:03 +0000)]
envsetup: Stop adding third_party/android_tools/sdk/tools to PATH.

If people want to run many binaries from that folder locally, they can add
that folder to their PATH locally.

Depends on:
* https://chrome-internal-review.googlesource.com/160476
* https://codereview.chromium.org/236313004/

BUG=330631
NOTRY=true

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

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

8 days agoMatch only make in gyp_chromium excludes.
bungeman@chromium.org [Tue, 15 Apr 2014 20:34:51 +0000 (20:34 +0000)]
Match only make in gyp_chromium excludes.

The current test also matches the CMake generator.

R=thakis@chromium.org

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

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

8 days agoImplement net in GN build.
brettw@chromium.org [Tue, 15 Apr 2014 19:26:44 +0000 (19:26 +0000)]
Implement net in GN build.

This does the net target. I started doing some more that are commented out at the bottom, and there are a few more after that do do. The deps of these targets started to build up to an unreasonable level and this should be a good checkpoint.

Fix grit rule.

Fix SSL dependent configs

Add TLD cleanup

Add gconf and gio targets for Linux.

Add sources filtering for ChromeOS, .rc, and .mm files. Remove built-in code that removes .rc and .mm files (this wasn't quite complete so causes problems).

BUG=
R=scottmg@chromium.org

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

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

8 days agoFix provision_devices after the DeviceUtils patch.
jbudorick@chromium.org [Tue, 15 Apr 2014 19:01:12 +0000 (19:01 +0000)]
Fix provision_devices after the DeviceUtils patch.

BUG=363702
NOTRY=true

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

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

8 days agoSet the default ASan options for executables built with ASan on Linux.
glider@chromium.org [Tue, 15 Apr 2014 18:39:20 +0000 (18:39 +0000)]
Set the default ASan options for executables built with ASan on Linux.

This is a copy of https://codereview.chromium.org/201153007 and https://codereview.chromium.org/213113002
updated to the current trunk with more descriptive comments in sanitizer_options.cc and the additional
legacy_pthread_cond=1 option for the Linux Official builds (legacy_pthread_cond is to be deprecated,
thus we don't enable it on the bots).

This CL introduces a module, base/debug/sanitizer_options.cc, which will override
the defaults for various dynamic tools (only ASan at this moment). For every executable
built with a dynamic tool this module will be linked into that executable, providing
weak functions to be called by the tool.

The existing declaration of __asan_default_options() in chrome/app/chrome_exe_main_gtk.cc
has been moved into sanitizer_options.cc (now every binary built with GOOGLE_CHROME_BUILD=1
will have the same options as google-chrome-asan.
The existing declaration of __asan_default_options() in chrome/nacl/nacl_helper_linux.cc
has been kept as is, but we had to remove -Wl,-u_sanitizer_options_link_helper to avoid picking sanitizer_options.cc.

The default options target is deliberately disabled on 32-bit Chromium OS builds,
where one of the host binaries (mksnapshot.ia32) is 32-bit despite host_arch==x86_64.

GYP changes for OSX and iOS will be committed separately.

TBR=thakis@chromium.org,mseaborn@chromium.org
BUG=302040

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

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

8 days agoUpdate ARM sysroot image.
sbc@chromium.org [Tue, 15 Apr 2014 17:08:10 +0000 (17:08 +0000)]
Update ARM sysroot image.

This new image include libexif and a few security fixes
from Ubuntu.

Also, cleanup the downloading code so it fails nicely.

R=thestig@chromium.org

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

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

8 days agoMake resource repacker work on Windows
scottmg@chromium.org [Tue, 15 Apr 2014 12:51:25 +0000 (12:51 +0000)]
Make resource repacker work on Windows

Add pragma compatible with cl, and make parser less strict
about the stuff around whitelisted_resource_NNNN.

R=aurimas@chromium.org,thakis@chromium.org
BUG=338759

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

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

9 days agoAdd SSL support to the GN build
brettw@chromium.org [Mon, 14 Apr 2014 23:25:13 +0000 (23:25 +0000)]
Add SSL support to the GN build

This moves files from the secondary tree to the main one, and renames the "meta" one to //crypto:ssl. It also adds the crypto unit tests.

R=rsleevi@chromium.org

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

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

9 days agoRevert 263625 "Set the default ASan options for executables buil..."
fischman@chromium.org [Mon, 14 Apr 2014 15:54:11 +0000 (15:54 +0000)]
Revert 263625 "Set the default ASan options for executables buil..."

Reason for revert: broke all 3 Linux ASan LSan Tests bots:
http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Tests%20%281%29/builds/1338
http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Tests%20%282%29/builds/1563
http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Tests%20%283%29/builds/1796

> Set the default ASan options for executables built with ASan on Linux.
>
> This is a copy of https://codereview.chromium.org/201153007 and https://codereview.chromium.org/213113002
> updated to the current trunk with more descriptive comments in sanitizer_options.cc and the additional
> legacy_pthread_cond=1 option for the Linux builds.
>
> This CL introduces a module, base/debug/sanitizer_options.cc, which will override
> the defaults for various dynamic tools (only ASan at this moment). For every executable
> built with a dynamic tool this module will be linked into that executable, providing
> weak functions to be called by the tool.
>
> The existing declaration of __asan_default_options() in chrome/app/chrome_exe_main_gtk.cc
> has been moved into sanitizer_options.cc (now every binary built with GOOGLE_CHROME_BUILD=1
> will have the same options as google-chrome-asan.
> The existing declaration of __asan_default_options() in chrome/nacl/nacl_helper_linux.cc
> has been kept as is, but we had to remove -Wl,-u_sanitizer_options_link_helper to avoid picking sanitizer_options.cc.
>
> The default options target is deliberately disabled on 32-bit Chromium OS builds, where one of the host binaries (mksnapshot.ia32) is 32-bit despite host_arch==x86_64.
>
> GYP changes for OSX and iOS will be committed separately.
>
> BUG=302040,341805
> TBR=thakis@chromium.org,mseaborn@chromium.org
>
> Review URL: https://codereview.chromium.org/230263003

TBR=glider@chromium.org

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

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

9 days agoSet the default ASan options for executables built with ASan on Linux.
glider@chromium.org [Mon, 14 Apr 2014 13:56:35 +0000 (13:56 +0000)]
Set the default ASan options for executables built with ASan on Linux.

This is a copy of https://codereview.chromium.org/201153007 and https://codereview.chromium.org/213113002
updated to the current trunk with more descriptive comments in sanitizer_options.cc and the additional
legacy_pthread_cond=1 option for the Linux builds.

This CL introduces a module, base/debug/sanitizer_options.cc, which will override
the defaults for various dynamic tools (only ASan at this moment). For every executable
built with a dynamic tool this module will be linked into that executable, providing
weak functions to be called by the tool.

The existing declaration of __asan_default_options() in chrome/app/chrome_exe_main_gtk.cc
has been moved into sanitizer_options.cc (now every binary built with GOOGLE_CHROME_BUILD=1
will have the same options as google-chrome-asan.
The existing declaration of __asan_default_options() in chrome/nacl/nacl_helper_linux.cc
has been kept as is, but we had to remove -Wl,-u_sanitizer_options_link_helper to avoid picking sanitizer_options.cc.

The default options target is deliberately disabled on 32-bit Chromium OS builds, where one of the host binaries (mksnapshot.ia32) is 32-bit despite host_arch==x86_64.

GYP changes for OSX and iOS will be committed separately.

BUG=302040,341805
TBR=thakis@chromium.org,mseaborn@chromium.org

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

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

10 days agoRemove the no-longer needed src/third_party/gold.
mithro@mithis.com [Sun, 13 Apr 2014 17:34:23 +0000 (17:34 +0000)]
Remove the no-longer needed src/third_party/gold.

BUG=352046,362119

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

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

11 days agoMove helper utilities for the DOM Distiller Viewer to core.
nyquist@chromium.org [Sat, 12 Apr 2014 10:45:51 +0000 (10:45 +0000)]
Move helper utilities for the DOM Distiller Viewer to core.

This CL moves the logic for creating HTML content, and retrieving resources
from the content/ subfolder of the component to core/.

It also provides a helper method for calling the right method on the
DomDistillerService based on a URL.

BUG=319881
TBR=jochen@chromium.org
TBR=jshin@chromium.org

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

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

12 days agoFix two typos in the documentation of some args of GN.
tfarina@chromium.org [Sat, 12 Apr 2014 06:47:18 +0000 (06:47 +0000)]
Fix two typos in the documentation of some args of GN.

1- s/useing/using
2- s/SHell/Shell

This was found running the following command line and reading the docs.

$ gn args

BUG=None
TBR=brettw@chromium.org
NOTRY=true

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

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

12 days ago[Android] Fix UIAutomator, Linker, and downstream tests.
jbudorick@chromium.org [Sat, 12 Apr 2014 05:26:08 +0000 (05:26 +0000)]
[Android] Fix UIAutomator, Linker, and downstream tests.

BUG=267773

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

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

12 days agoozone: Initial test blacklist for ChromeOS build
spang@chromium.org [Sat, 12 Apr 2014 05:16:31 +0000 (05:16 +0000)]
ozone: Initial test blacklist for ChromeOS build

We are missing some IME & event functionality. Disable those tests until
we've implemented them, so that we can run more tests.

BUG=361341

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

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

12 days agoRemove stale GPU code on Windows/Mac
piman@chromium.org [Sat, 12 Apr 2014 05:09:08 +0000 (05:09 +0000)]
Remove stale GPU code on Windows/Mac

- on Windows: with Aura + removal of the accelerated surface support, the only
"transport"-type surfaces are TextureImageTransportSurfaces and
IOSurfaceImageTransportSurface. The GpuHostMsg_AcceleratedSurface* messages can
go directly to the UI thread without the need for filtering on the IO thread.
- on both Mac and Windows, we use the compositor thread always, so we don't need
the ViewMsg_SwapBuffers_ACK and associated logic any more.

BUG=None

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

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

12 days ago[Android] Clean up a few uses of the not-yet-existent DeviceUtils interface.
jbudorick@chromium.org [Sat, 12 Apr 2014 03:16:11 +0000 (03:16 +0000)]
[Android] Clean up a few uses of the not-yet-existent DeviceUtils interface.

BUG=267773

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

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

12 days agoRemove a few dead MSVS_VERSION checks.
thakis@chromium.org [Sat, 12 Apr 2014 03:07:46 +0000 (03:07 +0000)]
Remove a few dead MSVS_VERSION checks.

We only support 2013.

TBR=robertshield@chromium.org
BUG=340358

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

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

12 days agoCheckpoint work to get GN builds working on Android.
dpranke@chromium.org [Sat, 12 Apr 2014 01:19:16 +0000 (01:19 +0000)]
Checkpoint work to get GN builds working on Android.

This fixes enough issues so that we can compile and link some libraries, but
it is not yet a working build.

Work remaining:
- base needs JNI support ('base_jni_headers', etc.)
- we need to figure out what should link and/or be APKs to run the tests
- we need to adjust GN to only build in the host config, not the target config.

R=brettw@chromium.org, cjhopman@chromium.org
TBR=jam
BUG=360936

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

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

12 days agoMake landmines.py ignore eclipse gyp generator.
newt@chromium.org [Sat, 12 Apr 2014 01:13:21 +0000 (01:13 +0000)]
Make landmines.py ignore eclipse gyp generator.

The eclipse gyp generator is used to generate eclipse project files, not
to actually build anything. Landmines should return early if the gyp
generator is eclipse rather than failing with the error "Unexpected
GYP_GENERATORS (eclipse)"

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

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

12 days agoGN Windows build fixes
brettw@chromium.org [Fri, 11 Apr 2014 23:06:17 +0000 (23:06 +0000)]
GN Windows build fixes

This also removes some patterns from the sources assignment filter, and adds scary comments not to make it bigger. The GYP version of this is out of control so I want to set a clear policy of what is included and not.

I removed X-related stuff from the filter (there are only about 50 files around the tree) and added manual rules for the affected files.

BUG=
R=scottmg@chromium.org

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

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

12 days agoFix get_device_configuration.py/device_utils.py
dtrainor@chromium.org [Fri, 11 Apr 2014 19:49:52 +0000 (19:49 +0000)]
Fix get_device_configuration.py/device_utils.py

- Fix get_device_configuration.py to use build_device.GetAttachedDevices
without the () as it is now a list not a method since () was added at
build_device.GetAttachedDevices
- Fix device_utils.py to check for unicode as well as str since the config
push_libraries was loading had unicode for the device id

NOTRY=true
BUG=267773

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

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

12 days agoRevert 263262 "Newly created profiles should have the grey silho..."
viettrungluu@chromium.org [Fri, 11 Apr 2014 18:11:26 +0000 (18:11 +0000)]
Revert 263262 "Newly created profiles should have the grey silho..."

Adds static initializers. See:
http://build.chromium.org/p/chromium/builders/Linux%20x64/builds/63585

# profile_avatar_icon_util.cc profiles::kNoHighResAvatar
# profile_avatar_icon_util.cc profiles::kDefaultAvatarIconResourceFileNames+0xd0

> Newly created profiles should have the grey silhouette as avatar.
> The name should still be automatically generated.
>
> Also, all the avatar icons are becoming squares (31x38 -> 38x38). New resources will be committed and necessary code changes have been made.
>
>
> BUG=337986
> TEST=When creating a new profile from anywhere except the settings. Also, verify avatar icons are still rendered correctly, whether GAIA, the placeholder or a cartoon icon.
> page, the automatically assigned avatar should the grey
> placeholder (silhouette).
>
> Review URL: https://codereview.chromium.org/212603011

TBR=mlerman@chromium.org

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

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

12 days agoAdd support for substring replacement in gypi_to_gn
brettw@chromium.org [Fri, 11 Apr 2014 17:23:40 +0000 (17:23 +0000)]
Add support for substring replacement in gypi_to_gn

Use substring replacement in skia source file list, and update the skia BUILD.gn file accordingly. The .gn file gets a bit simpler now: we can do the substring replacement to make absolute paths rather than doing rebase for all the files.

R=djsollen@google.com, dpranke@chromium.org

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

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

12 days agoNewly created profiles should have the grey silhouette as avatar.
mlerman@chromium.org [Fri, 11 Apr 2014 16:51:28 +0000 (16:51 +0000)]
Newly created profiles should have the grey silhouette as avatar.
The name should still be automatically generated.

Also, all the avatar icons are becoming squares (31x38 -> 38x38). New resources will be committed and necessary code changes have been made.

BUG=337986
TEST=When creating a new profile from anywhere except the settings. Also, verify avatar icons are still rendered correctly, whether GAIA, the placeholder or a cartoon icon.
page, the automatically assigned avatar should the grey
placeholder (silhouette).

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

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

12 days agoUse -fuse-ld=gold when supported to select gold.
mithro@mithis.com [Fri, 11 Apr 2014 08:05:11 +0000 (08:05 +0000)]
Use -fuse-ld=gold when supported to select gold.

Allow usage of gold on gcc 4.8 systems such as debian.

BUG=352046

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

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

12 days agoUpdating the gold location.
mithro@mithis.com [Fri, 11 Apr 2014 07:56:03 +0000 (07:56 +0000)]
Updating the gold location.

BUG=362119

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

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

13 days agoFix screenshot.py
cjhopman@chromium.org [Fri, 11 Apr 2014 03:00:42 +0000 (03:00 +0000)]
Fix screenshot.py

build/android/screenshot.py was creating a DeviceUtils. Then grabbing
it's old_interface (i.e. an AndroidCommands) and calling screenshot on
that. That then called TakeScreenshot from
build/android/pylib/screenshot.py passing itself. But
build/android/pylib/screenshot.py expects a DeviceUtils, not an
AndroidCommands. So wrap the AndroidCommands back into a DeviceUtils
first.

build/android/screenshot.py could probably just call into
build/android/pylib/screenshot.py directly.

TBR=jbudorick
NOTRY=true

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

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

13 days agoAndroid: Fix default filename for kTraceStartup file
sievers@google.com [Fri, 11 Apr 2014 01:28:54 +0000 (01:28 +0000)]
Android: Fix default filename for kTraceStartup file

Putting chrometrace.log in the current working directory is not
useful on Android. Generate a filepath on the sdcard instead the
same way we do it for intent triggered tracing.

R=piman@chromium.org, wangxianzhu@chromium.org

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

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

13 days agoenvsetup: Unbreak android_gyp for a while.
thakis@chromium.org [Fri, 11 Apr 2014 00:28:35 +0000 (00:28 +0000)]
envsetup: Unbreak android_gyp for a while.

r262292 accidentally broke android_gyp. Unbreak it for a while, by using
an env var with a long name that hopefully nobody will refer to from anywhere
in the 3-4 weeks it's going to be in the tree.

BUG=330631
NOTRY=true

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

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

13 days agoAdd GN buildfile for gfx
brettw@chromium.org [Thu, 10 Apr 2014 21:16:59 +0000 (21:16 +0000)]
Add GN buildfile for gfx

Adds libjpeg and some global flags for Ozone from the GYP build.

Improves error messages for mismatched item types I noticed when writing this patch (changes in builder.cc).

BUG=
R=bradnelson@google.com

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

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

13 days agoAdd ui/wm to aura_builder target
oshima@chromium.org [Thu, 10 Apr 2014 21:13:48 +0000 (21:13 +0000)]
Add ui/wm to aura_builder target
 This is to mitigate the issue crbug.com/354775 by at least compiling wm_core_unittsts on bots.

BUG=354775
R=jamescook@chromium.org

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

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

13 days ago[Android] Change object types from AndroidCommands to DeviceUtils in build/android/.
jbudorick@chromium.org [Thu, 10 Apr 2014 19:37:30 +0000 (19:37 +0000)]
[Android] Change object types from AndroidCommands to DeviceUtils in build/android/.

BUG=267773

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

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

13 days agoadd new test targets in chromium_builder_dbg_drmemory_win for DrMemory build
zhaoqin@google.com [Thu, 10 Apr 2014 19:35:34 +0000 (19:35 +0000)]
add new test targets in chromium_builder_dbg_drmemory_win for DrMemory build
- add gpu_unittests

R=bruening@chromium.org
NOTRY=true

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

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

13 days agoMake sure binutils values are always defined.
mithro@mithis.com [Thu, 10 Apr 2014 14:01:57 +0000 (14:01 +0000)]
Make sure binutils values are always defined.

BUG=352046

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

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

13 days agoFix variables expansion in build/mac/asan.gyp
glider@chromium.org [Thu, 10 Apr 2014 14:01:18 +0000 (14:01 +0000)]
Fix variables expansion in build/mac/asan.gyp

BUG=344836, 235466
TBR=thakis@chromium.org

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

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

13 days ago[Android]: Update Arm64 and x86_64 build targets to use the experimental ndk.
rmcilroy@chromium.org [Thu, 10 Apr 2014 13:42:46 +0000 (13:42 +0000)]
[Android]: Update Arm64 and x86_64 build targets to use the experimental ndk.

BUG=354405,346626

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

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

2 weeks agoFix the condition to disable the warning when building with Xcode
sdefresne@chromium.org [Wed, 9 Apr 2014 20:53:00 +0000 (20:53 +0000)]
Fix the condition to disable the warning when building with Xcode

The warning -Wshorten-64-to-32 is automatically enabled by Xcode 5.1
when building for 64-bit architecture. Disable when the generator is
Xcode, not ninja.

BUG=359107

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

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

2 weeks agoAdd bookmark undo manager commands to the ios grit whitelist.
erikchen@chromium.org [Wed, 9 Apr 2014 20:41:47 +0000 (20:41 +0000)]
Add bookmark undo manager commands to the ios grit whitelist.

This is in preparation for bringing the bookmark undo manager to ios.

BUG=NONE

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

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

2 weeks agoWhen "linux_use_gold_binary" is set, we now use gold from third_party/binutils rather...
mithro@mithis.com [Wed, 9 Apr 2014 20:20:45 +0000 (20:20 +0000)]
When "linux_use_gold_binary" is set, we now use gold from third_party/binutils rather than third_party/gold (allowing us to remove third_party/gold dependency).

As third_party/binutils is 2.24, this also satisfies the binutils version requirements for Debug Fission. Thus, when building with clang on Ubuntu Precise it should now be enabled. This should cause a significant speedup in linking for most chrome developers on Linux.

Requires;
 * https://codereview.chromium.org/209853003/ - Adding binutils as a DEPS to allow DebugFission on Ubuntu Precise when compiling with clang.

BUG=352046

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

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

2 weeks agoAdd Skia to the GN build.
brettw@chromium.org [Wed, 9 Apr 2014 19:55:29 +0000 (19:55 +0000)]
Add Skia to the GN build.

This makes the GN build of Skia compile and adds it to the GN build.
Previously, the configuration of the GN Skia tried to match the confusing array
of skia targets. This new version dispenses with that and just adds all files
(except SSE ones) to one target. I'm not even sure if it's necessary to split
out the SSE ones, but it seems nice since that target will get more complicated
when we add Arm and MIPS support.

I audited the defines again and added some to the main build config that had
been added to the GYP build recently.

This improves some operator error messages that I found confusing when I had errors in my file.

BUG=
R=djsollen@google.com, scottmg@chromium.org

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

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

2 weeks agoSplit toolchain update and install lookup to make generation faster
scottmg@chromium.org [Wed, 9 Apr 2014 01:56:20 +0000 (01:56 +0000)]
Split toolchain update and install lookup to make generation faster

runhooks does toolchain update, saving relevant information to
build/win_toolchain.json, and during generation this is loaded
and used. This worked almost the same previously, however update
was called at generation time and the .json was deleted after
it was used. This speeds up generation time by a few seconds.

(In preference to https://codereview.chromium.org/228093002/)

R=dpranke@chromium.org
BUG=360878

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

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

2 weeks agoUse instrumented libc++ by default in TSan builds.
glider@chromium.org [Tue, 8 Apr 2014 23:56:09 +0000 (23:56 +0000)]
Use instrumented libc++ by default in TSan builds.

BUG=318770
TBR=earthdok@chromium.org

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

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

2 weeks agoTemplate invocation fixes in GN
brettw@chromium.org [Tue, 8 Apr 2014 22:35:18 +0000 (22:35 +0000)]
Template invocation fixes in GN

This adds an error check when invoking templates which caused really confusing messages if the template invocation encountered an error (because we'd continue running).

Hooks up the provider for programatically defined variables in template invocations so those can be used.

Sets the current directory in a template invocation to be that of the invoking file.

No longer define the target-related programatic variables in an import. Using these in an import will give the directory relative to the import, which is probabyl not what you want.

Fix the Windows build by adding a missing library. Add a warning not to add more to the main list (this added .lib is pretty obscure).

BUG=
R=cjhopman@chromium.org

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

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

2 weeks agoUpdate usb charging command to wait for it to succeed.
qsr@chromium.org [Tue, 8 Apr 2014 21:39:21 +0000 (21:39 +0000)]
Update usb charging command to wait for it to succeed.

R=tonyg@chromium.org,pliard@chromium.org

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

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

2 weeks agoFix the Mac toolchain definition in the GN build.
brettw@chromium.org [Tue, 8 Apr 2014 20:01:51 +0000 (20:01 +0000)]
Fix the Mac toolchain definition in the GN build.

I did not update this when I updated the template syntax.

This also fixes a typo in the GCC toolchain definition (it was just in some helpful assertions and didn't affect anything).

BUG=

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

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