GitLab

Three video related tests are failing flakily on the Android FYI
trybots. Failures have been seen on Nexus 9 and Nvidia Shield primarily,
so will set them flaky for nvidia boards.

TBR: kbr@chromium.org
Bug: 891456
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
Change-Id: I64d768be4cabf246b6e0ab007c87c2292a8adf42
Reviewed-on: https://chromium-review.googlesource.com/c/1263037Reviewed-by: kylechar <kylechar@chromium.org>
Commit-Queue: kylechar <kylechar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597155}

Before this patch, the overlay was displayed when Autofill Assistant was
created.

This is confusing: the overlay is meant to reflect the fact that scripts
are running, but no scripts are running at that point and there might
never be any scripts.

With this patch, the overlay is only displayed when a script is run.

Bug: 806868
Change-Id: I2e1916a0253efb164b4b000c92f2cfef6a1703ad
Reviewed-on: https://chromium-review.googlesource.com/c/1264677Reviewed-by: Ganggui Tang <gogerald@chromium.org>
Commit-Queue: Ganggui Tang <gogerald@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597154}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/060ca87d..949f77f4

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_trusty_blink_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel

TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com

Change-Id: I5cfbc94e8650675582b8f2e6169b9cb6943e6e93
Reviewed-on: https://chromium-review.googlesource.com/c/1264735Reviewed-by: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com>
Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#597153}

This reverts commit e9f339db.

Reason for revert: issue has been solved.

Original change's description:
> [WPT export] More temporary logging to debug crash
> 
> The working theory is that the locale isn't correctly set in the
> Python process. The interactive shell on the bot does have a UTF-8
> locale, but the environment of the Python process could be different. To
> verify this, we add some more logging to wpt-export.
> 
> This CL will be reverted once the issue is resolved.
> 
> Bug: 891831
> Change-Id: Ice6478a2b4f3805f2a68b4e2b782f0f23e9aeb99
> Reviewed-on: https://chromium-review.googlesource.com/c/1261974
> Reviewed-by: Quinten Yearsley <qyearsley@chromium.org>
> Commit-Queue: Robert Ma <robertma@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#596782}

TBR=qyearsley@chromium.org,robertma@chromium.org

Change-Id: Icf05a7a7163a948fba10ff3991df2ccc34b4865a
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 891831
Reviewed-on: https://chromium-review.googlesource.com/c/1265017Reviewed-by: Robert Ma <robertma@chromium.org>
Commit-Queue: Robert Ma <robertma@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597152}

Using wpt-import in Chromium 72378c90.
With Chromium commits locally applied on WPT:
65d72632 "Make DTMF tone change more deterministic"


Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/25878

Note to sheriffs: This CL imports external tests and adds
expectations for those tests; if this CL is large and causes
a few new failures, please fix the failures by adding new
lines to TestExpectations rather than reverting. See:
https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md

NOAUTOREVERT=true
TBR=markdittmer

No-Export: true
Change-Id: I01f5467192473c33c0fbec6d80115159657bba63
Reviewed-on: https://chromium-review.googlesource.com/c/1264738
Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org>
Reviewed-by: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597151}

Speculative fix to single process mash content_browsertests
start-up hang by allowing nested task while waiting for display.

Bug: 892173
Change-Id: Ia146293d8f2553cd8233feae80786e56402c7c55
Reviewed-on: https://chromium-review.googlesource.com/c/1262925Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597150}

Add two UKM fields to Media.Engagement.SessionFinished to
track the total playbacks from audio contexts and media
elements.

BUG=887619

Change-Id: Icb1ed1300114b09737bab2393b5c3d40e870fc5f
Reviewed-on: https://chromium-review.googlesource.com/c/1236924Reviewed-by: Steven Holte <holte@chromium.org>
Commit-Queue: Becca Hughes <beccahughes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597149}

BUG=892436

Change-Id: I1576d16bb1c370710a6374482fd146e639145b91
Reviewed-on: https://chromium-review.googlesource.com/c/1263353Reviewed-by: Ned Nguyen <nednguyen@google.com>
Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org>
Commit-Queue: Victor Miura <vmiura@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597148}

Change-Id: I7c061bdd3c4cc4ee084b0a9311a0c1d65f533494
TBR: dcheng@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/c/1259290
Commit-Queue: Mustaq Ahmed <mustaq@chromium.org>
Reviewed-by: Mustaq Ahmed <mustaq@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597147}

