GitLab

Sheriff: Disable ArcSettingsServiceTest.InitialSettingsAppliedForInstanceBeforeProvisioning due to flake

TBR=djacobo@chromium.org

Bug: 1004630
Change-Id: I464abc961b4877459525e81ef3464beab1389e03
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1807644Reviewed-by: Fergus Dall <sidereal@google.com>
Cr-Commit-Position: refs/heads/master@{#697092}

https://skia.googlesource.com/skia.git/+log/b2ce62ac41eb..f8486f2c5fb6

git log b2ce62ac41eb..f8486f2c5fb6 --date=short --no-merges --format='%ad %ae %s'
2019-09-17 recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com Roll recipe dependencies (trivial).

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

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/skia-autoroll
Please CC brianosman@google.com on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

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

CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-blink-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
TBR=brianosman@google.com

Bug: None
Change-Id: I72e4e4e7b01e9140fbe8874d7fa1011af3023aec
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808381Reviewed-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@{#697091}

TBR=shuchen@chromium.org
TBR=keithlee@chromium.org
TBR=azurewei@chromium.org

Bug: 1004628
Change-Id: Ifc892a0f640dab2e7fc910e2b921f1eb7393640e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1807643Reviewed-by: Fergus Dall <sidereal@google.com>
Cr-Commit-Position: refs/heads/master@{#697090}

This reverts commit 2234ecd0.

Reason for revert: A fix has been landed on the search issue. crbug.com/1003277.

Original change's description:
> Revert "Reland "[App Management] Enable App Management by default.""
> 
> This reverts commit 8875dfe6.
> 
> Reason for revert:
> Causes OS settings page to go blank after any search.
> http://crbug.com/1003277
> 
> Original change's description:
> > Reland "[App Management] Enable App Management by default."
> > 
> > This is a reland of 2fd01309
> > 
> > Original change's description:
> > > [App Management] Enable App Management by default.
> > > 
> > > Enable App Management by default. Current entry points are about:settings
> > > and about:app-management.
> > > 
> > > Bug: 906508
> > > Change-Id: I0118a4b51eda6e65bc72186e771fc2ede2acb45f
> > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1610658
> > > Reviewed-by: Dominick Ng <dominickn@chromium.org>
> > > Commit-Queue: Jeevan Shikaram <jshikaram@chromium.org>
> > > Cr-Commit-Position: refs/heads/master@{#659341}
> > 
> > Bug: 906508
> > Change-Id: I08def499ce0c87ff44ac722b3bdcd8a238938959
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1782363
> > Commit-Queue: Jeevan Shikaram <jshikaram@chromium.org>
> > Commit-Queue: Dominick Ng <dominickn@chromium.org>
> > Auto-Submit: Jeevan Shikaram <jshikaram@chromium.org>
> > Reviewed-by: Dominick Ng <dominickn@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#692990}
> 
> TBR=dominickn@chromium.org,jshikaram@chromium.org
> 
> # Not skipping CQ checks because original CL landed > 1 day ago.
> 
> Bug: 906508, 1003277
> Change-Id: Iff5bbb0f33d87bd4931e2f756e21e287af5d6edc
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1803902
> Reviewed-by: James Cook <jamescook@chromium.org>
> Commit-Queue: James Cook <jamescook@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#696472}

TBR=jamescook@chromium.org,dominickn@chromium.org,jshikaram@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 906508, 1003277
Change-Id: Id9bfd21cd5c6b37b4f91e00158d08d5bff2bda1a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808662Reviewed-by: Jeevan Shikaram <jshikaram@chromium.org>
Reviewed-by: Dominick Ng <dominickn@chromium.org>
Commit-Queue: Jeevan Shikaram <jshikaram@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697089}

TBR=isherman@chromium.org

Bug: 1003126
Change-Id: Id26978b2f9e2017acd9c14bb27b41a6deeba49e6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1798077Reviewed-by: Joel Hockey <joelhockey@chromium.org>
Reviewed-by: Julian Watson <juwa@google.com>
Commit-Queue: Joel Hockey <joelhockey@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697088}

crrev.com/697042 changed the autofocus behavior, and this CL just changes
the IDL attribute.

Note: We need to add 'autofocus' to svg_attribute_names.json5
because generated code for 'Reflect' needs svg_names::kAutofocusAttr.
html_names::kAutofocusAttr and svg_names::kAutofocusAttr have
an identical value.

I2IS: https://groups.google.com/a/chromium.org/d/msg/blink-dev/g8MP8KUf96w/360rLewrDwAJ

Bug: 995679
Change-Id: I4c7a1ad72d14c4e1e54d8455935b1255805fb384
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808617
Commit-Queue: Kent Tamura <tkent@chromium.org>
Reviewed-by: Keishi Hattori <keishi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697087}

The legacy drive sync client is being deleted in favour of switching to
DriveFS, so remove the feature flag and clean up code using the flag.

Bug: 1002339
Change-Id: I51035d863569d484748a22dd6b07b5aaf26a7057
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1797943
Commit-Queue: Austin Tankiang <austinct@chromium.org>
Reviewed-by: Sam McNally <sammc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697086}

