GitLab

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/f835a9107ae607130e9c658067a23c7334b22f83 Revert of lightweight builds archiving for mac and win64 (patchset #8 id:140001 of https://codereview.chromium.org/2279953002/ ) (miimnk@google.com)

TBR=martiniss@chromium.org,phajdan.jr@chromium.org
BUG=604452

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://codereview.chromium.org/2298873003
Cr-Commit-Position: refs/heads/master@{#415770}

Both drag and drop and context menu use the same gesture: long press on Android.
In order to maintain backwards compatibility, we need to suppress drag and drop
for links on Android and continue to use long press gesture for context menus.

This change also applies to other platforms with a touch screen, so that touch
screen behavior is consistent on all platforms.

BUG=636005

Review-Url: https://codereview.chromium.org/2247963002
Cr-Commit-Position: refs/heads/master@{#415769}

Add a PRESUBMIT rule that enforces that both ax_enums.idl and automation.idl
have matching enums so we don't accidentally let them get out of sync. Fix
one missing enum, and add a few missing enums directly to ChromeVox too.

BUG=641179
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2290763002
Cr-Commit-Position: refs/heads/master@{#415768}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/3d16d190..4877527d

Please follow these instructions for assigning/CC'ing issues:
https://github.com/v8/v8/wiki/Triaging%20issues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

TBR=hablich@chromium.org,machenbach@chromium.org,littledan@chromium.org,vogelheim@chromium.org

Review-Url: https://codereview.chromium.org/2301643002
Cr-Commit-Position: refs/heads/master@{#415767}

https://pdfium.googlesource.com/pdfium.git/+log/fc9b11358a47..21b111fcf71e

$ git log fc9b11358..21b111fcf --date=short --no-merges --format='%ad %ae %s'
2016-08-31 thestig Attempt to fix potential integer overflow in CFX_FaceCache::LoadGlyphPath().
2016-08-31 dsinclair Use CheckedNumeric for strength calculation.

BUG=641333,639506

TBR=dsinclair@chromium.org

Review-Url: https://codereview.chromium.org/2300793002
Cr-Commit-Position: refs/heads/master@{#415766}

Delete chrome/chrome_renderer.gypi.

TBR=dpranke@chromium.org

Review-Url: https://codereview.chromium.org/2301453003
Cr-Commit-Position: refs/heads/master@{#415765}

CronetUrlRequestTest#testFailures() simulates callback
failures in each response step. However, the response step
is not updated in onFailed, so maybeThrowCancelOrPause()
will throw an exception again. This results in a flake
in the Java implementation.

This CL updates response step in TestUrlRequestCallback,
and adds two more tests to make sure throwing exception
or canceling does nothing in terminal callbacks (onFailed,
onCanceled, onSucceeded)

R=kapishnikov@chromium.org
BUG=635026

Review-Url: https://codereview.chromium.org/2292113002
Cr-Commit-Position: refs/heads/master@{#415764}

StrongBinding means GpuServiceInternal is deleted when the pipe is
closed, defeating the singleton nature. By converting to Binding when the pipe is closed GpuServiceInternal is not deleted.

BUG=none
TEST=none
R=sadrul@chromium.org

Review-Url: https://codereview.chromium.org/2302523002
Cr-Commit-Position: refs/heads/master@{#415763}

And use that to add a test for ng_block_layout_algorithm that tests layout
with children.

BUG=635619
R=eae@chromium.org,glebl@chromium.org,ikilpatrick@chromium.org

Review-Url: https://codereview.chromium.org/2291273002
Cr-Commit-Position: refs/heads/master@{#415762}

Redirect all mouse input to Alt+Tab window when it's visible,
effectively disabling the mouse. Capture is reset when the widget is
destroyed.

original review: https://codereview.chromium.org/2284763002/

BUG=641171

Review-Url: https://codereview.chromium.org/2296763002
Cr-Commit-Position: refs/heads/master@{#415761}

This CL creates a BlimpView that is owned by the native
BlimpContentsViewAndroid. It is created whenever the
BlimpContentsViewAndroid is created, which is done when
BlimpContentsImpl is created.

The BlimpView needs access to the current Android Context, which
is now passed through the call to CreateBlimpContents() as a
NativeWindow which on Android is a WindowAndroid.

When a BlimpContents is available and Blimp is enabled, this also
hooks up the BlimpView to TabContentViewParent.

The BlimpView is with this patch attached to the view hierarchy,
and sends onSizeChanged events to the BlimpContentsImpl. The
touch events are piped through to the BlimpCompositorManager.

The old BlimpView in //blimp/client/app was colliding with the new,
so it has been moved in to the namespace blimp::client:app.

BUG=608759

Review-Url: https://codereview.chromium.org/2270323004
Cr-Commit-Position: refs/heads/master@{#415760}

This test is still failing after the suppression in [1]:
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=css3%2Ffilters%2Ffilter-repaint-turbulence.html

This patch marks the test as pass/fail for both debug and release.

[1] https://crrev.com/079ac671bc155405d5d544ce2fe27b017e668e8e

BUG=598051
NOTRY=true

Review-Url: https://codereview.chromium.org/2301523004
Cr-Commit-Position: refs/heads/master@{#415759}

Temporarilly set FLAGS_quic_enable_version_35, FLAGS_quic_enable_version_36 and
FLAGS_quic_enable_version_36_v2 to true.

BUG=642709
TBR=rch

Review-Url: https://codereview.chromium.org/2301623002
Cr-Commit-Position: refs/heads/master@{#415758}

This changes both cc proxies to no longer create default begin frame
sources.  In cc, it must either be the case that an external begin frame
source is provided (with use_external_begin_frame_source set) or the
output surface provides a begin frame source (with
use_output_surface_begin_frame_source).

This changes the world from three states of BFS creation (default,
output surface provided, and external) to only two (output surface
provided and external).  In the future, external will also be removed.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Review-Url: https://codereview.chromium.org/2083423006
Cr-Commit-Position: refs/heads/master@{#415757}

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/50bd0c2f545a5cff594a628ca45182915632dd82 Remove gerrit_rebase_patch_ref from run_presubmit.py (andybons@chromium.org)

TBR=martiniss@chromium.org,phajdan.jr@chromium.org
BUG=602906

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://codereview.chromium.org/2303453002
Cr-Commit-Position: refs/heads/master@{#415756}

BUG=none
TBR=thestig@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2298013004
Cr-Commit-Position: refs/heads/master@{#415755}

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/08327b978d86..7764fcd16aac

$ git log 08327b978..7764fcd16 --date=short --no-merges --format='%ad %ae %s'
2016-08-31 alexandermont Change some remaining instances of power to powerInW.

TBR=catapult-sheriff@chromium.org

Review-Url: https://codereview.chromium.org/2296763005
Cr-Commit-Position: refs/heads/master@{#415754}

- allow importing all domains in single header;
- support for <includes.h>;
- separate export macro/header for lib, protocol and exported;
- improved code for defaults.

BUG=637032

Review-Url: https://codereview.chromium.org/2296043004
Cr-Commit-Position: refs/heads/master@{#415753}

Revert of Use webrtc::VideoFrame timestamp in RTCVideoEncoder (patchset #7 id:260001 of https://codereview.chromium.org/2205623002/ )

Reason for revert:
This CL caused some regressions regarding BWE stats and HW encoder
performance. Reasons include:
1) Modifying scoped_refptr<media::VideoFrame>'s timestamp causes problems
for other clients using the same media::VideoFrame.
2) WebRTC's RTP timestamp isn't suitable for using as presentation timestamp
in Mac and Win HW encoders.

BUG=641230

Original issue's description:
> Use webrtc::VideoFrame timestamp in RTCVideoEncoder
>
> This CL fixes input timestamp mismatch in RTCVideoEncoder, which
> broke googAvgEncodeMs and googEncodeUsagePercent stats in webrtc-internals
> for hardware encoders.
> With this change, we start using WebRTC given timestamp() so that
> OveruseFrameDetector can match the timestamps and calculate the stats.
>
> BUG=597087
> TEST=googAvgEncodeMs and googEncodeUsagePercent works on Mac(H264) and
> veyron_jerry(VP8).
>
> Committed: https://crrev.com/e3195490a63d9545fb1bfe560aa21680ba0b5843
> Cr-Commit-Position: refs/heads/master@{#414589}

TBR=wuchengli@chromium.org,pbos@chromium.org,posciak@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=597087

Review-Url: https://codereview.chromium.org/2296273002
Cr-Commit-Position: refs/heads/master@{#415752}

UI remains the same.

BUG=626878
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2300433002
Cr-Commit-Position: refs/heads/master@{#415751}

The new domain is clientservices.googleapis.com.

Also, removes testing config for VarationsServiceControl
30m interval trial, since that's the client default.
Updates the parameter code for the fetch interval to fix
the typo in the trial name.

Note: Should not be submitted until we verify that the new
URL is working.

BUG=635680

Review-Url: https://codereview.chromium.org/2223263002
Cr-Commit-Position: refs/heads/master@{#415750}

Revert of Disabled flaky test AppBackgroundPageApiTest.NoJsBackgroundPage (patchset #1 id:1 of https://codereview.chromium.org/2294113003/ )

Reason for revert:
The bug is now fixed so disabling this test is no longer needed.
See https://codereview.chromium.org/2293293002/

Original issue's description:
> Disabled flaky test AppBackgroundPageApiTest.NoJsBackgroundPage
>
> BUG=642482
> TBR=stanisc@chromium.org,nick@chromium.org
>
> Committed: https://crrev.com/940f8f135b20b2e5bca8c76ad036eb98e142def0
> Cr-Commit-Position: refs/heads/master@{#415620}

TBR=nick@chromium.org,battre@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=642482

Review-Url: https://codereview.chromium.org/2299803002
Cr-Commit-Position: refs/heads/master@{#415749}

Arc enable sync pref is false by default if user never modify sync settings. This
patch enables arc sync service to start when user never manually modify sync
settings.

BUG= http://b/31156650
TEST=Manual test on a machine that sync settings is never modified.

Review-Url: https://codereview.chromium.org/2289163003
Cr-Commit-Position: refs/heads/master@{#415748}

Revert of Disable flaky TaskManagerBrowserTest.DevToolsNew{Und,D}ockedWindow tests (patchset #1 id:1 of https://codereview.chromium.org/2301493002/ )

Reason for revert:
The bug is now fixed so disabling these tests is no longer needed.
See https://codereview.chromium.org/2293293002/

Original issue's description:
> Disable flaky TaskManagerBrowserTest.DevToolsNew{Und,D}ockedWindow tests
>
> BUG=642482
> TBR=stanisc@chromium.org,nick@chromium.org:
>
> Committed: https://crrev.com/a01d97dff593c530fe6b204421bdaa39252801e9
> Cr-Commit-Position: refs/heads/master@{#415595}

TBR=nick@chromium.org,battre@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=642482

Review-Url: https://codereview.chromium.org/2298213003
Cr-Commit-Position: refs/heads/master@{#415747}

In the major CL, I did not notice the designed actually changed
the statement a little, namely "would like" was changed to
"wants"

This CL fix this.

BUG=602478, 642778

Review-Url: https://codereview.chromium.org/2296063002
Cr-Commit-Position: refs/heads/master@{#415746}

It looks like reject() is defined in extension_view.js. Perhaps this code was pulled from there at some point. In any case, it seems that this should just be a call to this.pendingLoad.reject().

BUG=642849

Review-Url: https://codereview.chromium.org/2298803004
Cr-Commit-Position: refs/heads/master@{#415745}

It's only needed for --run-in-mash, which is chromeos specific.

BUG=none
TEST=none
R=sadrul@chromium.org

Review-Url: https://codereview.chromium.org/2298323002
Cr-Commit-Position: refs/heads/master@{#415744}

The vast majority of the tests don't need content, so having to build
content all the time is a drain (especially on windows).

BUG=641026
TEST=test only changes
R=jamescook@chromium.org, dpranke@chromium.org

Review-Url: https://codereview.chromium.org/2296973002
Cr-Commit-Position: refs/heads/master@{#415743}

This modifies our build process so regardless of whether debug_devtools is enabled
a release mode version is built specifically for our tests. This
doesn't affect the normal use or development of DevTools because it creates
a separate copy of the app in release mode in "out/Release/resources/inspector_release".

People can opt out of using release mode by passing "--additional-drt-flag='--debug-devtools'"
to run_layout_tests.py if they are trying to debug a layout test.

This will decrease the time of our test suite by ~25% (20 seconds faster).

Stats for running our tests:
16 process - debug mode: average 83 seconds (3 runs)
16 process - release mode: average 63 seconds (4 runs)

Even in fully parallel, this speeds up our test from ~50 seconds to ~40 seconds,
however fully parallel mode had a wide variance from 40 to 60 seconds
between different runs.

Tests included:
inspector inspector-protocol inspector-enabled http/tests/inspector

Review-Url: https://codereview.chromium.org/2262743002
Cr-Commit-Position: refs/heads/master@{#415742}

My initial change was logging the internal value,
rather than the actual switch value. This change
performs the necessary conversion.

BUG=639304

Review-Url: https://codereview.chromium.org/2298223002
Cr-Commit-Position: refs/heads/master@{#415741}

https://chromium.googlesource.com/skia.git/+log/982e7c502f44..19ff1035d333

$ git log 982e7c502..19ff1035d --date=short --no-merges --format='%ad %ae %s'
2016-08-31 egdaniel Always add a barrier when old layout was general in vulkan.
2016-08-31 senorblanco Revert of Screenspace AA tessellated path rendering. (patchset #37 id:730001 of https://codereview.chromium.org/1152733009/ )

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
TBR=bsalomon@google.com

Review-Url: https://codereview.chromium.org/2297033003
Cr-Commit-Position: refs/heads/master@{#415740}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/a7db6458..3d16d190

Please follow these instructions for assigning/CC'ing issues:
https://github.com/v8/v8/wiki/Triaging%20issues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

TBR=hablich@chromium.org,machenbach@chromium.org,littledan@chromium.org,vogelheim@chromium.org

Review-Url: https://codereview.chromium.org/2295293002
Cr-Commit-Position: refs/heads/master@{#415739}

This is refactoring to allow MusBrowserCompositorOutputSurface to
provide an ExternalBeginFrameSource that it owns rather than
passing ownership to Display.

This should also make it possible to remove begin frame sources from
OutputSurface in the future after the OutputSurface/CompositorFrameSink
split.
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Review-Url: https://codereview.chromium.org/2286273003
Cr-Commit-Position: refs/heads/master@{#415738}

Add steel@ and xiyuan@ as sockets owners.

BUG=None

Review-Url: https://codereview.chromium.org/2296193004
Cr-Commit-Position: refs/heads/master@{#415737}

Makes them owned by the same people that own other Memory.*
histograms.

Also, adds a summary for a TBD histogram and adds units to
process count histograms.

BUG=642626

Review-Url: https://codereview.chromium.org/2300643003
Cr-Commit-Position: refs/heads/master@{#415736}

Revert of Increase maximum size of the HPACK decoder dynamic table to 64 kB. (patchset #1 id:1 of https://codereview.chromium.org/2300683002/ )

Reason for revert:
Suspected patch causing Android Cronet Builder and other bots to fail BidirectionalStreamTest#testSimpleGet. Will re-land if not.

Original issue's description:
> Increase maximum size of the HPACK decoder dynamic table to 64 kB.
>
> * Send out SETTINGS_HEADER_TABLE_SIZE = 64 kB in the initial SETTINGS frame on
>   each HTTP/2 connection.
> * Immediately notify HpackDecoder about the change so that it allows the encoder
>   to update the dynamic table size up to this limit.  It is safe to do so before
>   receiving the SETTINGS ACK, because the new limit is larger than the default 4
>   kB.  In fact, a server following RFC 7540 Section 6.5.3 word-by-word might
>   already use the larger limit before sending an ACK.
>
> BUG=642784
>
> Committed: https://crrev.com/30537e17941018ecffe781c009b39c7a8c4c8f30
> Cr-Commit-Position: refs/heads/master@{#415718}

TBR=rch@chromium.org,bnc@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=642784

Review-Url: https://codereview.chromium.org/2302573002
Cr-Commit-Position: refs/heads/master@{#415735}

This CL:
1) Adds MdTabStrip and MdTab as subclasses of TabStrip and Tab;
2) Uses those classes to implement the Harmony visuals.

BUG=635176

Review-Url: https://codereview.chromium.org/2297193002
Cr-Commit-Position: refs/heads/master@{#415734}

Code leftover from the pre-OOPIF days was causing us to exit early from
the RenderAccessibilityImpl constructor for some child frames that were
already loaded. Everything worked fine if accessibility was already enabled
when loading the frame, but if the frame was already loaded and then
accessibility was enabled, this could cause it to fail to create an
accessibility tree.

The code in RenderAccessibilityImpl is no longer needed because now we
have exactly one accessibility tree per frame.

This wasn't caught by tests because we didn't cover both scenarios, we
always enabled accessibility first.

Added two variants of existing tests that load the page first and then
enable accessibility.

BUG=640231

Review-Url: https://codereview.chromium.org/2299673002
Cr-Commit-Position: refs/heads/master@{#415733}

https://pdfium.googlesource.com/pdfium.git/+log/4d9df422c340..fc9b11358a47

$ git log 4d9df422c..fc9b11358 --date=short --no-merges --format='%ad %ae %s'
2016-08-31 npm Move maxNumDataBytes and maxNumEcBytes calculation to fix crash
2016-08-31 tsepez Make CPDF_ColorState have a CFX_ColorStateData rather than inheriting

TBR=dsinclair@chromium.org

Review-Url: https://codereview.chromium.org/2298603005
Cr-Commit-Position: refs/heads/master@{#415732}

- Remove unused function GetStartupTemperature()
- Move GetHardFaultCountForCurrentProcess() to the anonymous namespace.

This cleanup is a first step toward refactoring startup_metric_utils.

BUG=580207

Review-Url: https://codereview.chromium.org/2283073002
Cr-Commit-Position: refs/heads/master@{#415731}