GitLab

This is a reland of d13a7e92.

Safe to land; bad service deploy caused the original notification.

Original change's description:
> Add Windows Release CI builders to milo
>
> Adds the compile builder, and the tester.
>
> Bug: 790710, 790644
> Change-Id: I83de37258aadc8d8e79cde3c32baca9d7d994c98
> Reviewed-on: https://chromium-review.googlesource.com/940431
> Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
> Reviewed-by: Ryan Tseng <hinoka@chromium.org>
> Commit-Queue: Stephen Martinis <martiniss@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#540314}

Bug: 790710, 790644
Change-Id: Ia7d91fc3be49f7c0dbe32c8ac4424fb3ce71b124
Reviewed-on: https://chromium-review.googlesource.com/944727Reviewed-by: Stephen Martinis <martiniss@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Stephen Martinis <martiniss@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540342}

Keep them tagged as scoped_refptr. Returning a raw pointer sort of
works, but because some refcounts have already been acquired in the
process, we're relying on at least one of those refcounts sticking
around long enough for the raw pointer to be cashed in.

LoginHandler is rather a mess and really should get rewritten (move
UI/IO hop within //content, surface out of WebContentsDelegate,
get rid of NotificationRegistrar and ref-counting), but in the
meantime fix up this local issue.

Tagging with https://crbug.com/814334 in case it's related, but I
don't think it is. (This isn't turning an owned raw pointer to a
scoped_refptr. It's turning a quasi-borrowed raw pointer to a
scoped_refptr.)

Bug: 814334
Change-Id: I542799c20f7453b76f46c1cfad582a8de833a684
Reviewed-on: https://chromium-review.googlesource.com/940021
Commit-Queue: David Benjamin <davidben@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Nick Carter <nick@chromium.org>
Reviewed-by: Richard Coles <torne@chromium.org>
Reviewed-by: Charlie Harrison <csharrison@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540341}

Adds a flag (--cast-app-background-color) that changes the background
color used when displaying web apps. This color is often shown briefly
before Chromium renders a web app.

Bug: internal b/70181260
Test: use flag on ATV
Change-Id: I3bc0989aeabbf639116c1b30fffb11f66b56c4cd
Reviewed-on: https://chromium-review.googlesource.com/897466
Commit-Queue: Thoren Paulson <thoren@chromium.org>
Reviewed-by: Stephen Lanham <slan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540340}

changes:
According to the design doc: go/cros-persistent-window-bounds, add UMA
for recording the number of windows restored for persistent window
restore in multi-display scenario.

Bug: 805046
Test: tested on device, checked chrome://histograms and added test coverage
Change-Id: I4f8c815c1b88dd605a02b509b27311d567998da1
Reviewed-on: https://chromium-review.googlesource.com/942186Reviewed-by: Ahmed Fakhry <afakhry@chromium.org>
Reviewed-by: Mark Pearson <mpearson@chromium.org>
Commit-Queue: Qiang Xu <warx@google.com>
Cr-Commit-Position: refs/heads/master@{#540339}

When we're using a button to trigger tab switches, we don't want the
description or icon hinting at switching, since the main suggestion
won't switch in this state.

In addition to these tiny changes, it adds a new icon for the button,
per the UI spec.

Bug: 780835
Change-Id: I01d0c913292852e6ecaf569c1feecccbebd0646d
Reviewed-on: https://chromium-review.googlesource.com/943831
Commit-Queue: Kevin Bailey <krb@chromium.org>
Reviewed-by: Justin Donnelly <jdonnelly@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540338}

Bug: 817653
Change-Id: I29fed9cfc734439aa4a97cf591dafc440232de12
Reviewed-on: https://chromium-review.googlesource.com/943938Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Commit-Queue: Pavel Feldman <pfeldman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540337}