Prior work has already detangled a lot of RenderWidgetHostImpl and
RenderViewHostImpl. Moving the registration of the RenderViewHostImpl
with the RenderWidgetHostImpl as late as possible in the construction
sequence further decouples the two classes.

This is preparation for moving RWHI ownership to RenderFrameHost.

Bug: 545684
Change-Id: I5a21a60d1eaeb77fe0270bc2506248162405fb37
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808184Reviewed-by: Ken Buchanan <kenrb@chromium.org>
Commit-Queue: Albert J. Wong <ajwong@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697085}

Bug: 1004585
Change-Id: I446a15b620c25b6f9fa92fb0d5fced200141f644
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808621Reviewed-by: Fergus Dall <sidereal@google.com>
Cr-Commit-Position: refs/heads/master@{#697084}

https://chromium.googlesource.com/chromiumos/chromite.git/+log/ca95a4ccd809..08a7e90d730a

git log ca95a4ccd809..08a7e90d730a --date=short --no-merges --format='%ad %ae %s'
2019-09-17 chrome-bot@chromium.org Update config settings by config-updater.

Created with:
  gclient setdep -r src/third_party/chromite@08a7e90d730a

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/chromite-chromium-autoroll
Please CC chrome-os-gardeners@google.com on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

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

CQ_INCLUDE_TRYBOTS=luci.chromium.try:chromeos-kevin-rel
TBR=chrome-os-gardeners@google.com

Bug: None
Change-Id: Idee36bcdf3b04a836c46acae860560a17284877b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808104Reviewed-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@{#697083}

https://pdfium.googlesource.com/pdfium.git/+log/542b7933970d..0c0b073aed22

git log 542b7933970d..0c0b073aed22 --date=short --no-merges --format='%ad %ae %s'
2019-09-17 thestig@chromium.org Remove binary data from two .in files.
2019-09-17 thestig@chromium.org Undo the fix from commit 870b5b67.
2019-09-17 thestig@chromium.org Convert bug_512557.in to bug_512557.pdf.
2019-09-17 thestig@chromium.org Change duplicate test file presubmit check to ignore deletes.
2019-09-17 thomasanderson@chromium.org Fix build with pdf_is_complete_lib=true and pdf_is_standalone=true

Created with:
  gclient setdep -r src/third_party/pdfium@0c0b073aed22

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/pdfium-autoroll
Please CC pdfium-deps-rolls@chromium.org on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

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

TBR=pdfium-deps-rolls@chromium.org

Bug: None
Change-Id: I11c07a24e570867614f0b81c8a4ffbbf38c6fe24
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808102Reviewed-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@{#697082}

TBR=hta@chromium.org

Bug: 1004546
Change-Id: I9b93678188ecb8743677fbc5912ae70f9b483377
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808660Reviewed-by: Fergus Dall <sidereal@google.com>
Cr-Commit-Position: refs/heads/master@{#697081}

Bug: 982201, 997581
Change-Id: Ida6c3e961a5bd23d89a8c19d74d7ce05a96a3f1b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1804195
Commit-Queue: Ricky Liang <jcliang@chromium.org>
Reviewed-by: Dan Sanders <sandersd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697080}

In https://chromium-review.googlesource.com/c/chromium/src/+/1788138,
these tests were enabled on Android, however, they seemed to still
fail on Nexus 6.

The failure is due to tab.action_runner.WaitForJavaScriptCondition(
    'window.domAutomationController._loaded') timed out

BUG=609629
TEST=bot
TBR=kbr@chromium.org

Change-Id: I8ec2e1c2a708cca23c960e7a3232e11598a5f5f8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808584
Commit-Queue: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697079}

VP8 control ID is is set to [V4L2_CTRL_TYPE_VP8_FRAME_HEADER]
but should be [V4L2_CID_MPEG_VIDEO_VP8_FRAME_HEADER] instead.
The wrong control ID causes the VP8 decoder to fail.

BUG=965375
TEST=ran tast tests with video.DecodeAccelVP8 and
video_decode_accelerator_tests on veyron_minnie-kernelnext

Change-Id: Ia7da9ed513da07450fd96969461ac008245bfad8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1807637Reviewed-by: Alexandre Courbot <acourbot@chromium.org>
Commit-Queue: Alexandre Courbot <acourbot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697078}

This change converts FileChooser mojom interface clients in pepper plugin
and blink to use BrowserInterfaceBroker, along with mocks in tests.

Bug: 936482
Change-Id: I48bf6d6adcf90a4991f8b3af17d1b3bc29d591a1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1788205Reviewed-by: Kent Tamura <tkent@chromium.org>
Reviewed-by: Bill Budge <bbudge@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
Commit-Queue: Oksana Zhuravlova <oksamyt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697077}

