chromium/src/base.git
65 min agoBring back the retry limit to 3. master
jam@chromium.org [Wed, 23 Jul 2014 19:02:05 +0000 (19:02 +0000)]
Bring back the retry limit to 3.

This reverts r284650. There is one test that is flaky 34% of the time, but that is the exception and not the norm. We don't need 12 retries to make browser_tests or other test suites not flaky. If there are very flaky tests, we should disable them instead. I've disabled all the tests that flake more than 5% of the time.

BUG=395189
R=brettw@chromium.org, sergeyberezin@chromium.org

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

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

2 hours agoDisable all the tests that are flaking more than 5% on Windows builders.
jam@chromium.org [Wed, 23 Jul 2014 17:20:15 +0000 (17:20 +0000)]
Disable all the tests that are flaking more than 5% on Windows builders.

Data from http://chromium-build-logs.appspot.com/flakiness_dashboard?master=chromium.win.

BUG=386891,387124,396373,396384,396386,396387,396388,396390,396392,387124,396393,396395,396399,396402,396403,396404,396405,393149,396409,95557,396413,396414,375894
R=viettrungluu@chromium.org
TBR=brettw, rvargas, sky

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

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

10 hours agoUpdate the TSan suppressions for uprv_malloc_xx and uprv_realloc_xx
glider@chromium.org [Wed, 23 Jul 2014 10:07:03 +0000 (10:07 +0000)]
Update the TSan suppressions for uprv_malloc_xx and uprv_realloc_xx

BUG=223352
TBR=bruening@chromium.org
NOTRY=true

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

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

19 hours agoAdded an annotation to label Integration tests.
yojan@google.com [Wed, 23 Jul 2014 00:15:01 +0000 (00:15 +0000)]
Added an annotation to label Integration tests.

An IntegrationTest label will be required for tests that may require
a greater timeout value than LargeTest allows. These tests will run
against real instances of other services (e.g. Search). They may use
command-line flags to change the backends that are being tested against.

BUG=None

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

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

24 hours agoGet rid of FileThreadDeserializer.
gab@chromium.org [Tue, 22 Jul 2014 19:30:17 +0000 (19:30 +0000)]
Get rid of FileThreadDeserializer.

Replace it with modern threading constructs:
 - PostTaskAndReplyWithResult gets rid of most of the logic FileThreadDeserializer was implementing.
 - The remainder logic didn't require any class state so it was moved to anonymous methods.

Also declare JsonPrefStore explicitly NonThreadSafe (the only actions outside the UI thread should happen by posting anonymous tasks to the |sequenced_task_runner_|).

This is a stepping stone in cleaning up JsonPrefStore to eventually get rid of PrefStore's ref-counting scheme.

BUG=393081

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

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

27 hours agoclang/win: Fix calling delete[] on void* in base.
thakis@chromium.org [Tue, 22 Jul 2014 16:56:22 +0000 (16:56 +0000)]
clang/win: Fix calling delete[] on void* in base.

Calling delete[] on void* is undefined behavior. Cast to the right
type before calling delete[]. Fixes a clang warning.

BUG=82385
TBR=siggi

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

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

27 hours agoModify TraceEventTestFixture.TraceBufferVectorReportFull to assert the overflowed_at_...
nednguyen@google.com [Tue, 22 Jul 2014 16:33:37 +0000 (16:33 +0000)]
Modify TraceEventTestFixture.TraceBufferVectorReportFull to assert the overflowed_at_ts is between the timestamp of last event and current trace_log's time.

BUG=395739

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

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