Bug: None
Change-Id: Ie31654577b2270f83380b06f753bb48f2740a289
Reviewed-on: https://chromium-review.googlesource.com/943344Reviewed-by: Steven Holte <holte@chromium.org>
Commit-Queue: Charlie Harrison <csharrison@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540336}

Bug: 816826
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I56839874977fb0563b556e198a5252f4d5937ed7
Reviewed-on: https://chromium-review.googlesource.com/941971Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540335}

If, on a non-secure page, the user edits an INPUT TYPE=FILE field by
selecting a file, send a signal to the browser that a form field has
been edited. This will result in the "Not Secure" string in the
security chip for the page.

Change-Id: I80528eba22a6e590d98d81c7e26389705a340833
Reviewed-on: https://chromium-review.googlesource.com/850588Reviewed-by: Emily Stark <estark@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Eric Lawrence <elawrence@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540334}

If we evict a surface before the previous CompositorFrame activates with it
as a fallback, then we will attempt to add an invalid reference and we may
end up with missing surfaces in surface aggregator. This CL combats the issue
by avoiding evicting frames during resize.

We will still only update the primary surface ID if the tab is not hidden,
and we will clear the fallback surface ID if the tab is hidden.

Bug: 813157, 672962
TBR: samans@chromium.org
Change-Id: I9f35ef24597f0bc88e139b23a2994e8b32abbc70
Reviewed-on: https://chromium-review.googlesource.com/938678Reviewed-by: Saman Sami <samans@chromium.org>
Commit-Queue: Fady Samuel <fsamuel@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#539639}
Reviewed-on: https://chromium-review.googlesource.com/943725Reviewed-by: Fady Samuel <fsamuel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540333}

This category is only exists in two files. It doesn't have a strong reason to
exist and appear as an option in tracing.

Bug: 817616
Change-Id: Icdbe89b67e5a32b1e3a994f246cf769df0261df1
Reviewed-on: https://chromium-review.googlesource.com/942191Reviewed-by: Christian Fremerey <chfremer@chromium.org>
Commit-Queue: Emircan Uysaler <emircan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540332}

Fully migrate PointerEventFactory to WebPointerEvent
by removing the last API that was expecting
WebMouseEvent.

Bug: 814471
Change-Id: Id4bebab1eefae842f7e368e0e2b328127f65e43f
Reviewed-on: https://chromium-review.googlesource.com/929861Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
Reviewed-by: Charlie Reis <creis@chromium.org>
Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
Reviewed-by: Ella Ge <eirage@chromium.org>
Reviewed-by: Mustaq Ahmed <mustaq@chromium.org>
Commit-Queue: Navid Zolghadr <nzolghadr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540331}

Launch bug: crbug.com/758005
Finch config: cl/187340325, cl/187517800

Change-Id: Ib47934856cc50d90dd71a2fea7e5a31fb9b02eb5
Reviewed-on: https://chromium-review.googlesource.com/940197Reviewed-by: Gayane Petrosyan <gayane@chromium.org>
Commit-Queue: Takumi Fujimoto <takumif@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540330}

Bug=None

Change-Id: Ic321c7b366891fba569b2a96d50f9293ee507a25
Reviewed-on: https://chromium-review.googlesource.com/941590
Commit-Queue: Peter Kotwicz <pkotwicz@chromium.org>
Reviewed-by: Matt Giuca <mgiuca@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540329}

This CL adds the initial count for download files.

The metric is recorded when Android code loaded the history database
records.

Bug: 817220
Change-Id: I571ecbca2aacc3a2be863cdf36310b32569fabe1
Reviewed-on: https://chromium-review.googlesource.com/941519
Commit-Queue: Xing Liu <xingliu@chromium.org>
Reviewed-by: David Trainor <dtrainor@chromium.org>
Reviewed-by: Mark Pearson <mpearson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540328}

Bug: 705114
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: I3f092caca160715854324acf58590c18e65a7281
Reviewed-on: https://chromium-review.googlesource.com/922940
Commit-Queue: Chris Mumford <cmumford@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540327}