We need media::VideoFrame for passing information of output buffers in
VideoDecoder migration at ARC++. This CL adds the mojo struct for it.

BUG=b:136716638
TEST=components_unittests --gtest_filter=VideoAcceleratorStructTraitsTest.*

Change-Id: I8ba1a47d20bdc07c5281e57ce7d78636144b36a0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1697502
Commit-Queue: Chih-Yu Huang <akahuang@chromium.org>
Reviewed-by: Dominick Ng <dominickn@chromium.org>
Reviewed-by: Hirokazu Honda <hiroh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697076}

Fix recipes for 1aauto, cvs, safishing,
sign_up_pass/adobe, notonthehighstreet (bug: 984664)

Enable indigo after bug 951847 fix

Disable wiley (bug: 996478) and modcloth (bug: 1003044)

Remove cryptopia (site no longer exists)

Update cipd versions for autofill and password recipe/archive files

Bug: 951847,984664,996478,1003044
Change-Id: I06a6dfe315a502189f68eb6f6f4b3b3a8ff94df0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1807178
Commit-Queue: Ian Struiksma <ianstruiksma@google.com>
Reviewed-by: Brian Sheedy <bsheedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697075}

Before this CL, the distilled content is injected on DidFinishLoad(),
or when the distillation finishes if it's later than DidFinishLoad().
When the DidFinishLoad() event propagates to TalkBack, the distilled
content is usually not there yet, so TalkBack would announce the
content of the place-holding page.

This CL populates the distilled content in DocumentLoadedInFrame()
instead of DidFinishLoad(). DocumentLoadedInFrame() is late enough to
execute JavaScript, and is early enough so that it's more likely that
the title and content can be picked up by TalkBack instead of the
placeholder.

If distillation is finished by DocumentLoadedInFrame(), onload() event
would also be delayed, so that the accessibility focus is more likely
to be on the web content. Otherwise, the focus is usually on the close
button of the CustomTab (CCT), or nowhere.

Note that this CL doesn't guarantee the distilled title is announced
by TalkBack. If distillation finishes later than
DocumentLoadedInFrame(), or if for some reason the accessibility focus
is on the close button of the CCT, the title would go unannounced like
before.

Bug: 811417, 803474
Change-Id: Iaf2ff988b427ca9af582622597b505e7a8aca8e1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1791048Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Reviewed-by: Matthew Jones <mdjones@chromium.org>
Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697074}

# This is an automated release commit.
# Do not revert without consulting chrome-pmo@google.com.
NOAUTOREVERT=true
TBR=kariah@chromium.org

Change-Id: I4c4df1c63bf7f8c193fc7b3ff955493811d404ab
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808176Reviewed-by: Chrome Release Bot (LUCI) <chrome-official-brancher@chops-service-accounts.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#697073}

https://chromium.googlesource.com/angle/angle.git/+log/b5560486ad22..db7a36f45698

git log b5560486ad22..db7a36f45698 --date=short --no-merges --format='%ad %ae %s'
2019-09-17 zmo@chromium.org Revert "Vulkan: SwiftShader integration."
2019-09-16 jmadill@chromium.org Capture/Replay: Refactor cpp replay param writes.
2019-09-16 jmadill@chromium.org Vulkan: SwiftShader integration.
2019-09-16 zmo@chromium.org Skip a angle_end2end_tests on Linux Intel OZone.

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

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/angle-chromium-autoroll
Please CC geofflang@chromium.org on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

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

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
TBR=geofflang@chromium.org