32 hours agoRevert of High resolution timer fix for Windows (https://codereview.chromium.org...
ksakamoto@chromium.org [Tue, 22 Jul 2014 11:32:40 +0000 (11:32 +0000)]
Revert of High resolution timer fix for Windows (https://codereview.chromium.org/395913006/)

Reason for revert:
This patch seems to make following browser_tests flakey

PPAPINaClNewlibTest.Graphics2D_FlushOffscreenUpdate
NetInternalsTest.netInternalsHSTSViewAddOverwrite
NetInternalsTest.netInternalsHSTSViewAddDelete
NetInternalsTest.netInternalsHSTSViewAddTwice

http://build.chromium.org/p/chromium.win/builders/XP%20Tests%20%282%29/builds/34734
http://build.chromium.org/p/chromium.win/builders/XP%20Tests%20%281%29/builds/32086
http://build.chromium.org/p/chromium.win/builders/Vista%20Tests%20%281%29/builds/47545

Original issue's description:
> This is jamesr@ code I am landing.
>
> On Windows the message pump code tried to manage the systemwide timer resolution to fire delayed tasks with better than 15ms resolution but it was buggy:
>
> 1- A short task that was not followed by any other task will leave the systemwide timer pegged to 1ms
>
> 2- After we decided to crank up the timer we would 'lease' the timer for 1 second, for no good reason.
>
> Both issues are detrimental to battery power.
>
> The source of both problems is that we tried to decide with incomplete information. This patch solves that by having 1 bit for each pending task that requires a high resolution timer and a sum of the number of tasks that require high res timers.
>
> BUG=153139
> TEST=included here, also see the bug for manual testing.
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=284625

TBR=jamesr@chromium.org,darin@chromium.org,cpu@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=153139

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

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

35 hours agoTemporarily increase retry limit in test_launcher
sergeyberezin@chromium.org [Tue, 22 Jul 2014 09:02:38 +0000 (09:02 +0000)]
Temporarily increase retry limit in test_launcher

In bot mode, test_launcher would normally retry tests up to 3 times.
This is sufficient to keep large test suites like browser_tests under
1% flakiness rate if the flakiness of individual tests is also within
1%. However, some tests especially in browser_tests are up to 33%
flaky, making the browser_tests suite 40% flaky.

Subsequent top-level retries effectively double the runtime of all
bots running these tests, and we need a solution to shorten the try
job runtime.

A longer-term solution is the ignorer bot (see the bug), but until
it's deployed, a quick hack is to increase the number of retries so
statistically we should see about 1% flakiness of browser_tests at the
expense of a slight runtime increase.

R=sergiyb@chromium.org
BUG=390600, 395189

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

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

36 hours agoChange instant event phase name from i to I
yurys@chromium.org [Tue, 22 Jul 2014 07:19:25 +0000 (07:19 +0000)]
Change instant event phase name from i to I

This matches current definition in the design doc[1] and unifies phase name between Chromium and Blink. Telemetry already expects both "i" and "I" values [2].

It used to be "I" in Chromium but it was changed in https://chromiumcodereview.appspot.com/12252058/

[1] https://docs.google.com/a/chromium.org/document/d/1CvAClvFfyA5R-PhYUmn5OOQtYMH4h6I0nSsKchNAySU/edit#
[2] https://code.google.com/p/chromium/codesearch#chromium/src/tools/telemetry/telemetry/timeline/trace_event_importer.py&q=%22elif%20phase%20==%20'I'%20or%20phase%20==%20'i':%22&sq=package:chromium&type=cs&l=252

BUG=None

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

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

39 hours agoThis is jamesr@ code I am landing.
cpu@chromium.org [Tue, 22 Jul 2014 04:29:58 +0000 (04:29 +0000)]
This is jamesr@ code I am landing.

On Windows the message pump code tried to manage the systemwide timer resolution to fire delayed tasks with better than 15ms resolution but it was buggy:

1- A short task that was not followed by any other task will leave the systemwide timer pegged to 1ms

2- After we decided to crank up the timer we would 'lease' the timer for 1 second, for no good reason.

Both issues are detrimental to battery power.

The source of both problems is that we tried to decide with incomplete information. This patch solves that by having 1 bit for each pending task that requires a high resolution timer and a sum of the number of tasks that require high res timers.

BUG=153139
TEST=included here, also see the bug for manual testing.

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

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

44 hours agobase: Enable lock dchecks with dchecks_always_on
boliu@chromium.org [Mon, 21 Jul 2014 23:11:26 +0000 (23:11 +0000)]
base: Enable lock dchecks with dchecks_always_on

Enable DCHECKs in Lock and ConditionVariable when compiled
with dchecks_always_on.

Note the correctness of DCHECKs relies on DCHECKs enabled
in both Lock and ConditionVariable. This CL keeps the
condition to enable DCHECKs in ConditionVariable the
same as Lock, to avoid any issues here.

BUG=

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

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

2 days agoSet dump_without_crashing_function_ in chrome.dll with value of exported chrome!DumpP...
wfh@chromium.org [Mon, 21 Jul 2014 17:13:58 +0000 (17:13 +0000)]
Set dump_without_crashing_function_ in chrome.dll with value of exported chrome!DumpProcessWithoutCrash.

BUG=391974
TEST=chrome://crashdump and verify crash dump appears and tab does not crash
TBR=darin@chromium.org

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

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

2 days agoclang/win: Fix a Wenum-compare warning and a few Wwritable-strings warnings.
thakis@chromium.org [Mon, 21 Jul 2014 17:03:17 +0000 (17:03 +0000)]
clang/win: Fix a Wenum-compare warning and a few Wwritable-strings warnings.

BUG=82385
R=rnk@chromium.org
TBR=jochen

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

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

2 days ago[Mac] Re-Enable timer coalescing on Mac.
jeremy@chromium.org [Sun, 20 Jul 2014 21:54:00 +0000 (21:54 +0000)]
[Mac] Re-Enable timer coalescing on Mac.

Disabled in r283311 to get a measurement on the effect on the bots.
Re-Enabling because we've collected the data we need.

BUG=356804

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

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

4 days agoDecrement CreateDoubleValue count
estade@chromium.org [Sat, 19 Jul 2014 04:40:29 +0000 (04:40 +0000)]
Decrement CreateDoubleValue count

BUG=160586

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

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

4 days agoAdd metadata that report whether the buffer was full, and the timestamp when that...
nednguyen@google.com [Sat, 19 Jul 2014 00:39:41 +0000 (00:39 +0000)]
Add metadata that report whether the buffer was full, and the timestamp when that happens.

BUG=393750

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

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

4 days ago[android_webview] Introduce AwAssets to reference assets inside the apk.
primiano@chromium.org [Fri, 18 Jul 2014 22:50:46 +0000 (22:50 +0000)]
[android_webview] Introduce AwAssets to reference assets inside the apk.

This change introduces a utility class AwAssets, accessible by native,
which is able to retrieve references (fd + offset + size) of assets
inside the apk. This is to enable direct mmap of uncompressed assets.
This change does NOT introduce yet any change to the WebView apk itself.
At current state, no behavioral change is intended.

BUG=394502

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

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

4 days agoFix data race with synthetic delay initialization
skyostil@chromium.org [Fri, 18 Jul 2014 21:27:12 +0000 (21:27 +0000)]
Fix data race with synthetic delay initialization

The synthetic delay macro initializes a local atomic variable to point
to a delay implementation class. Because initializing the class involves
writing data to memory which is subsequently deferences through the
atomic pointer, we need to use memory barries when updating the pointer
to make sure the data behind it is committed.

Thanks to Alexander Potapenko for spotting this.

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

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

5 days agoRoll ICU to icu52
jshin@chromium.org [Fri, 18 Jul 2014 18:47:57 +0000 (18:47 +0000)]
Roll ICU to icu52

The ICU update does the following among other things:
  - Update EUC-JP (no more JIS X 212 round-trip coverage) per WHATWG
    encoding spec.
  - Add CP866
  - Add far better/more extensive support for plural, format, etc
  - Numerous locale data update
  - Khmer/Lao line/word-breaking support
  - More compact Chinese/Japanese dictionary (now a part of the upstream
    instead of our local patch)
  - Better plural support that we plan to use right away.

- Update icu/BUILD.gn : file list update, new defines, etc.
- Update tests
  a. icu_string_conversions_unittests : drop iscii (nobody uses it) and
     remove JIS X 212 code points from the round-trip test
  b. l10n_util_unittests: Display names for zh-*.
  c. ThaiSnippet's expected result. (due to a Unicode grapheme definition change).
  d. Update Expected results in file_manager_browsertest/test_util.js
     (see http://crbug.com/383930)
- Update SpellcheckerWordIterator
  a. Add U+0027 back to MidNumLet
  b. Add Lao and Khmer scripts to the list of scripts requiring dictionary-based
     word-breaking.
- l10n_util : alias 'tl' to 'fil' (Tagalog to Filipino)
- Update IdentityInternalsSingleTokenWebUITest.verifyGetters (DateTimeFormat-Full
  has 'at' before time, which is not handled by Date.parse()
- update license.py because we don't have a separate Chinese/Japanese
  dictionary any more. It's now a part of ICU

Note to Sheriff :
1. 140 ~ 200 kB size increase is expected for all platforms.
This size increase was singed off by laforge@.
The expected size has to be updated with both reva/revb pointing to the
revision corresponding to this CL.

2.  http://codereview.chromium.org/348913002 has to be applied before running make_expectations.py to update tools/perf_expectations/perf_expectations.json
after this CL is landed.

BUG=132145,277023,136148,360433,364072
TEST=base_unittests:*ICU*, net_unittests:*IDN*, ui_unittests:*Lang*,
     browser_tests (encoding related, Iden*verifyGet*),
     unit_tests:Ext*L10*, unit_tests:*Lang*, component_tests:*Snipp*,
     layout tests (encoding, date/calendar, line breaking, and others),
     build 'base_i18n_nacl', GN build bots.

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

R=arv@chromium.org, brettw@chromium.org, fgorski@chromium.org, groby@chromium.org, hirono@chromium.org, tony@chromium.org, toyoshim@chromium.org

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

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

5 days agoFix StackOverFlow in AdvancedMockContext.
nyquist@chromium.org [Fri, 18 Jul 2014 07:12:52 +0000 (07:12 +0000)]
Fix StackOverFlow in AdvancedMockContext.

The AdvancedMockContext previously did not override registerComponentCallbacks
and unregisterComponentCallbacks. The implementation of these methods in
Context calls getApplicationContext before delegating the call to it, and since
AdvancedMockContext returns |this| in getApplicationContext, this leads to a
loop.

This CL adds overrides for these two methods that make the calls to the base
context instead, which will typically either be a MockContext or the target
context being instrumented.

BUG=394464

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

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

5 days agoWorkaround removal of __system_property_get in Android NDK.
rmcilroy@chromium.org [Fri, 18 Jul 2014 05:41:54 +0000 (05:41 +0000)]
Workaround removal of __system_property_get in Android NDK.

The 'L' release of Android will remove __system_property_get from the NDK.
Until we can replace all calls of __system_property_get, this CL adds a
temporary work-around for builds compiled against the 'L' API.  The workaround
involves creating a stub __system_property_get, which uses dlsym to dynamically
load the address of the real __system_property_get symbol in libc.so (which is
still available as a hidden symbol on the devices libc.so dispite having been
removed from the NDK).

BUG=392191,393903

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

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

5 days agoAvoid macro redefinition in tools_sanity_unittest.cc
tzik@chromium.org [Fri, 18 Jul 2014 02:40:40 +0000 (02:40 +0000)]
Avoid macro redefinition in tools_sanity_unittest.cc

This causes build failure by macro redefinition on ASan-enabled build without -w flag.

BUG=162783

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

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

6 days agoMacro out unused functions in memory_mac.mm
tzik@chromium.org [Thu, 17 Jul 2014 13:28:41 +0000 (13:28 +0000)]
Macro out unused functions in memory_mac.mm

These functions are defined but not used on ASan-enabled build.
That causes build failure on ASan-enabled build without -w flag.

BUG=162783

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

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

6 days agoMake ObserverList non-inline to save 90KB with gcc
bratell@opera.com [Thu, 17 Jul 2014 12:39:34 +0000 (12:39 +0000)]
Make ObserverList non-inline to save 90KB with gcc

With gcc, ObserverList expands to 6-700 bytes every time it is
iterated. By not trying to force OberservListBase to be inline
that shrinks some. clang already ignores the hints to inline
the code and actually doesn't change a single bit in the generated
program with this change.

raw data:
Total change: -90875 bytes
==========================
  262 added, totalling +39188 bytes across 2 sources
  49 removed, totalling -27573 bytes across 16 sources
  23 grown, for a net change of +3600 bytes (34968 bytes before, 38568 bytes after) across 8 sources
  228 shrunk, for a net change of -106090 bytes (239819 bytes before, 133729 bytes after) across 53 sources

The additions is about 2-300 different flavours of ObserverListBase, 100-300 bytes each.

The savings are 200-600 bytes each every time an observer list is used. For example:

-870: content::RenderFrameImpl::didFailLoad(blink::WebLocalFrame*, blink::WebURLError const&) type=t, (was 1635 bytes,
now 765 bytes)
-895: content::RenderFrameImpl::didFinishLoad(blink::WebLocalFrame*) type=t, (was 1646 bytes, now 751 bytes)
-583: content::WebContentsImpl::DidFailProvisionalLoadWithError(content::RenderFrameHostImpl*, FrameHostMsg_DidFailProvisionalLoadWithError_Params const&) type=t, (was 768 bytes, now 185 bytes)
-611: content::WebContentsImpl::DidStartProvisionalLoad(content::RenderFrameHostImpl*, GURL const&, bool, bool) type=t, (was 895 bytes, now 284 bytes)

BUG=394311

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

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

6 days agoSeparate ResizeHelper from RenderWidgetHelper
ccameron@chromium.org [Thu, 17 Jul 2014 09:30:42 +0000 (09:30 +0000)]
Separate ResizeHelper from RenderWidgetHelper

Create a separate RenderWidgetResizeHelper to handle intercepting
and early-processing of IPCs that need to be run during an active
resize. This functionality was previously handled in
RenderWidgetHelper.

The only functional changes made to RenderWidgetResizeHelper during
its excision is that it is a single global structure, as compared with
the per-RenderProcessImpl RenderWidgetHelper. It will execute any IPCs
from any renderer or the GPU processes that are forwarded to it. This is
preferable, as that makes it so that live resize in one window does
not destroy performance in other windows.

Because RenderWidgetResizeHelper can handle GPU process IPCs, we can
tear out the bizarre behavior where we'd translate GPU IPCs to synthetic
renderer IPCs get them to channel through RenderWidgetHelper (this is
all of the deleted ViewHostMsg_CompositorSurfaceBuffersSwapped code).

Mark this entire structure as Mac-only, and put it in Mac-only files,
since it doesn't get used on other platforms.

Future changes will make RenderWidgetResizeHelper have a TaskRunner,
which we will pass to the compositor, so that we can pump new frames
while inside -[RenderWidgetHostViewMac setFrameSize:].

BUG=392031

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

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

7 days agoFix the build when use_glib is set to 0.
zhaoze.zhou@partner.samsung.com [Wed, 16 Jul 2014 19:12:40 +0000 (19:12 +0000)]
Fix the build when use_glib is set to 0.
use_glib=0 can work on some embedded setup that doesn't use glib.

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

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

7 days agoTrack size of files serialized by the JsonPrefStore.
gab@chromium.org [Wed, 16 Jul 2014 06:04:44 +0000 (06:04 +0000)]
Track size of files serialized by the JsonPrefStore.

In an effort to see what percentage of the population has exceedingly large pref files.

BUG=355722

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

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

7 days ago[Mac] Temporarily disable timer coalescing on Mac.
jeremy@chromium.org [Wed, 16 Jul 2014 00:59:28 +0000 (00:59 +0000)]
[Mac] Temporarily disable timer coalescing on Mac.

In the interest of conserving power, several incremental changes to timer slack have been committed over the last few weeks.  This CL disables those changes so we can get some numbers for the combined effect in terms of power usage and idle wakeup numbers.

This needs to be checked in because we can only get a good comparative measure for this when run on the bots in a controlled environment.

BUG=356804

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

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

7 days ago[Android][gn] Add android resources templates
cjhopman@chromium.org [Wed, 16 Jul 2014 00:11:03 +0000 (00:11 +0000)]
[Android][gn] Add android resources templates

This adds support for android resources to gn.

Two new templates are introduced:

java_string_grd: This is like gyp's build/java_strings_grd.gypi. It runs
grit and generates Android strings.xml files and then zips them
together.
android_resources: This is the target for android resources. It is
mostly a wrapper around process_resources.py. This is *not part* of
android_library (like it is in gyp). Making these two things separate is
more like facebook's BUCK and google's internal build rules (and gyp's
java.gypi and java_apk.gypi have gotten way too big and complicated).

Changes to the actual build scripts are very minor except for the added
support for andoid_resources to write_build_config.py. Building
resources requires getting all the transitive resource dependencies, and
so this reuses the simple sorted transitive dependency thing from
write_ordered_libraries.py.

TBR=rch@chromium.org
BUG=359249

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

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

8 days agoAdd a unit test that filenames aren't unintentionally converted to URLs.
dcheng@chromium.org [Tue, 15 Jul 2014 19:04:37 +0000 (19:04 +0000)]
Add a unit test that filenames aren't unintentionally converted to URLs.

Also fixes two issues in OSExchangeDataProviderWin:
- It used a disjoint set of clipboard formats when handling
  GetUrl(..., true /* filename conversion */) vs GetFilenames(...), so the
  actual returned results would vary depending on which one was called.
- It incorrectly used ::DragFinish() instead of ::ReleaseStgMedium().
  ::DragFinish() is only meant to be used in conjunction with WM_DROPFILES.

BUG=346135

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

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

8 days agoadd double support to base::Pickle
mostynb@opera.com [Mon, 14 Jul 2014 22:50:32 +0000 (22:50 +0000)]
add double support to base::Pickle

Since WebTimeRange now uses doubles instead of floats[1], we should
probably add double support to base::Pickle.

[1] https://codereview.chromium.org/337923006

BUG=393552

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

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

8 days agoAdd the required xvfb testing tool to the isolate target for some tests.
petermayo@chromium.org [Mon, 14 Jul 2014 22:47:08 +0000 (22:47 +0000)]
Add the required xvfb testing tool to the isolate target for some tests.

Missing from base_unit_tests, unit_tests and browser_tests.

TBR=brettw@chromium.org
R=maruel@chromium.org, brettw@chromium.org
BUG=None
TEST=local clean ; gyp ; build ; run isolate for base

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

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

9 days agoLeftover cleanups from https://codereview.chromium.org/324493002 .
erikwright@chromium.org [Mon, 14 Jul 2014 17:28:14 +0000 (17:28 +0000)]
Leftover cleanups from https://codereview.chromium.org/324493002 .

Moves one more method from PersistentPrefStore to WritablePrefStore. This allows most "writing" operations to be possible without the more complex PersistentPrefStore API.

BUG=None

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

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

9 days agoThe message length should be serialized as a 64-bit value.
wtc@chromium.org [Mon, 14 Jul 2014 14:46:09 +0000 (14:46 +0000)]
The message length should be serialized as a 64-bit value.

Also optimize the swapends function.

R=mark@chromium.org,rsleevi@chromium.org,yukawa@chromium.org
BUG=348333

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

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

11 days agoAdd missing files/deps/defines
cjhopman@chromium.org [Sat, 12 Jul 2014 04:59:22 +0000 (04:59 +0000)]
Add missing files/deps/defines

TBR=darin@chromium.org

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

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

11 days agomac: Add metrics to record Bluetooth availability and capabilities.
erikchen@chromium.org [Sat, 12 Jul 2014 01:27:44 +0000 (01:27 +0000)]
mac: Add metrics to record Bluetooth availability and capabilities.

The new Handoff feature in OSX 10.10 only works with devices that support
Bluetooth LE. Record metrics to determine the percentage of Chrome users that
this affects.

BUG=392166

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

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

11 days agoIntroduce task_runner() accessors for both base::Thread and base::MessageLoop
rsleevi@chromium.org [Sat, 12 Jul 2014 01:10:52 +0000 (01:10 +0000)]
Introduce task_runner() accessors for both base::Thread and base::MessageLoop

This is so that callers can code against a TaskRunner interface
directly, rather than converting message_loop_proxy() into a TaskRunner.

BUG=391045

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

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

13 days agoDCHECK(CalledOnValidThread()) for ScopedUserPrefUpdate
battre@chromium.org [Thu, 10 Jul 2014 17:09:51 +0000 (17:09 +0000)]
DCHECK(CalledOnValidThread()) for ScopedUserPrefUpdate

BUG=392781

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

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

13 days agoSet parcel data position to 0 when cloning the relro bundle.
simonb@chromium.org [Thu, 10 Jul 2014 10:55:36 +0000 (10:55 +0000)]
Set parcel data position to 0 when cloning the relro bundle.

Rename local variable from 'p' to 'parcel.  Call parcel.setDataPosition(0)
between write and read.  Fix a log message for punctuation.

Fixes (from logcat):
  E/Bundle  ( 5115): readBundle: bad magic number
  E/Bundle  ( 5115): readBundle: trace = java.lang.RuntimeException
  E/Bundle  ( 5115):      at android.os.Bundle.readFromParcelInner(Bundle.java:1677)
  E/Bundle  ( 5115):      at android.os.Bundle.readFromParcel(Bundle.java:1670)
  E/Bundle  ( 5115):      at org.chromium.base.library_loader.Linker.useSharedRelros(Linker.java:494)
  E/Bundle  ( 5115):      at org.chromium.content.app.ChildProcessService$1.setupConnection(ChildProcessService.java:100)
  E/Bundle  ( 5115):      at org.chromium.content.common.IChildProcessService$Stub.onTransact(IChildProcessService.java:57)
  E/Bundle  ( 5115):      at android.os.Binder.execTransact(Binder.java:388)
  E/Bundle  ( 5115):      at dalvik.system.NativeStart.run(Native Method)

BUG=388356

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

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

13 days agobase: Default android version to 4.4.99
boliu@chromium.org [Thu, 10 Jul 2014 08:05:21 +0000 (08:05 +0000)]
base: Default android version to 4.4.99

Under-development Android versions do not have a numeric
version, so falls back to a "default version".

Using a real released Android version like 4.3.0 will
incorrectly trigger code paths, such has version-specific
gpu blacklists.

First attempt at fixing this was using 0.0.0, which caused
websites parsing Android version in the user agent to break.

So this is the second attempt, use the latest public major
version and minor version, and use a higher bug release
version to avoid conflicts with the gpu blacklist.

BUG=387179

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

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

13 days agoFixes for re-enabling more MSVC level 4 warnings: base/third_party/dmg_fp/ edition
pkasting@chromium.org [Thu, 10 Jul 2014 08:04:42 +0000 (08:04 +0000)]
Fixes for re-enabling more MSVC level 4 warnings: base/third_party/dmg_fp/ edition

This contains fixes for the following sorts of issues:
* Assignment inside conditional

This updates the relevant .patch and README files, combining these fixes with
the previous set of MSVC warning fixes.  This also removes the now-unused
vs2013-optimization.patch file.

BUG=81439
TEST=none

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

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

13 days agoFixes for re-enabling more MSVC level 4 warnings: base/ edition #2
pkasting@chromium.org [Thu, 10 Jul 2014 04:38:02 +0000 (04:38 +0000)]
Fixes for re-enabling more MSVC level 4 warnings: base/ edition #2

This contains fixes for the following sorts of issues:
* Signedness mismatch

Defining UChar32 as a signed type may look wrong, but it's actually how the type
is already defined over in third_party/icu/.

BUG=81439
TEST=none

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

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

2 weeks agoRename source_prereqs to inputs in the GN build.
brettw@chromium.org [Wed, 9 Jul 2014 17:29:31 +0000 (17:29 +0000)]
Rename source_prereqs to inputs in the GN build.

This variable changed name.

R=jamesr@chromium.org

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

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

2 weeks agoGN: Move mock_pref_change_callback.h to the correct section.
jbroman@chromium.org [Wed, 9 Jul 2014 15:15:37 +0000 (15:15 +0000)]
GN: Move mock_pref_change_callback.h to the correct section.

This resolves the following 'gn check' error:

  ERROR at //base/prefs/mock_pref_change_callback.cc:5:11: Include not allowed.
  #include "base/prefs/mock_pref_change_callback.h"
            ^-------------------------------------
  It is not in any dependency of //base:prefs_test_support
  The include file is in the target(s):
    //base:base_unittests
  which should somehow be reachable from //base:prefs_test_support

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

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

2 weeks agobase: Increase hard limit for emulated discardable memory.
reveman@chromium.org [Wed, 9 Jul 2014 05:05:25 +0000 (05:05 +0000)]
base: Increase hard limit for emulated discardable memory.

This increases the emulated discardable memory limit to 512MB by
setting the soft limit to 32MB and the hard limit expiration time
to 1 second.

BUG=383361,372158

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

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

2 weeks agoRemove base/memory/scoped_handle.h from BUILD.gn.
jbroman@chromium.org [Wed, 9 Jul 2014 04:56:27 +0000 (04:56 +0000)]
Remove base/memory/scoped_handle.h from BUILD.gn.

This file went away in https://codereview.chromium.org/324773002

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

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

2 weeks agoMove logging.h's definitions of operator<< into namespace std.
jyasskin@chromium.org [Tue, 8 Jul 2014 23:03:06 +0000 (23:03 +0000)]
Move logging.h's definitions of operator<< into namespace std.

See the bug for a more detailed discussion of the problem when we don't make
this accessible through ADL.

BUG=391117

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

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

2 weeks ago[Android] Spill out CHECK error message in official build
feng@chromium.org [Tue, 8 Jul 2014 22:52:33 +0000 (22:52 +0000)]
[Android] Spill out CHECK error message in official build

In M35 release, we had a lot of user feedbacks of Chrome native crash.
We had great difficulty to investigate these native crashes
because user feedbacks don't have minidumps.

By keeping the CHECK error message in logcat, it'd be a lot easier
to investigate these user reports of Chrome crashes.

The binary size increase: ~0.5% (188k out of 34.4M).

BUG=378974

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

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

2 weeks agoFix compilation error encountered when DEBUG set to 1.
simonb@chromium.org [Tue, 8 Jul 2014 20:23:52 +0000 (20:23 +0000)]
Fix compilation error encountered when DEBUG set to 1.

Fails to compiler if DEBUG is set to 1.  If DEBUG is 0 the error is hidden
because LOG_INFO() expands to an empty operation.

Fixes:
  ../../base/android/linker/linker_jni.cc:270:60: error: 'lib_basename' was not declared in this scope

BUG=390618

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

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

2 weeks agobase: Add soft memory limit to DiscardableMemoryManager.
reveman@chromium.org [Tue, 8 Jul 2014 19:31:26 +0000 (19:31 +0000)]
base: Add soft memory limit to DiscardableMemoryManager.

This adds a soft memory limit to the discardable memory manager. Only
allocations that haven't been used in specific amount of time are
affected by this limit. This allows a client to use a much higher hard
limit so temporary spikes in required usage can be handled efficiently
without causing an increase in usage while idle.

A "reduce memory usage" function is added to the discardable memory
interface to allow the client to effectively trigger purging of memory
that is affected by the soft limit when system reaches what would be
considered an idle state.

BUG=383361,372158
TEST=base_unittests --gtest_filter=DiscardableMemoryManagerTest.ReduceMemoryUsage

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

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

2 weeks agoFix tracing 64-bit to 32-bit truncations.
wfh@chromium.org [Tue, 8 Jul 2014 14:21:16 +0000 (14:21 +0000)]
Fix tracing 64-bit to 32-bit truncations.

BUG=392035

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

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

2 weeks agoSet CONFIGURATION_NAME to be used in base.isolate.
scheib@chromium.org [Tue, 8 Jul 2014 12:46:13 +0000 (12:46 +0000)]
Set CONFIGURATION_NAME to be used in base.isolate.

Enables precise control of the isolate depending on correct
versions of msvc*.dll.

BUG=390686

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

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

2 weeks agoChange g_native_tls_key to be 32-bit wide to match underlying type.
wfh@chromium.org [Tue, 8 Jul 2014 07:37:04 +0000 (07:37 +0000)]
Change g_native_tls_key to be 32-bit wide to match underlying type.

BUG=392037
TEST=base_unittests --gtest_filter=ThreadLocalStorageTest.*

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

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

2 weeks agoExtend workaround from https://codereview.chromium.org/98403005 to clang/win.
thakis@chromium.org [Fri, 4 Jul 2014 22:10:39 +0000 (22:10 +0000)]
Extend workaround from https://codereview.chromium.org/98403005 to clang/win.

BUG=82385
TBR=ch.dumez@samsung.com

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

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

2 weeks agoAdd CancelableTaskTracker::PostTaskAndReplyWithResult
sdefresne@chromium.org [Fri, 4 Jul 2014 11:19:50 +0000 (11:19 +0000)]
Add CancelableTaskTracker::PostTaskAndReplyWithResult

The helper function PostTaskAndReplyWithResult is useful but did
not work with CancelableTaskTracker, so add the support to the
CancelableTaskTracker class with a template method.

BUG=None

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

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

2 weeks agoRemove ProcessMetrics::CalculateFreeMemory.
wfh@chromium.org [Thu, 3 Jul 2014 22:24:06 +0000 (22:24 +0000)]
Remove ProcessMetrics::CalculateFreeMemory.

This function is not used anywhere, and doesn't work on Win8 64-bit with HE ASLR winheap.

BUG=None
TEST=base_unittests

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

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

2 weeks agodeadlock fix in SymbolContext ctor
cpu@chromium.org [Thu, 3 Jul 2014 21:09:20 +0000 (21:09 +0000)]
deadlock fix in SymbolContext ctor

If PathService dcheks or checks then we deadlock since the symbolizer
uses path service itself. See bug for details.

BUG=391331
NOTRY=true

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

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

2 weeks agoFixes for re-enabling more MSVC level 4 warnings: base/ edition
pkasting@chromium.org [Thu, 3 Jul 2014 07:03:39 +0000 (07:03 +0000)]
Fixes for re-enabling more MSVC level 4 warnings: base/ edition

This contains fixes for the following sorts of issues:
* Signedness mismatch
* Assignment inside conditional
* Nonstandard extension: unnamed struct/union
* Octal escape sequence terminated by decimal number
* Possibly-uninitialized local variable

This also contains a very small number of other cleanups/simplifications to
nearby code.

BUG=81439
TEST=none

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

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

2 weeks agoFix a typo in a comment for Closure.
wtc@chromium.org [Thu, 3 Jul 2014 06:10:13 +0000 (06:10 +0000)]
Fix a typo in a comment for Closure.

R=ajwong@chromium.org
BUG=

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

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

3 weeks agoRevert of Revert of Revert 221427 "Add instrumentation to the MessagePumpMac family...
rsesek@chromium.org [Wed, 2 Jul 2014 16:39:33 +0000 (16:39 +0000)]
Revert of Revert of Revert 221427 "Add instrumentation to the MessagePumpMac family ..." (https://codereview.chromium.org/366873002/)

Reason for revert:
Trying again. Will clobber builder since I can't repro this.

Original issue's description:
> Revert of Revert 221427 "Add instrumentation to the MessagePumpMac family ..." (https://codereview.chromium.org/360373004/)
>
> Reason for revert:
> This revert seems to cause build failure:
>
> http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Builder/builds/8982
> http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Full/builds/6497
> http://build.chromium.org/p/chromium.linux/builders/Linux%20Builder/builds/34221
>
>
> Original issue's description:
> > Revert 221427 "Add instrumentation to the MessagePumpMac family ..."
> >
> > This code is no longer needed, since the experiment is not viable.
> >
> > > Add instrumentation to the MessagePumpMac family of classes.
> > >
> > > This adds UMA-uploaded histograms of sampling intervals for key points of data:
> > > * Total time spent in a CFRunLoop
> > > * Total time waiting in CFRunLoop
> > > * Total time servicing MessagePump::Delegate (the MessageLoop)
> > > * The MessageLoop queue size and queueing delay before servicing
> > >
> > > It adds 1 second sampling for 15 seconds at startup, only for the main thread
> > > (NSApplication-based) run loops.
> > >
> > > The data will be used to determine if adding scheduling-like behavior to the
> > > MessagePump will result in more efficient processing of the MessageLoop work.
> > >
> > > An unforunate side effect of this change is exposing another method on the
> > > MessagePump::Delegate interface, but there does not appear to be a better way
> > > to do this.
> > >
> > > BUG=264886
> > > R=jar@chromium.org, mark@chromium.org
> > >
> > > Review URL: https://codereview.chromium.org/22911026
> >
> > TBR=rsesek@chromium.org
> >
> > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=280939
>
> TBR=rsesek@chromium.org,rsesek@chromium.org
> NOTREECHECKS=true
> NOTRY=true
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=280948

TBR=rsesek@chromium.org,tzik@chromium.org
NOTREECHECKS=true
NOTRY=true

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

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

3 weeks agoResolve build issues for ashmem_allocator/libc++
pasko@chromium.org [Wed, 2 Jul 2014 15:02:19 +0000 (15:02 +0000)]
Resolve build issues for ashmem_allocator/libc++

This is mechanically replacing void* with uintptr_t without much thinking,
motivated by the fact that libc++ does not have a hash function for void*.

The external interface is left intact, the internal classes (AshmemRegion,
FreeChunk) are switched to operate only with uintptr_t.

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

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

3 weeks agoAdd android_library template and build_configs
cjhopman@chromium.org [Wed, 2 Jul 2014 10:46:46 +0000 (10:46 +0000)]
Add android_library template and build_configs

This is the GN equivalent of build/java.gypi. It is a template for
creating an Android library including java code and Android resources.

It currently only compiles java files (including files in srcjars from
srcjar targets like java_cpp_template) and zips them together in a .jar
and creates the corresponding .jar.TOC.

Some of the things still to do: proguard_preprocess, android_lint,
emma coverage, dex, everything resources.

Adds android_java_library rule for base_java, guava, and jsr-305.

This add the --java-srcjars argument to javac.py. This will accept a .zip of
.java files and include those files in the compilation. This approach is
preferred over using the --src-gendirs option.

Many of the parts of building Android stuff (libraries, resources, apks)
require knowledge of the dependents of that thing. Examples: javac classpath,
for resources aapt needs to know about all dependents, dexing for an apk
needs to know about all java code going into that apk. For gyp, this is done
primarily with all_dependent_settings. There is then some of this logic in
two particular steps (dexing and proguard). These steps, when building an
instrumentation apk, need to exclude the things in the tested apk and this is
done by having the tested apk essentially write a file saying what it did in
those steps and the test apk reading that file and excluding stuff.

In GN, all_dependent_settings doesn't really work. This change introduces a new
way of calculating and using this information. Specifically .build_config files
and build_utils.ExpandFileArgs(). The build_config file for a target contains
the information that depends on dependents. The logic in write_build_config and
the logic in the template specification are very much tied together (in some
sense, write_build_config is just the part of the template specification
that can actually inspect the dependency graph). With
build_utils.ExpandFileArgs() all the other build scripts are essentially
unaware of the .build_config files and can just be written in a (mostly)
straightforward way.

A large part of the information calculated by the build_config is finding input
files to later actions. This requires that those later actions writes a depfile
that contains any inputs that are specified by the build_config (in the case of
this change, javac and the classpath files). Since a action's script shouldn't
really know about the build_config file and what information it got
from that, it is safest for the action to write *all* of its inputs into the
depfile (but to be correct it only has to write those that aren't explicitly
specified in the build files).

Depends on: https://codereview.chromium.org/341823003/

BUG=359249

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

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

3 weeks agoRevert of Revert 221427 "Add instrumentation to the MessagePumpMac family ..." (https...
tzik@chromium.org [Wed, 2 Jul 2014 03:31:54 +0000 (03:31 +0000)]
Revert of Revert 221427 "Add instrumentation to the MessagePumpMac family ..." (https://codereview.chromium.org/360373004/)

Reason for revert:
This revert seems to cause build failure:

http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Builder/builds/8982
http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Full/builds/6497
http://build.chromium.org/p/chromium.linux/builders/Linux%20Builder/builds/34221

Original issue's description:
> Revert 221427 "Add instrumentation to the MessagePumpMac family ..."
>
> This code is no longer needed, since the experiment is not viable.
>
> > Add instrumentation to the MessagePumpMac family of classes.
> >
> > This adds UMA-uploaded histograms of sampling intervals for key points of data:
> > * Total time spent in a CFRunLoop
> > * Total time waiting in CFRunLoop
> > * Total time servicing MessagePump::Delegate (the MessageLoop)
> > * The MessageLoop queue size and queueing delay before servicing
> >
> > It adds 1 second sampling for 15 seconds at startup, only for the main thread
> > (NSApplication-based) run loops.
> >
> > The data will be used to determine if adding scheduling-like behavior to the
> > MessagePump will result in more efficient processing of the MessageLoop work.
> >
> > An unforunate side effect of this change is exposing another method on the
> > MessagePump::Delegate interface, but there does not appear to be a better way
> > to do this.
> >
> > BUG=264886
> > R=jar@chromium.org, mark@chromium.org
> >
> > Review URL: https://codereview.chromium.org/22911026
>
> TBR=rsesek@chromium.org
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=280939

TBR=rsesek@chromium.org,rsesek@chromium.org
NOTREECHECKS=true
NOTRY=true

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

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

3 weeks agoRevert 221427 "Add instrumentation to the MessagePumpMac family ..."
rsesek@chromium.org [Wed, 2 Jul 2014 02:03:57 +0000 (02:03 +0000)]
Revert 221427 "Add instrumentation to the MessagePumpMac family ..."

This code is no longer needed, since the experiment is not viable.

> Add instrumentation to the MessagePumpMac family of classes.
>
> This adds UMA-uploaded histograms of sampling intervals for key points of data:
> * Total time spent in a CFRunLoop
> * Total time waiting in CFRunLoop
> * Total time servicing MessagePump::Delegate (the MessageLoop)
> * The MessageLoop queue size and queueing delay before servicing
>
> It adds 1 second sampling for 15 seconds at startup, only for the main thread
> (NSApplication-based) run loops.
>
> The data will be used to determine if adding scheduling-like behavior to the
> MessagePump will result in more efficient processing of the MessageLoop work.
>
> An unforunate side effect of this change is exposing another method on the
> MessagePump::Delegate interface, but there does not appear to be a better way
> to do this.
>
> BUG=264886
> R=jar@chromium.org, mark@chromium.org
>
> Review URL: https://codereview.chromium.org/22911026

TBR=rsesek@chromium.org

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

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

3 weeks agoRe-land "Replaces use of ProcessInformationCopyDictionary ..."
leng@chromium.org [Tue, 1 Jul 2014 22:05:39 +0000 (22:05 +0000)]
Re-land "Replaces use of ProcessInformationCopyDictionary ..."

This reverts commit 5c1848ea87157db1a455a73b1868a4a962afa215.

Original CL was reverted because ProcessInfoRec::processAppSpec does not
exist on x64.  (Google search results claim it is named processAppRec instead.)

This patch changes the original CL by initializing the whole
ProcessInfoRect struct to 0 before use, rather than setting individual
members to NULL.

Original CL:
https://codereview.chromium.org/354243003/

Replaces use of ProcessInformationCopyDictionary with GetProcessInformation.

ProcessInformationCopyDictionary is crashing on 10.7.
This changes WasLaunchedAsLoginOrResumeItem to use lower-level calls
instead of ProcessInformationCopyDictionary, with the hope that it will
bypass the 10.7 crash. Other reports of crashes in this function on 10.7
claim to have been worked around, so there is reason to believe this will
not crash.

BUG=383029

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

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

3 weeks agoEnable the crazy linker for arm64.
simonb@chromium.org [Tue, 1 Jul 2014 18:13:47 +0000 (18:13 +0000)]
Enable the crazy linker for arm64.

Turn off the check that disables the chromium linker on arm64.

BUG=369133

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

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

3 weeks agoAdd switch for turning on library in zip file support.
anton@chromium.org [Tue, 1 Jul 2014 13:16:56 +0000 (13:16 +0000)]
Add switch for turning on library in zip file support.

Note this patch doesn't enable the support (that is done in GYP).

BUG=

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

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

3 weeks agoIncrease storage for Java exception information.
ohrn@opera.com [Tue, 1 Jul 2014 10:08:54 +0000 (10:08 +0000)]
Increase storage for Java exception information.

Nested exceptions have multiple stack traces with the most important one at the end.

BUG=389474

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

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

3 weeks agoPopulate NativeLibraryLoadError on mac on dlopen failure
jamesr@chromium.org [Tue, 1 Jul 2014 00:50:30 +0000 (00:50 +0000)]
Populate NativeLibraryLoadError on mac on dlopen failure

BUG=353771

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

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

3 weeks agoRevert 280679 "Replaces use of ProcessInformationCopyDictionary ..."
shrikant@chromium.org [Mon, 30 Jun 2014 23:04:50 +0000 (23:04 +0000)]
Revert 280679 "Replaces use of ProcessInformationCopyDictionary ..."
R=leng

> Replaces use of ProcessInformationCopyDictionary with GetProcessInformation.
>
> ProcessInformationCopyDictionary is crashing on 10.7.
> This changes WasLaunchedAsLoginOrResumeItem to use lower-level calls
> instead of ProcessInformationCopyDictionary, with the hope that it will
> bypass the 10.7 crash.  Other reports of crashes in this function on 10.7
> claim to have been worked around, so there is reason to believe this will
> not crash.
>
> BUG=383029
>
> Review URL: https://codereview.chromium.org/354243003

TBR=leng@chromium.org

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

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

3 weeks agoReplaces use of ProcessInformationCopyDictionary with GetProcessInformation.
leng@chromium.org [Mon, 30 Jun 2014 22:11:17 +0000 (22:11 +0000)]
Replaces use of ProcessInformationCopyDictionary with GetProcessInformation.

ProcessInformationCopyDictionary is crashing on 10.7.
This changes WasLaunchedAsLoginOrResumeItem to use lower-level calls
instead of ProcessInformationCopyDictionary, with the hope that it will
bypass the 10.7 crash.  Other reports of crashes in this function on 10.7
claim to have been worked around, so there is reason to believe this will
not crash.

BUG=383029

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

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

3 weeks ago[Android]: Replace calls to getdtablesize with sysconf(_SC_OPEN_MAX) in tools.
rmcilroy@chromium.org [Sat, 28 Jun 2014 20:54:43 +0000 (20:54 +0000)]
[Android]: Replace calls to getdtablesize with sysconf(_SC_OPEN_MAX) in tools.

The getdtablesize functions has been deprecated by Posix and has been removed
from Android's LB64 ABI. Replace calls to it with the modern equivalent
sysconf(_SC_OPEN_MAX).

This fixes a build breakage in tryserver.blink/builders/android_blink_compile_rel/dbg.

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

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

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

3 weeks ago[Android] Add an error message for java exception crash
boliu@chromium.org [Sat, 28 Jun 2014 01:58:04 +0000 (01:58 +0000)]
[Android] Add an error message for java exception crash

We intentionally crash if jni called java code results in exception.
However the CHECK is in native code. So many developers report the
native stack, but not the java stack, which is usually more helpful.

Add a message to the CHECK(false) for this. And also restructure the
code a bit to avoid repeating the message.

BUG=

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

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

3 weeks agoAdd base::SizeTToString.
wtc@chromium.org [Fri, 27 Jun 2014 19:10:38 +0000 (19:10 +0000)]
Add base::SizeTToString.

Change some users of base::Uint64ToString to base::SizeTToString.

R=brettw@chromium.org,rlarocque@chromium.org

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

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

3 weeks agoChromium Linker support for loading lib*.so from within an APK file
anton@chromium.org [Fri, 27 Jun 2014 18:57:23 +0000 (18:57 +0000)]
Chromium Linker support for loading lib*.so from within an APK file

Split from https://codereview.chromium.org/301223005/

BUG=

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

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

3 weeks agoAdd support for uncompress library in APK to the build system
anton@chromium.org [Fri, 27 Jun 2014 16:59:46 +0000 (16:59 +0000)]
Add support for uncompress library in APK to the build system

BUG=

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

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

3 weeks agoRemoves bulach@ from OWNERS.
bulach@chromium.org [Fri, 27 Jun 2014 13:11:13 +0000 (13:11 +0000)]
Removes bulach@ from OWNERS.

May our paths cross again soon!

BUG=
TBR=jochen@chromium.org,thakis@chromium.org

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

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

3 weeks agostub out debug::StackTrace().Print() when using uclibc
mostynb@opera.com [Fri, 27 Jun 2014 09:21:03 +0000 (09:21 +0000)]
stub out debug::StackTrace().Print() when using uclibc

This is a followup from https://codereview.chromium.org/228753003 which expands the subset of chromium that should be possible to build with uClibc.

BUG=386961

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

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

3 weeks agoFix comparison between signed and unsigned integer expressions.
yfriedman@chromium.org [Fri, 27 Jun 2014 07:40:45 +0000 (07:40 +0000)]
Fix comparison between signed and unsigned integer expressions.

Only see on internal bots.

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

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

3 weeks agoRevert 280152 "Base: Disable the scoped handle verifier"
rvargas@google.com [Thu, 26 Jun 2014 23:20:16 +0000 (23:20 +0000)]
Revert 280152 "Base: Disable the scoped handle verifier"

> Base: Disable the scoped handle verifier
>
> This is intended for the M37 branch
>
> BUG=389346
> TBR=cpu@chromium.org
>
> Review URL: https://codereview.chromium.org/337123006

TBR=rvargas@google.com

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

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

3 weeks agoBase: Disable the scoped handle verifier
rvargas@google.com [Thu, 26 Jun 2014 23:05:34 +0000 (23:05 +0000)]
Base: Disable the scoped handle verifier

This is intended for the M37 branch

BUG=389346
TBR=cpu@chromium.org

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

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

3 weeks agoExpose a low-end device mode override flags for non-android OSs as well
c.shu@samsung.com [Thu, 26 Jun 2014 16:55:27 +0000 (16:55 +0000)]
Expose a low-end device mode override flags for non-android OSs as well

BUG=324824

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

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

3 weeks agoSuppress data race reports:
glider@chromium.org [Thu, 26 Jun 2014 13:45:42 +0000 (13:45 +0000)]
Suppress data race reports:
 - on system_base_info in usrsctp (issue 348984)
 - in WebRtcBrowserTest tests (issue 389098)

BUG=389098,348984
TBR=eugenis@chromium.org
NOTRY=true

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

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

3 weeks agoMake sure ThreadSanitizer v2 reports a data race in ToolsSanityTest.DataRace.
glider@chromium.org [Thu, 26 Jun 2014 13:10:50 +0000 (13:10 +0000)]
Make sure ThreadSanitizer v2 reports a data race in ToolsSanityTest.DataRace.

BUG=351259
R=phajdan@chromium.org
TBR=thakis@chromium.org

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

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

3 weeks agoEdited comment to say "callback will not be run" instead of "callback will not be...
brandonsalmon@chromium.org [Thu, 26 Jun 2014 06:07:45 +0000 (06:07 +0000)]
Edited comment to say "callback will not be run" instead of "callback will not be issued".

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

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

3 weeks agoMove all remaining appcache-related code to content namespace
pilgrim@chromium.org [Wed, 25 Jun 2014 20:57:55 +0000 (20:57 +0000)]
Move all remaining appcache-related code to content namespace

This introduces NO FUNCTIONAL CHANGES. It is strictly a file move and namespace change.

BUG=338338
R=michaeln@chromium.org
TBR=cevans, darin

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

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

4 weeks agoRevert of Revert "[Mac] Reland r276808 Use a native MessagePump instead of a MessageP...
jeremy@chromium.org [Wed, 25 Jun 2014 17:40:57 +0000 (17:40 +0000)]
Revert of Revert "[Mac] Reland r276808 Use a native MessagePump instead of a MessagePumpDefault" (https://codereview.chromium.org/354673005/)

Reason for revert:
Original revert was due to a perf regression, which turned out to be nothing - see crbug.com/388360 for details.

Original issue's description:
> Revert "[Mac] Reland r276808 Use a native MessagePump instead of a MessagePumpDefault"
>
> This reverts commit e82339d44d0dde725295785b9fea8c448d382621.
>
> See if this fixes the perf regression.
>
> BUG=388360
> TBR=jeremy@chromium.org
> NOTRY=true
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=279474

TBR=miletus@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=388360

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

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

4 weeks agoMemory tools suppressions:
glider@chromium.org [Wed, 25 Jun 2014 14:11:43 +0000 (14:11 +0000)]
Memory tools suppressions:
 - data race on g_next_user_script_id in extensions::ContentScriptsHandler::Parse (issue 388730)
 - memory leak in DataReductionProxyUsageStatsTest.isDataReductionProxyUnreachable (issue 388668)

BUG=388668,388730
TBR=eugenis@chromium.org
NOTRY=true

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

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

4 weeks agoAdd history_size=7 to the TSan default options.
glider@chromium.org [Wed, 25 Jun 2014 12:50:13 +0000 (12:50 +0000)]
Add history_size=7 to the TSan default options.

BUG=None
TBR=earthdok@chromium.org

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

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

4 weeks agoAdd depfile support to android build scripts
cjhopman@chromium.org [Tue, 24 Jun 2014 23:38:17 +0000 (23:38 +0000)]
Add depfile support to android build scripts

This adds the --depfile option to a bunch of android's build scripts
(the ones used by gn). It also adds a simple function in build_utils to
calculate the paths of the imported python files.

Currently, the written depfiles just contain that list of python files
(which is much more accurate than the way this is handled in gyp).

BUG=359249

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

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

4 weeks agoCreate WebSocketTransportClientSocketPool
ricea@chromium.org [Tue, 24 Jun 2014 22:53:31 +0000 (22:53 +0000)]
Create WebSocketTransportClientSocketPool

It is a variant of TransportClientSocketPool that performs RFC6455-style
connection throttling.

Design doc: https://docs.google.com/a/chromium.org/document/d/1a8sUFQsbN5uve7ziW61ATkrFr3o9A-Tiyw8ig6T3puA/edit

BUG=343107

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

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

4 weeks agoFixes "restore windows" behavior for users who have never changed it.
leng@chromium.org [Tue, 24 Jun 2014 20:33:48 +0000 (20:33 +0000)]
Fixes "restore windows" behavior for users who have never changed it.

According to Apple's documentation, the plist which records the "reopen
windows..." setting will not exist unless a login item has been changed.  As
the "reopen windows" is the default behavior, if the plist does not exist, it is
safe to assume the user did not change the setting.

Also makes sure that Chrome does not have this behavior when running on
pre-lion Macs.

BUG=373356

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

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

4 weeks agoRevert "[Mac] Reland r276808 Use a native MessagePump instead of a MessagePumpDefault"
miletus@chromium.org [Tue, 24 Jun 2014 20:05:16 +0000 (20:05 +0000)]
Revert "[Mac] Reland r276808 Use a native MessagePump instead of a MessagePumpDefault"

This reverts commit e82339d44d0dde725295785b9fea8c448d382621.

See if this fixes the perf regression.

BUG=388360
TBR=jeremy@chromium.org
NOTRY=true

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

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

4 weeks agoMaximise timer slack for Sync thread
jeremy@chromium.org [Tue, 24 Jun 2014 19:51:02 +0000 (19:51 +0000)]
Maximise timer slack for Sync thread

Add an API to base::Thread to allow specifying timer slack on thread creation.
Use maximal Timer slack for Sync thread.

BUG=356804

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

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

4 weeks agoRemove base/platform_file.*
rvargas@chromium.org [Tue, 24 Jun 2014 05:40:09 +0000 (05:40 +0000)]
Remove base/platform_file.*

Fix remaining users of platform_file.h

BUG=322664
R=brettw@chromium.org
TBR=thestig@chromium.org, satorux@chromium.org, jochen@chromium.org, rockot@chromium.org, viettrungluu@chromium.org

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

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

4 weeks agoIncrease the test output line limit 50 -> 5000
phajdan.jr@chromium.org [Tue, 24 Jun 2014 03:44:18 +0000 (03:44 +0000)]
Increase the test output line limit 50 -> 5000

BUG=386566
R=sky@chromium.org

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

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

4 weeks agoUpdate the class comment for StringPiece to reflect the internal guidance
rch@chromium.org [Tue, 24 Jun 2014 02:55:21 +0000 (02:55 +0000)]
Update the class comment for StringPiece to reflect the internal guidance
that it is preferable to pass by value instead of by reference.

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

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

4 weeks agobase: Default android version to 0.0.0
boliu@chromium.org [Mon, 23 Jun 2014 18:17:37 +0000 (18:17 +0000)]
base: Default android version to 0.0.0

If the android version string cannot be parsed for some reason, default
the version to 0.0.0.

BUG=387179
TBR=ajwong@chromium.org

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

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

4 weeks agoMSan: introduce a GYP flag to control origin tracking.
earthdok@chromium.org [Mon, 23 Jun 2014 17:14:15 +0000 (17:14 +0000)]
MSan: introduce a GYP flag to control origin tracking.

By setting msan_track_origins, the user can now choose between no origin
tracking (0), simple tracking (1) and chained origins (2). The default is
still 1.

Note that this doesn't affect instrumented libraries build, and is not
currently compatible with instrumented libraries for values of
msan_track_origins other than 1.

BUG=178409
TBR=glider@chromium.org, thakis@chromium.org
NOTRY=true
TEST=compile

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

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