The icon represents download state (not started, in progress, success,
error).

UI Mocks: https://docs.google.com/presentation/d/1GzbAoJrpW9IAQF78afh5SZLWJWErNcC67t_ctujjEus/edit#slide=id.g2b7a689b42_0_0

Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I3ab857e8f1ab6b615102aa704c3b848072f88ff7
Reviewed-on: https://chromium-review.googlesource.com/944504
Commit-Queue: Eugene But <eugenebut@chromium.org>
Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540326}

Also restores missing metric on pre-ui-refresh NTP, and adds action items to
first user action recorder.

Bug: 805635
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I75581d1e7fbee2b80fc7cbbea3ac82f4e5dbb64f
Reviewed-on: https://chromium-review.googlesource.com/941361
Commit-Queue: Justin Cohen <justincohen@chromium.org>
Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Reviewed-by: Gauthier Ambard <gambard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540325}

This extracts the logic for categorizing network errors as retry-able proxy errors to proxy_fallback.cc.

It is a bit awkward to have a single function in this file, however it serves as a good place to document general proxy fallback behaviors, and the more logic from proxy fallback could be moved here in the future.

Bug: 817052
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: Ic6356c4de3012c41a4bc91feb1e71b5586330fa0
Reviewed-on: https://chromium-review.googlesource.com/940626Reviewed-by: Peter Beverloo <peter@chromium.org>
Reviewed-by: Helen Li <xunjieli@chromium.org>
Commit-Queue: Eric Roman <eroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540324}

Bug: 729800
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I57c1ec0fbb6ec96164376d63c3cb5cfc30b5a610
Reviewed-on: https://chromium-review.googlesource.com/943402Reviewed-by: Randy Smith <rdsmith@chromium.org>
Reviewed-by: Matt Menke <mmenke@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540323}

This CL adds a nullptr check for NavigationController in
LoginHandler::Observe().

Bug: 817968
Change-Id: I08d9de516f7d3e788e644c38173ecb969ef8cb58
Reviewed-on: https://chromium-review.googlesource.com/944255Reviewed-by: Asanka Herath <asanka@chromium.org>
Reviewed-by: Mustafa Emre Acer <meacer@chromium.org>
Commit-Queue: Jun Cai <juncai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540322}

This reverts commit d13a7e92.

Reason for revert: Invalid config

Original change's description:
> Add Windows Release CI builders to milo
> 
> Adds the compile builder, and the tester.
> 
> Bug: 790710, 790644
> Change-Id: I83de37258aadc8d8e79cde3c32baca9d7d994c98
> Reviewed-on: https://chromium-review.googlesource.com/940431
> Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
> Reviewed-by: Ryan Tseng <hinoka@chromium.org>
> Commit-Queue: Stephen Martinis <martiniss@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#540314}

TBR=hinoka@chromium.org,nodir@chromium.org,tandrii@chromium.org,martiniss@chromium.org

Change-Id: Ic6c39527288cc97f06fad1846cb5f6596b213563
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 790710, 790644
Reviewed-on: https://chromium-review.googlesource.com/944607Reviewed-by: Stephen Martinis <martiniss@chromium.org>
Commit-Queue: Stephen Martinis <martiniss@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540321}

Results of diagnose_bloat:
StaticInitializersCount
        -2 count count
MonochromePublic.apk_Breakdown (-19,164 bytes)
       -84 bytes Native resources (no l10n) size
    -2,301 bytes Zip Overhead
       +87 bytes Non-compiled Android resources size
   -12,400 bytes Native code size
        -4 bytes Package metadata size
    -5,806 bytes Java code size
    +1,344 bytes Compiled Android resources size
MonochromePublic.apk_Specifics
   -34,572 bytes normalized apk size
   -12,400 bytes main lib size
   -15,408 bytes main dex size