Bug: chromium:b/140251624,chromium:b/140251624,chromium:1003971
Change-Id: Ia3e420836a73229cc1752173a5e9c8b58d982eac
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808587Reviewed-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@{#697072}

DriveFS is going to be used regardless of its feature flag, and the
legacy drive sync client will be deleted, so clean up code that relies
on the legacy drive sync client.

Bug: 1002339
Change-Id: I2bf7f3d587ab856f493802a27a7f6f8333551f94
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1800139Reviewed-by: Joel Hockey <joelhockey@chromium.org>
Commit-Queue: Austin Tankiang <austinct@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697071}

conformance/rendering/multisample-corruption.html

It fails frequent enough, so just mark it as Fail.

BUG=1004581
TEST=bot
TBR=kbr@chromium.org

Change-Id: Iaffb3efa7bfbee23d25636915baa443e1ad31e51
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808577Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Commit-Queue: Zhenyao Mo <zmo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697070}

This CL adds a checkbox in "Themes" settings screen, which is only
shown when "System default" or "Dark" is selected. The checkbox can
toggle the feature of auto-darkening web contents.

Bug: 993391
Change-Id: Ic18930e8986747923f3f00a2edf4c47aed2923c5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1762543
Commit-Queue: Lijin Shen <lazzzis@google.com>
Reviewed-by: Theresa  <twellington@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697069}

This CL converts TestServicePtr and TestServiceRequest
to new Mojo types.

It uses Remote, PendingReceiver and Receiver instead of
QueryableDataStorePtr, QueryableDataStoreRequest and Binding.

Bug: 955171
Change-Id: I61f3d6b66ac6f5faffe533800bbe482a2db772ce
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1803948
Commit-Queue: Julie Kim <jkim@igalia.com>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Oksana Zhuravlova <oksamyt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697068}

This change doesn't include any action taken by the InfobarBadgeTabHelper
since currently there is no significant state change needed. A following
CL will have changes to the model that makes the banner dismissal action
significant. A test for this method will be added to BadgeMediatorUnittests
with that CL.

Bug: 976901
Change-Id: I44df4e1df9282c6c43c52ae6c40178995c714962
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1800521
Commit-Queue: Chris Lu <thegreenfrog@chromium.org>
Reviewed-by: Sergio Collazos <sczs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697067}

Follow up for https://skia-review.googlesource.com/c/skia/+/240824. This
change rebaselines the layout test which passes after the change above
and switches passthrough tests skipped previously to failed.

R=fserb@chromium.org, kbr@chromium.org

Bug: 900706
Change-Id: I1e783570cf73835c09dfb9cfd503abb42facd8c1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808076
Commit-Queue: Khushal <khushalsagar@chromium.org>
Commit-Queue: Kenneth Russell <kbr@chromium.org>
Auto-Submit: Khushal <khushalsagar@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697066}

This CL changes linear resampling to scroll predictor's default predictor
type and also fix related unit tests.
This is to prepare the enabling of scroll resampling. So when the flag
turn on, it will default to use linear resampling predictor type.

This is to get a better scroll experience on scrolling, detail see the doc:
https://docs.google.com/document/d/1KrlQ0m5Ty9RKE1IorjEXDldYFJvUV_Ww10lnjh-TRN0/

Bug: 1000682
Change-Id: I3928428700a36f8c25ac2c0c1294b3a4478bc579
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1786868
Commit-Queue: Ella Ge <eirage@chromium.org>
Reviewed-by: David Bokan <bokan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697065}

https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/e76231afda98..007200e00a46


Created with:
  gclient setdep -r src-internal@007200e00a46

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll
Please CC jbudorick@google.com on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

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

CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome
TBR=jbudorick@google.com

Bug: chromium:None
Change-Id: I6cbbf07abb6ba53f07010875dc0b659cbe7665f0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808103Reviewed-by: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com>
Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#697064}

https://skia.googlesource.com/skia.git/+log/b47704b0bd34..b2ce62ac41eb

git log b47704b0bd34..b2ce62ac41eb --date=short --no-merges --format='%ad %ae %s'
2019-09-17 recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com Roll recipe dependencies (trivial).

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

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/skia-autoroll
Please CC brianosman@google.com on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

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

CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-blink-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
TBR=brianosman@google.com