Opening a drop-down menu in a webpage will create a new
RenderWidget with a new LayerTreeHostImpl, whose value
of the top_controls_shown_ratio_ (initialized to 0) will
be pushed to the browser when a new compositor frame is
generated. This will cause top-chrome to hide briefly
on opening the menu which will result in closing the
menu immediately.

This CL fixes the issue by allowing only the main frame
widget to set the shown ratio.

BUG=891471
TEST=Manual, added a new browser test.

Change-Id: I8ead7f903d56e68523a7654abc21aa6e5ca7b690
Reviewed-on: https://chromium-review.googlesource.com/c/1258348Reviewed-by: David Bokan <bokan@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Ahmed Fakhry <afakhry@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597146}

Ensure the print button starts out with a default "Print" value instead
of waiting for the destination to load to display text.

Bug: 892093
Change-Id: I1ca54476107a112ac4cccde463b8f1def1e6e501
Reviewed-on: https://chromium-review.googlesource.com/c/1262619
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597145}

The current implementation of find in the the PDF viewer will not find
a match for text, if the matching text in the document is broken over
two lines. This is rooted in the fact that the page text will have
extra characters like \r and \n representing the line break in it.

This CL adds in collapsing all line breaks and whitespace regions into
single ' ' characters when searching. A test has been added to confirm
that searching over line breaks works correctly.

Originally landed as https://chromium-review.googlesource.com/c/1258042
Reverted in https://chromium-review.googlesource.com/c/1262618, due to
an old version of the CL being landed.

BUG=chromium:58402

Change-Id: Iad69ab5e9965658f89df4b209dba06a322b15374
Reviewed-on: https://chromium-review.googlesource.com/c/1262658
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597144}

Only DetachTimeFg16x is added because it is the most
different variation from control, while other variations
test values in between.

Bug: 847501
Change-Id: Ifbf70af688249d48772779229e2af56a12e1b274
Reviewed-on: https://chromium-review.googlesource.com/c/1263042Reviewed-by: Robert Kaplow (sloooow) <rkaplow@chromium.org>
Commit-Queue: Etienne Pierre-Doray <etiennep@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597143}

TBR: weiliangc@chromium.org
Bug: none
Change-Id: I0809911bbd85d7b7431c052422ce28437b33af4a
Reviewed-on: https://chromium-review.googlesource.com/c/1264817
Commit-Queue: kylechar <kylechar@chromium.org>
Reviewed-by: kylechar <kylechar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597142}

https://skia.googlesource.com/skia.git/+log/b3ce196a4d9d..032f2756df64


git log b3ce196a4d9d..032f2756df64 --date=short --no-merges --format='%ad %ae %s'
2018-10-05 recipe-roller@chromium.org Roll recipe dependencies (trivial).


Created with:
  gclient setdep -r src/third_party/skia@032f2756df64

The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel
TBR=mtklein@chromium.org

Change-Id: Icf07e3b693c664b66ab4b8b2228ff08c55672549
Reviewed-on: https://chromium-review.googlesource.com/c/1264630Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#597141}

The tests are failed due to FieldTrialSynchronizer is created early and posts
task to BrowserThread::UI before the UI thread is initialized. In this case,
the FieldTrialSynchronizer doesn't need to post task to UI thread since the
renderers are not created yet.

Bug: 892636
Change-Id: Ib7d46709f3fca6a749b82d0ba2f601dcf551a8d0
Reviewed-on: https://chromium-review.googlesource.com/c/1264737
Commit-Queue: Xi Han <hanxi@chromium.org>
Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597140}

The same anchor element may be stored more than once in
anchor_element_metrics_sender. Use a HashSet to avoid
duplicate storage. This allows the
navigation prediction logic in browser process is unable
to accurately determine the visibility ratio of a target
HREF.

Change-Id: I087e653f15094f8e32d66daa0d2cc6464d90f306
Bug: 891719
Reviewed-on: https://chromium-review.googlesource.com/c/1258822Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Commit-Queue: Tarun Bansal <tbansal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597139}

Style sharing is gone. No need for SetUnique here.

Change-Id: I1959397a2939ee2ab86e7301ac4fb5a47a8cd100
Reviewed-on: https://chromium-review.googlesource.com/c/1264581Reviewed-by: Anders Ruud <andruud@chromium.org>
Commit-Queue: Rune Lillesveen <futhark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597138}

This is a reland of 581a728b
Fix for the revert in http://crrev.com/c/1264175