Bug: 785118, 764446
Change-Id: I645134b61ef818a547a4eae239c067758d06fcf4
Reviewed-on: https://chromium-review.googlesource.com/943942
Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
Reviewed-by: agrieve <agrieve@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540320}

Bug: None
Change-Id: I12ed377d2cbab940a9f45a5d71777144166f69d8
Reviewed-on: https://chromium-review.googlesource.com/943931Reviewed-by: Meggin Kearney <mkearney@chromium.org>
Commit-Queue: Crystal Lambert <crystallambert@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540319}

Roll in the most recent version of:
testharness.js, testdriver.js, idlharness.js and webidl2.js

Some baselines are adjusted accordingly (mostly due to
https://github.com/w3c/web-platform-tests/pull/9051).

Bug: 685854
Change-Id: Ic74431ad0bbbc13e0c9ab2fa8bb4701f69806d32
Reviewed-on: https://chromium-review.googlesource.com/941915Reviewed-by: Philip Jägenstedt <foolip@chromium.org>
Commit-Queue: Robert Ma <robertma@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540318}

histogram. Also rename the enums to be readable, as per the histogram
team's advice.

BUG=815241

Change-Id: Ic4a15efed46f367b2500ef5e553645323355020a
Reviewed-on: https://chromium-review.googlesource.com/944262
Commit-Queue: Ryan Hamilton <rch@chromium.org>
Reviewed-by: Zhongyi Shi <zhongyi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540317}

All these tests pass locally and on the bots so removing.

Bug: 789184
Change-Id: I5b04ccdee8c39837d0f31ead05f206b00069e59c
Reviewed-on: https://chromium-review.googlesource.com/944010Reviewed-by: Philip Rogers <pdr@chromium.org>
Commit-Queue: David Bokan <bokan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540316}

In HandleScheduleOverlayPlaneCHROMIUM assign values to local variables
from the correct command parameters.

Bug: None
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_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
Change-Id: Idcba758ac7088af9ba141f8e9eec05023237a4d7
Reviewed-on: https://chromium-review.googlesource.com/928561Reviewed-by: Antoine Labour <piman@chromium.org>
Commit-Queue: Antoine Labour <piman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540315}

Adds the compile builder, and the tester.

Bug: 790710, 790644
Change-Id: I83de37258aadc8d8e79cde3c32baca9d7d994c98
Reviewed-on: https://chromium-review.googlesource.com/940431Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Ryan Tseng <hinoka@chromium.org>
Commit-Queue: Stephen Martinis <martiniss@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540314}

Bug: 817656
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_vr;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: I5faf83e3b1eae38ed977716a1329bfddb3fb271c
Reviewed-on: https://chromium-review.googlesource.com/943925
Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: Christopher Grant <cjgrant@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540313}

Now that base::MakeUnique is no more, we don't need this anymore.

Bug: 755727
Test: git cl upload
Change-Id: Ia90edbd28bbaca79afa5aa554677855e3b98cdfd
Reviewed-on: https://chromium-review.googlesource.com/944511Reviewed-by: Yusuke Sato <yusukes@chromium.org>
Commit-Queue: Luis Hector Chavez <lhchavez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540312}

Bug: 749879
Change-Id: Iff4a6e047b029bb12fa225bbf2cdeffc9172a1b7
Reviewed-on: https://chromium-review.googlesource.com/941966Reviewed-by: Philip Jägenstedt <foolip@chromium.org>
Commit-Queue: Robert Ma <robertma@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540311}

Bug: 812428
Change-Id: I8561fce9eb8a2da3cf45530ccf1494d6182f48d5
Reviewed-on: https://chromium-review.googlesource.com/942226
Commit-Queue: Robbie Iannucci <iannucci@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540310}

https://github.com/w3c/web-platform-tests/tree/master/accname

All the tests in this directory are manual. And worse, they can run to
complete but produce flaky outputs (and hence block the importer).

Skip them as we don't need these at the moment.