Bug: None
Change-Id: Iafac4d3ddb106117b0a5e2238190c34fe6fd1de4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808230Reviewed-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@{#697063}

conformance/textures/image_bitmap_from_video/conformance/textures/image_bitmap_from_video/

BUG=1002269
TEST=bot
TBR=kbr@chromium.org

Change-Id: Ic02ff5afca8012e218822f4f5d6ba0134b148f53
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808411Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Commit-Queue: Zhenyao Mo <zmo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697062}

TBR=ftirelo@chromium.org
TBR=dvadym@chromium.org
TBR=cfroussios@chromium.org
TBR=mahmadi@chromium.org
TBR=battre@chromium.org
TBR=tmartino@chromium.org
TBR=kolos@chromium.org
TBR=sebsg@chromium.org
TBR=estade@chromium.org
TBR=rogerm@chromium.org

Bug: 1004585
Change-Id: I5f27138ce281f342cf52f6681139de50ce6f8fff
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808295Reviewed-by: Fergus Dall <sidereal@google.com>
Cr-Commit-Position: refs/heads/master@{#697061}

https://chromium-review.googlesource.com/c/chromium/src/+/1801232 made
a mistake and disabled the wrong test suite.

BUG=1002313
TEST=bot
TBR=kbr@chromium.org,jmadill@chromium.org

Change-Id: Ibd89171c0a6787f03a08b1c5f6dcafeff4b2b7ba
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808500Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Commit-Queue: Zhenyao Mo <zmo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697060}

Currently we remove canceled and non resumable downloads on Android.
However, we still see some users having memory issues with thousands of
download records on other platforms.

This CL removes canceled, and all interrupted download after 90 days
on all platforms.

Browser tests will be added in following CL.

Bug: 851650
Change-Id: I4decd1626ee088c646c20d5c4bca10fab6378324
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1799694
Commit-Queue: Xing Liu <xingliu@chromium.org>
Reviewed-by: Min Qin <qinmin@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697059}

Implements reading config from Finch with unittest.

Bug: 1004498
Change-Id: Ib4326314d849cf66d8e702359a71fba74649471d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808098
Commit-Queue: Hesen Zhang <hesen@chromium.org>
Auto-Submit: Hesen Zhang <hesen@chromium.org>
Reviewed-by: Xing Liu <xingliu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697058}

This fixes a regression introduced between PS14 and PS15 of CL [1] where
blend_toward_max_contrast() was switched for blend_for_min_contrast(). But for
dimmed text, we specifically don't want to blend for min contrast because this
will usually result in the same text color. What we want is a new text color
distinct from the original text color, specifically with less contrast.

Also updated COLOR_OMNIBOX_RESULTS_ICON while we're at it to use
derive_default_icon_color(), which logically makes more sense, and gives results
quite close to the colors before CL [1].

Updated colors: [2]

[1] https://chromium-review.googlesource.com/c/chromium/src/+/1754155/14..15
[2] https://docs.google.com/spreadsheets/d/1Nev_DSPECQ9vGrFDO-ARjLudL6O2NjOTnBu_9mTq_cU/edit?usp=sharing

Bug: 1004135
R=pkasting

Change-Id: I243f809b409de7178408601a1fb80e4bddce7087
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808191
Commit-Queue: Thomas Anderson <thomasanderson@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Auto-Submit: Thomas Anderson <thomasanderson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697057}

DriveFS is going to be used regardless of its feature flag, and the
legacy drive sync client will be deleted, so clean up code that relies
on the legacy drive sync client.

Bug: 1002339
Change-Id: I85827f39bd174362a9a9e37eb5721384fea37d69
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1798947Reviewed-by: Min Qin <qinmin@chromium.org>
Commit-Queue: Austin Tankiang <austinct@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697056}

Supports Interface.operations, operation_groups.  Also fixes
several minor things.

Bug: 839389
Change-Id: I77fbd80172d3d7ca5f385a1bc2ceabc40079556f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1800856
Commit-Queue: Hitoshi Yoshida <peria@chromium.org>
Reviewed-by: Hitoshi Yoshida <peria@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697055}

BUG=1002351

Change-Id: I8df9f7f59599d5874c7170555a285b46141bb125
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1803035Reviewed-by: Dominick Ng <dominickn@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Nancy Wang <nancylingwang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697054}

Bug: 998949
Change-Id: I29fbedca05f986d33f486684c1db8909bcd6b14a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1805296Reviewed-by: Yusuf Ozuysal <yusufo@chromium.org>
Reviewed-by: Mei Liang <meiliang@chromium.org>
Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697053}