Original change's description:
> synthetic gesture: Allow high-frequency dispatch.
>
> For synthesized scroll/drag/pinch gestures, dispatch the events
> at a high frequency (120Hz) so that there's at least one event in
> each frame. Dispatching events at a lower frequency (60Hz) means
> the timer can go out of sync with begin-frame, and so there can
> be frames where there was to event dispatch. Our telemetry code
> reports this as jank, which is incorrect. Dispatching at a higher
> frequency for the scroll/drag/pinch gestures resolves this issue.
>
> BUG=783034
>
> Change-Id: I1102f76ed743231cbb10fdb872e8c66f895fa223
> Reviewed-on: https://chromium-review.googlesource.com/c/1255152
> Reviewed-by: Victor Miura <vmiura@chromium.org>
> Reviewed-by: Navid Zolghadr <nzolghadr@chromium.org>
> Reviewed-by: Lan Wei <lanwei@chromium.org>
> Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#596507}

TBR=vmiura@, nzolghadr@, lanwei@ since reland without changes

Bug: 783034
Change-Id: I7cb96177df54729b24f5414c897cca97456e9484
Reviewed-on: https://chromium-review.googlesource.com/c/1263882Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org>
Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597137}

This is a first step in moving the system-wide metric measurement code out of base/process/

TBR: jam@chromium.org, dalecurtis@chromium.org
Bug: 891864
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
Change-Id: I615190d5413937640a7bd29b68b18028445e4c88
Reviewed-on: https://chromium-review.googlesource.com/c/1260003Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Reviewed-by: Takashi Toyoshima <toyoshim@chromium.org>
Reviewed-by: François Doray <fdoray@chromium.org>
Commit-Queue: Sébastien Marchand <sebmarchand@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597136}

The dropdown shouldn't be closed on random clicks on Mac.
The focus changes are already handled by the cross-platform code.

Bug: 888491
Change-Id: Ic5d40f7de3ae32270716e2df33e9944a1120bc75
Reviewed-on: https://chromium-review.googlesource.com/c/1264595Reviewed-by: Tommy Martino <tmartino@chromium.org>
Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597135}

Bug: 887914
Change-Id: I2e5c1be711c9242b5efc9a8d8013c95334bba0fb
Reviewed-on: https://chromium-review.googlesource.com/c/1264679Reviewed-by: Dominic Battré <battre@chromium.org>
Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597134}

Bug: 806868
Change-Id: Idc3b29bd5278166644650d7a248db0112be1373a
Reviewed-on: https://chromium-review.googlesource.com/c/1258166
Commit-Queue: Jordan Demeulenaere <jdemeulenaere@chromium.org>
Reviewed-by: Theresa <twellington@chromium.org>
Reviewed-by: Ganggui Tang <gogerald@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597133}

AcceleratorDelegate's only purporse was to create AcceleratorRouter and
route events to it. This made sense early in on mash work, but the separation
is no longer useful. So, I removed AcceleratorDelegate and made
AcceleratorRouter implement ::wm::AcceleratorDelegate.

Renamed AcceleratorRouter to PreTargetAcceleratorHandler, to better indicate
when this class is called and what it handles.

Renamed AshFocusManagerFactory::Delegate to PostTargerAcceleratorHandler.
Again, this better indicates it's responsibilities. PostTargerAcceleratorHandler
also deals with focus changes, but I figure the name
PostTargerAcceleratorHandler is better than Delegate.

BUG=none
TEST=covered by tests

Change-Id: Ib92dd35b4bb87b8c0772a499564ee7d6e86c2175
Reviewed-on: https://chromium-review.googlesource.com/c/1263417
Commit-Queue: Scott Violet <sky@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597132}

When a ServiceWorker script passes a Response back to FetchEvent.respondWith()
the browser must get the resulting Response body data back to the outer
network loader.  For non-blob bodies this is currently done by creating a new
mojo::DataPipe and copying the body into its producer handle.  The pipe's
consumer handle is then passed back to the outer loader.

In many cases, however, there is already a mojo::DataPipe for the Response
body created by the network code.  Its sub-optimal to allocate a new pipe and
copy data across to it in these cases.

This CL will attempt to extract the underlying mojo::DataPipe from the
Response and pass it directly back to the outer loader.  Currently this will
only trigger for cases where the FetchEvent.preloadResponse is passed to
respondWith().  In the future, however, we should be able to make this work
for "pass-through" cases where the Response is produced by fetch().  That
work is ongoing in https://crrev.com/c/1251841.

R=kinuko@chromium.org

Bug: 884807
Change-Id: I5a2964ff2e9463492ef72e5b48c5f383ebf36a86
Reviewed-on: https://chromium-review.googlesource.com/c/1239394
Commit-Queue: Ben Kelly <wanderview@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597131}