No-Try: True
Change-Id: I7719de411dadffe5077b405f1d0b3905105d6639
Reviewed-on: https://chromium-review.googlesource.com/944506Reviewed-by: Philip Jägenstedt <foolip@chromium.org>
Commit-Queue: Philip Jägenstedt <foolip@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540309}

This cl contains the following changes:
- Add video detector observers to mojo VideoDetector in order to be notified
about the video start/end activities.
- Fix a bug to reset suspend reason upon SuspendDone.

Bug: 784232
Change-Id: If626b6971413c3bc78687e3391e671766aa0fbde
Reviewed-on: https://chromium-review.googlesource.com/942649Reviewed-by: Dan Erat <derat@chromium.org>
Commit-Queue: Jia Meng <jiameng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540308}

The "glGetQueryObject(GL_QUERY_RESULT_AVAILABLE) blocks unexpectedly on
Adreno" issue is fixed in Android 8.1, un-blacklist it on that version.

According to Qualcomm, the issue affected Adreno 4xx and newer, and
penghuang@ also saw it on Adreno 3xx on Android K (4.x). The extension
is separately blacklisted by entry id=112 for Adreno 3xx, but only on
Android 5.0 and up, so it seems cleanest to apply the blacklist to
all of Adreno 3xx/4xx/5xx for the blocking issue.

For consistency, also disable GL_EXT_disjoint_timer_query_webgl2 for the
pre-existing Adreno blacklist entries. If the underlying calls are
broken, this is expected to affect both variants.

BUG=477514,750306,764526

Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_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
Change-Id: Iace887cb6e168d691329ff5aaf29634e509ab5c0
Reviewed-on: https://chromium-review.googlesource.com/941549Reviewed-by: Antoine Labour <piman@chromium.org>
Commit-Queue: Klaus Weidner <klausw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540307}

Gtest will use operator<< which is available for all build
targets for most classes that had PrintTo(). The extra binary code
for operator<< is small, and will be removed from binaries if not
used.

Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I588f303293fb2c4d0f3e72093fed27c3d3aea75b
Reviewed-on: https://chromium-review.googlesource.com/941964
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Reviewed-by: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540306}

Reference to a WeakPtr was to a WeakPtr that was deleted in
certain cases. Fixed by just making a copy.

Bug: 817866
Change-Id: I9fae68d018c4178a8224b3e79ab44c8a1282be87
Reviewed-on: https://chromium-review.googlesource.com/943983Reviewed-by: Bence Béky <bnc@chromium.org>
Commit-Queue: Paul Jensen <pauljensen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540305}

The previous API, OnReceivedSyncFavicon(), received a parameter
|icon_bytes| that was actually only used in tests. This prevented the
visit time from being updated (in-memory).

While fixing this, we move some code (the loop that iterates all
navigations of a sync_pb::SessionTab) to favicon_cache.cc, to make
SessionsSyncManager smaller and couple FaviconCache's API more with
how it is actually used (single caller).

Bug: 817348
Change-Id: I8fb7457177e298759f40e0bcf8d576ffc9fb7f9a
Reviewed-on: https://chromium-review.googlesource.com/940136Reviewed-by: Marc Treib <treib@chromium.org>
Reviewed-by: Nicolas Zea (slow) <zea@chromium.org>
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540304}

Test video:
https://drive.google.com/file/d/1x3lFo4c7QE912uDUTGzal3Fd8s6UDL-H/view?usp=sharing

Will persist payment app's icon in the following up CL.

Bug: 782270
Change-Id: Ib1ebebcac224033b4272c4e5cfaf3c3997b730f0
Reviewed-on: https://chromium-review.googlesource.com/941988Reviewed-by: Nico Weber <thakis@chromium.org>
Reviewed-by: Rouslan Solomakhin <rouslan@chromium.org>
Commit-Queue: Ganggui Tang <gogerald@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540303}