URLFetcher will stop working with advent of Network Service, and
SimpleURLLoader is the replacement API for most clients.
This CL migrates Android's WarmupURLFetcher and the
respective unittests away from URLFetcher.

Also, the CL relax an NOTREACHED scenario in TestSharedURLLoaderFactory,
now that some tests call ::Clone on it.

Bug: 879777
Cq-Include-Trybots: luci.chromium.try:linux_mojo
Change-Id: I7b1664a141e409cc2cf43fe8a79eb771e9a9988b
Reviewed-on: https://chromium-review.googlesource.com/c/1251641
Commit-Queue: Antonio Gomes <tonikitoo@igalia.com>
Reviewed-by: Matt Menke <mmenke@chromium.org>
Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597130}

This already happens for enterprise, guest and legacy supervised users.
This change hides powerwash section also for managed child accounts.

Bug: 891493
Change-Id: I3b573c56f2e09d4e9220415a1df0b191fe5d6cd0
Reviewed-on: https://chromium-review.googlesource.com/c/1258262Reviewed-by: Michael Giuffrida <michaelpg@chromium.org>
Commit-Queue: Bruno Santos <brunoad@google.com>
Cr-Commit-Position: refs/heads/master@{#597129}

TBR: twellington
Bug: 860467
Change-Id: I7e8b0f0391befdab1e68fd52eaab6a4480f9882a
Reviewed-on: https://chromium-review.googlesource.com/c/1264795Reviewed-by: Finnur Thorarinsson <finnur@chromium.org>
Commit-Queue: Finnur Thorarinsson <finnur@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597128}

This is a reland of 2eeb21dd

Original change's description:
> [FileSystem] Remove WebFileSystem
>
> - Adds blink.mojom.FileSystemManager to the frame and worker
>   registries, allowing it to be accessed directly from
>   ExecutionContext::GetInterfaceProvider()->GetInterface()
> - Changes ownership of blink::FileSystemDispatcher: now owned by
>   ExecutionContext
> - Omits render_frame_id_ from ChooseEntry
> - Removes WebFileSystem and content::FileSystemDispatcher
>
> Bug: 787281, 873661
> Change-Id: Ie41d48c5a206b5d1f387fcee3bd47bc30a708f0f
> Reviewed-on: https://chromium-review.googlesource.com/1217506
> Commit-Queue: Adithya Srinivasan <adithyas@chromium.org>
> Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
> Reviewed-by: Marijn Kruisselbrink <mek@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#593983}

Bug: 787281, 873661
Change-Id: Ic5d211e2102ebcfef12b51ad59c7eefc6373f5a6
Reviewed-on: https://chromium-review.googlesource.com/c/1258202Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Marijn Kruisselbrink <mek@chromium.org>
Reviewed-by: Victor Costan <pwnall@chromium.org>
Commit-Queue: Adithya Srinivasan <adithyas@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597127}

Bug: 892372
Change-Id: I139ead7a8a8c9ded7150c31e0779bb8e03728ea7
Reviewed-on: https://chromium-review.googlesource.com/c/1264496Reviewed-by: Thomas Tangl <tangltom@chromium.org>
Commit-Queue: David Roger <droger@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597126}

Ensure the rest of the UI gets re-enabled if an error is cleared
Always focus the custom pages input if the custom radio button is
clicked

Bug: 892024, 892023
Change-Id: Iec93a4ca313dff63fc143aa90c916b3111f37a9a
Reviewed-on: https://chromium-review.googlesource.com/c/1262412
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597125}

This test was flaky when run with Polymer 2, specifically failing to
find > 0 non-hidden list items in the dialog. This appears to have been
due to a timing issue with the setTimeout() call in the test setup
causing the test to fail inconsistently. This failure could not be
reproduced locally with the original test. However, exchanging
setTimeout() for waiting for the dialog to open resulted in the same
failure occurring consistently with Polymer 2 (but not with Polymer 1).
The failure was then fixed by adding Polymer.dom.flush() to ensure the
dialog had finished filtering languages before querying for the items.

Bug: 875454, 891955
Change-Id: I05c2b60ad0d12619e388a5469a0c1a0418e35a8c
Reviewed-on: https://chromium-review.googlesource.com/c/1262973Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597124}

Should close dialog on Cmd+Period not Cmd+Minus

Bug: 892103
Change-Id: Ia2795c3ccc796c9afa676d401502dc4c3efa6005
Reviewed-on: https://chromium-review.googlesource.com/c/1262433
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597123}

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/f98905e8f03c..2fb631023730


git log f98905e8f03c..2fb631023730 --date=short --no-merges --format='%ad %ae %s'
2018-10-05 borenet@google.com Reland gerrit_util: Support OAuth2 bearer tokens in CookieAuthenticator


Created with:
  gclient setdep -r src/third_party/depot_tools@2fb631023730

The AutoRoll server is located here: https://autoroll.skia.org/r/depot-tools-chromium-autoroll

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=agable@chromium.org

Change-Id: Ia6705b9bf3995b2945fa5edcdad9373bdd5fa794
Reviewed-on: https://chromium-review.googlesource.com/c/1264624Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#597122}

Apart from the code in resource_coordinator_features.cc/.h and
the code in the memory_instrumentation/ sub-directory, the code
in services/resource_coordinator/public/cpp is never used from
child processes.

This CL splits the resource_coordinator_cpp component into
3 separate components (resource_coordinator_cpp,
resource_coordinator_features and memory_instrumentation) so that
code that runs in child processes doesn't have to depend on all of
services/resource_coordinator/public/cpp/ (this is important to
avoid security concerns as we add functionality to
services/resource_coordinator/public/cpp/).

TBR=clamy@chromium.org

Change-Id: Ic2045164621d99b11d218d1a01c5ce160a4dc855
Reviewed-on: https://chromium-review.googlesource.com/c/1250006
Commit-Queue: François Doray <fdoray@chromium.org>
Reviewed-by: Raymond Toy <rtoy@chromium.org>
Reviewed-by: Sigurður Ásgeirsson <siggi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597121}

virtual/gpu/fast/canvas/canvas-arc-circumference.html
virtual/gpu/fast/canvas/canvas-drawImage-antiAlias.html
virtual/gpu/fast/canvas/canvas-ellipse-circumference.html
virtual/gpu/fast/canvas/canvas-ellipse-circumference-fill.html
virtual/gpu/fast/canvas/fillrect_gradient.html
virtual/gpu/fast/canvas/gradient-add-second-start-end-stop.html
virtual/gpu-rasterization/images/cross-fade-background-size.html

TBR=

Bug: 884166,889940,888675,802896
Change-Id: I43312a60d4e8784cd915533bda5cf54141160adb
Reviewed-on: https://chromium-review.googlesource.com/c/1262895Reviewed-by: Florin Malita <fmalita@chromium.org>
Reviewed-by: Stephen White <senorblanco@chromium.org>
Commit-Queue: Florin Malita <fmalita@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597120}

Bug: 879211
Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;master.tryserver.chromium.android:android_cronet_tester
Change-Id: Id3f234c20fb0bf28fab51a4697d8fab7b473ba73
Reviewed-on: https://chromium-review.googlesource.com/c/1207196
Commit-Queue: Paul Jensen <pauljensen@chromium.org>
Reviewed-by: Misha Efimov <mef@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597119}

https://chromium.googlesource.com/angle/angle.git/+log/2343836c4210..5476e805a321


git log 2343836c4210..5476e805a321 --date=short --no-merges --format='%ad %ae %s'
2018-10-05 oetuaho@nvidia.com Fix constructing void array zero nodes


Created with:
  gclient setdep -r src/third_party/angle@5476e805a321

The AutoRoll server is located here: https://autoroll.skia.org/r/angle-chromium-autoroll

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel

BUG=chromium:890581
TBR=cwallez@chromium.org

Change-Id: Id38bb6fa5c1d4bab11e459f18508e793b765be1c
Reviewed-on: https://chromium-review.googlesource.com/c/1264627Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#597118}

If Window::UpdateLocalSurfaceIdFromEmbeddedClient() is called, then
the ClientSurfaceEmbedder needs to be updated as well as the server.

BUG=890969
TEST=covered by test

Change-Id: Ic7fc94166cc5a3f495b01bdd63e41be16092b0d7
Reviewed-on: https://chromium-review.googlesource.com/c/1257365Reviewed-by: Fady Samuel <fsamuel@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597117}

Removes DeviceSettingsProvider dep on DeviceLocalAccount, since
the settings package is to be moved into src/chromeos/settings,
so the less deps it can have, the easier it will be to move.

This change relies on the fact that there are two identical
enums that are kept in sync - a proto one, and a native C++ one.
The code already has a dependency on the proto, so we can easily
switch to using that enum to avoid depending on DeviceLocalAccount.

Bug: 446937
Change-Id: I1bddb3d7f35b208540f6f02fa027bc11c46d0903
Reviewed-on: https://chromium-review.googlesource.com/c/1264582Reviewed-by: Julian Pastarmov <pastarmovj@chromium.org>
Commit-Queue: A Olsen <olsen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597116}