GitLab

Those tests are consistently timing out on iPad, disabling them
on iPad until the root cause is fixed to turn the bots green.

Bug: 745798
Change-Id: I5f0dab981cae4fbef9f81d144ffa0492a16cfa12
Reviewed-on: https://chromium-review.googlesource.com/576171Reviewed-by: Jérôme Lebel <jlebel@chromium.org>
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487506}

This change addresses a couple of issues:
- Simplifies the color correction code path in ImageBitmap
by using the same general code path for all the constructors
(except ImageData which needs a separate code path).
- Adds resize option to the constructor from ImageData.
- Updates ImageBitmap resize layout tests to enlarge the
source (instead of shrinking) such that the correct application
of filter quality can be checked.

Bug: 627855,665919,734587,734588
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: I73d021a8e07c5a2387111c8daec0122cddb85ebb
Reviewed-on: https://chromium-review.googlesource.com/542959Reviewed-by: Justin Novosad <junov@chromium.org>
Commit-Queue: Mohammad Reza Zakerinasab <zakerinasab@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487505}

Bug: 745776
Change-Id: I95868b7bead32c96c5fb4ccf084d887e8f28eb19
Reviewed-on: https://chromium-review.googlesource.com/576211Reviewed-by: Florin Malita <fmalita@chromium.org>
Commit-Queue: Brian Salomon <bsalomon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487504}

There was a server-side proto change on June 30.

Bug: 725124
Change-Id: I90806333e3571e426126110a899781d2f4c58c8d
Reviewed-on: https://chromium-review.googlesource.com/574601
Commit-Queue: Thiemo Nagel <tnagel@chromium.org>
Reviewed-by: Roman Sorokin <rsorokin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487503}

With this patch set, we permit setting transitions on UiElement
instances in the same way one might set CSS transitions on DOM elements.
This change also includes some syntactic sugar to make this easier to
accomplish (namely UiElement::SetTranslate, UiElement::SetRotate, and
UiElement::SetScale).

Bug: None
Change-Id: Ib52e83e638a580f3048ea6614d0c393de09952ba
Reviewed-on: https://chromium-review.googlesource.com/575813
Commit-Queue: Ian Vollick <vollick@chromium.org>
Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487502}

With jumbo (unity builds, merged translation units) a "using blink"
statement intended for just the local translation unit will affect
many other translation units which causes various issues.

There is also (with jumbo) a warning about such usage that will prevent
things from compiling.

Without this patch this file will have to be manually excluded from
jumbo builds and I am trying to avoid such exclusion lists for
performance and maintenance.

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

Modify the view hierarchy to allow BottomSheetContent to draw beneath
the toolbar. Exposes a new BottomSheetContent#applyDefaultTopPadding()
interface method. Classes that do not need to draw beneath the toolbar
return true and BottomSheet handles setting padding correctly so that
contents will be shown below the toolbar holder.

Also adds BottomSheetNewTabController#Observer interface ands registers
SuggestionsBottomSheetContent as an observer.

This is scaffolding necessary for BottomSheetNewTabController to display
a doodle on the NTP UI.

BUG=735556

Change-Id: Ia24f13dc222bf2a0cf3804aac4db59e40c91bca4
Reviewed-on: https://chromium-review.googlesource.com/572160Reviewed-by: Matthew Jones <mdjones@chromium.org>
Reviewed-by: Min Qin <qinmin@chromium.org>
Reviewed-by: Michael van Ouwerkerk <mvanouwerkerk@chromium.org>
Commit-Queue: Theresa <twellington@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487500}

BUG=

Change-Id: I0068ef6a064ead8886fdda68cfcd71ca5af0e806
Reviewed-on: https://chromium-review.googlesource.com/576140Reviewed-by: Min Qin <qinmin@chromium.org>
Commit-Queue: Theresa <twellington@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487499}

r485361 has made AreCrossProcessFramesPossible return true on all
platforms, including Android.  This CL makes an equivalent change for
layout tests - after this CL all platforms will return |true| from
LayoutTestContentClient::IsSupplementarySiteIsolationModeEnabled.  This
will mean that URLs matching the *oopif.test pattern will be caught by
LayoutTestContentBrowserClient::DoesSiteRequireDedicatedProcess and
pushed into a separate renderer process.

Bug: 745064
Change-Id: I1311c9e89b1fe45d30575ef923e3b78a097a9dbf
Reviewed-on: https://chromium-review.googlesource.com/574970Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
Commit-Queue: Lukasz Anforowicz <lukasza@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487498}

The animation logic in the BrowserControlsOffsetManager did not
previously account for bottom controls when triggering an early exit.
This change only skips animation if both top and bottom browser
controls are 0.

BUG=713911

Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I9b4a24abfcb3cdd9184370d7bc401dbdec37daa9
Reviewed-on: https://chromium-review.googlesource.com/557953Reviewed-by: Alexandre Elias <aelias@chromium.org>
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Commit-Queue: Matthew Jones <mdjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487497}

It's banned, and the code is simpler with C++11 lambda anyhow.

Change-Id: I68e4edad5c529b457fe59292278b8c97423b6336
Reviewed-on: https://chromium-review.googlesource.com/574428
Commit-Queue: Rouslan Solomakhin <rouslan@chromium.org>
Reviewed-by: Rouslan Solomakhin <rouslan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487496}

GpuRootCompositorFrameSink creates an instance of HitTestAggregator
and provides the notifications required when surfaces are added to the
next CompositorFrame, destroyed and when the next frame begins.

BUG=732399

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

BUG=None

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

DCHECK always references the condition, even in Release. So no need for
DCHECK_IS_ON.

Bug: 661794
Change-Id: I255be0acbadc38b29f9b39e8694de331ca0b1c22
Reviewed-on: https://chromium-review.googlesource.com/575056
Commit-Queue: Jérôme Lebel <jlebel@chromium.org>
Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487493}

Change-Id: Idb78bd2d8fdc6a194f1bda9db233f56fb21090eb
Reviewed-on: https://chromium-review.googlesource.com/573527
Commit-Queue: Quinten Yearsley <qyearsley@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487492}

This is in preparation for removing title bars work. See mocks in the bug and screenshots for more details.
This approach of adding a back button expands the current app list button to be modal and have two functionalities
depending on which side is pressed. The alternative (seperate back button) is IMO very difficult to achieve what
is shown in the mocks (namely the color in between and animation (not in this CL) of said color in between). The
exact functionality of the back button is not 100% defined yet, this CL just sends a VKEY_BROWSER_BACK event in the
meantime.

Screenshots:
https://screenshot.googleplex.com/ZVNP0aq0cc3
https://screenshot.googleplex.com/fDcGTuoyZg5
https://screenshot.googleplex.com/ZFBzk72LLVt

Test: ash_unittests --gtest_filter="ShelfViewInkDropTest.*"
Bug: 721940
Change-Id: I0b2be5cc66583d521721ec1378a19e0026c5c7e1
Reviewed-on: https://chromium-review.googlesource.com/572434
Commit-Queue: Sammie Quon <sammiequon@chromium.org>
Reviewed-by: Michael Wasserman <msw@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487491}

resource.html runs 4 test cases, representing [in]secure resource load
on an [in]secure page.

The expected result is that Clear-Site-Data is honored iff the resource
is secure. The embedding page does not matter.

Bug: 607897
Change-Id: Id17bc6d52bca4da46fab214bcc71ca7c7070cdb0
Reviewed-on: https://chromium-review.googlesource.com/571458Reviewed-by: Mike West <mkwst@chromium.org>
Commit-Queue: Martin Šrámek <msramek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487490}

This CL constructs the DOM-to-TextContent offset mapping of a block. It adds
the following API:

NGOffsetMappingResult NGInlineNode::BuildOffsetMapping() const;

See ng_offset_mapping_result.h for information about NGOffsetMappingResult.
See ng_inline_node_offset_mapping_test.cc for examples.

Note 1: The patch can be made cleaner if we do not write whitespace-collapsed
strings back to LayoutText after layout.

Note 2: The built offset mapping is wrong when text-transform expands/removes
characters (e.g., ß => SS)

BUG=699017
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng

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

BUG=668223

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel

Change-Id: Icaa5c0f7cf3130770d89b34eb46afbfd8379bee9
Reviewed-on: https://chromium-review.googlesource.com/574688Reviewed-by: Kenneth Russell <kbr@chromium.org>
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Commit-Queue: Geoff Lang <geofflang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487488}

After crrev.com/c/565510 the PID is received asynchronously
after the start of a service. It is possible to get a dump
request between the point when a client process is started
and the point when the PID is received. In this case just
temporarily skip that client's dumps.

Bug: 744722
Change-Id: I69a70747408232063a1d57933d8d0ff985e153c9
Reviewed-on: https://chromium-review.googlesource.com/575127
Commit-Queue: Hector Dearman <hjd@chromium.org>
Reviewed-by: Primiano Tucci <primiano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487487}

This CL adds a build flag (INCLUDE_TRANSPORT_SECURITY_STATE_PRELOAD_LIST)
for //net embedders to configure whether to include the preload list
(~258KiB) in their binary.

BUG=603597

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

MediaControlsImplTest and MediaControlsOrientationLockDelegate use the
same class names for Mock classes. Such as MockChromeClient and
StubLocalFrameClient. That is normally no big deal but in jumbo builds
they can be compiled in the same translation unit and then the classes
will collide. This patch gives the class names unique suffixes so that
media_controls tests don't have to be excluded from jumbo.

Bug: 745732

Change-Id: I6c3a4d5d92ae690ece53aa093ffb4039eb79137e
Reviewed-on: https://chromium-review.googlesource.com/575140
Commit-Queue: Daniel Bratell <bratell@opera.com>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487485}

ContentVideoView uses both an alpha channel and a transparent
background at the same time.  AndroidOverlay will need only the
former.  Currently, they're set together in CompositorImpl.

This CL separates them with SetRequiresAlphaChannel, and updates
CVV callers to call both.

Bug: 744706
Change-Id: I7c9e5d258e68fa35c7a5f8a611d942dc9c460389
Reviewed-on: https://chromium-review.googlesource.com/574640Reviewed-by: Ted Choc <tedchoc@chromium.org>
Reviewed-by: Bo Liu <boliu@chromium.org>
Commit-Queue: Frank Liberato <liberato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487484}

The observer references in the UI config caused memory leaks after the bottom
sheet was closed and (to a lesser extent) when cards were scrolled out of view.

Also, drop thumbnail bitmaps from cards when they get recycled. This also avoids
(non-discardable) references to bitmaps staying around after a comes stops
being visible.

Bug: 718925
Change-Id: I6e59dd6d68b559eeab63dc6e172c42eddf427e33
Reviewed-on: https://chromium-review.googlesource.com/574373
Commit-Queue: Bernhard Bauer <bauerb@chromium.org>
Reviewed-by: Theresa <twellington@chromium.org>
Reviewed-by: Nicolas Dossou-Gbété <dgn@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487483}

Bug: 710535
Change-Id: I6ce8350af1f1bad0ef1577b67b927381f1c4842f
Reviewed-on: https://chromium-review.googlesource.com/575031Reviewed-by: Kent Tamura <tkent@chromium.org>
Commit-Queue: Quinten Yearsley <qyearsley@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487482}

This fixes hangs in https://codereview.chromium.org/2890853002/.

New regression test hangs without matching test_browser_thread.cc 
changes.

We support adding a TestBrowserThreadBundle on top of an explicit
ScopedTaskEnvironment (i.e. to add named threads in unit tests that
already otherwise inherit a ScopedTaskEnvironment or need to provide a
custom one). However if the ScopedTaskEnvironment uses a QUEUED mode,
trying to flush it in ~TestBrowserThreadBundle() results in a hang.
Flushing TaskScheduler when it's not managed by that
TestBrowserThreadBundle was wrong anyways.

Bug: 708584, 689520
Change-Id: I3087537ad4f1065fc974170b00c25a676ff5e4c9
Reviewed-on: https://chromium-review.googlesource.com/575593Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Gabriel Charette <gab@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487481}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/b56b94be..1c7f2213

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;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;master.tryserver.chromium.android:android_optional_gpu_tests_rel

TBR=hablich@chromium.org,machenbach@chromium.org,kozyatinskiy@chromium.org

Change-Id: If56fcb26c5ecde062ad923553f459c55a6946f8a
Reviewed-on: https://chromium-review.googlesource.com/575874Reviewed-by: v8 autoroll <v8-autoroll@chromium.org>
Commit-Queue: v8 autoroll <v8-autoroll@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487480}

Both PresentationReceiverTest and RemotePlaybackTest use a local test
class named MockEventListener. That is normally no big deal
but in jumbo builds they can be compiled in the same translation unit
and then the classes will collide. This patch gives the classes
unique prefixes so that no tests have to be excluded from jumbo.

Bug: 745732

Change-Id: Ic9a5e4e896b226d5ef7f99f2549cd3d4bb66fc42
Reviewed-on: https://chromium-review.googlesource.com/575141
Commit-Queue: Daniel Bratell <bratell@opera.com>
Reviewed-by: Mounir Lamouri <mlamouri@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487479}

This CL adds Zucchini utilities related to patch manipulation, along
with tests. Details:
- PatchHeader struct
- ElementPatchHeader struct
- Encode/Decode varint functions

Bug: 729154
Change-Id: I081752542cb8b6ffe35c7dde079df70c19ca71e5
Reviewed-on: https://chromium-review.googlesource.com/572306
Commit-Queue: Samuel Huang <huangs@chromium.org>
Reviewed-by: Erik Chen <erikchen@chromium.org>
Reviewed-by: Samuel Huang <huangs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487478}

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/6fc5e4196940..26f2e66be57e

$ git log 6fc5e4196..26f2e66be --date=short --no-merges --format='%ad %ae %s'
2017-07-18 perezju [devil] Replace logging with custom logger object

Created with:
  roll-dep src/third_party/catapult


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

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls


CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel
TBR=sullivan@chromium.org

Change-Id: I52a93ca2f43d12c86bdd9ed4022fb755cdf8517f
Reviewed-on: https://chromium-review.googlesource.com/576052
Reviewed-by: <catapult-deps-roller@chromium.org>
Commit-Queue: <catapult-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487477}

Add an internal WebUI interface at chrome://media-engagement for
viewing scores stored in the Media Engagement service.

BUG=734525

Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I9300479df06f48b98049026c9016084bd0abb902
Reviewed-on: https://chromium-review.googlesource.com/549297
Commit-Queue: Becca Hughes <beccahughes@chromium.org>
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Reviewed-by: Chrome Cunningham <chcunningham@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Mounir Lamouri <mlamouri@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487476}

Both AudioContextTest and BaseAudioContextTest use a local test
class named MockWebAudioDevice. That is normally no big deal
but in jumbo builds they can be compiled in the same translation unit
and then the classes will collide. This patch gives the classes
unique prefixes so that no tests have to be excluded from jumbo.

Bug: 745732

Change-Id: Ie736a0541082c56af745d51061897a081d2d1da5
Reviewed-on: https://chromium-review.googlesource.com/575142
Commit-Queue: Daniel Bratell <bratell@opera.com>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487475}

Updated location of some dependencies; moved expected failures due to 
flag requirements to "NeverFixTests"

Bug: 
Change-Id: I88d25a20b811ab8c8e9961aca3b77b79ec0cbcc1
Reviewed-on: https://chromium-review.googlesource.com/573141
Commit-Queue: Mircea Trofin <mtrofin@chromium.org>
Reviewed-by: Brad Nelson <bradnelson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487474}

OpenSplitModeExtensionOptionsPageIncognito flaked six times today, in
builds 61651-61654, 61658 and 61659, e.g.:

https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/61659

TBR=catmullings@chromium.org
BUG=745729

Change-Id: I15a827faea250f5bbda74c791b2141f00df3d594
Reviewed-on: https://chromium-review.googlesource.com/575051
Commit-Queue: Peter Beverloo <peter@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Reviewed-by: Roger Tawa <rogerta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487473}

Some of the BrowserTest suite was flaky, and disabled. The various sources of
the flakes have been since addressed. I'm turning them back on the FYI bot to
verify.

TBR=sky@chromium.org
TEST=BrowserTest in mash_browser_tests

Bug: 707195
Change-Id: I36f3bf2c98135842a68217b6bf2982065f9dec3b
Reviewed-on: https://chromium-review.googlesource.com/574613
Commit-Queue: Jonathan Ross <jonross@chromium.org>
Reviewed-by: Jonathan Ross <jonross@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487472}

https://pdfium.googlesource.com/pdfium.git/+log/ffbc0d9a08f8..61a3cb81a298

$ git log ffbc0d9a0..61a3cb81a --date=short --no-merges --format='%ad %ae %s'
2017-07-18 dsinclair Roll third_party/freetype/src cf8d9b4ce..38bdf22bf (9 commits)

Created with:
  roll-dep src/third_party/pdfium


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

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls


TBR=dsinclair@chromium.org

Change-Id: I83441d830d700c984e5a77966740ec84c950be20
Reviewed-on: https://chromium-review.googlesource.com/576051
Reviewed-by: <pdfium-deps-roller@chromium.org>
Commit-Queue: <pdfium-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487471}

This is preparation to remove MockCryptohomeClient,
as preparation to replace Callback by OnceCallback in chromeos/dbus.

BUG=739622
TEST=Ran trybot.

Change-Id: I77b9dfed9ce8662530e40e2dda085eb2852ae171
Reviewed-on: https://chromium-review.googlesource.com/569902
Commit-Queue: Hidehiko Abe <hidehiko@chromium.org>
Reviewed-by: Bartosz Fabianowski <bartfab@chromium.org>
Reviewed-by: Ryo Hashimoto <hashimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487470}

Change-Id: Ifccf02402e0513c9e2c96dad335d05ab23b705ef
Reviewed-on: https://chromium-review.googlesource.com/574919Reviewed-by: Anita Woodruff <awdf@chromium.org>
Commit-Queue: Christian Dullweber <dullweber@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487469}

Bug: 683256
Change-Id: I7dc24bcae81a2ca43f3f82427d5b654b5ebe5b1f
Reviewed-on: https://chromium-review.googlesource.com/575019Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Commit-Queue: Yipeng Wang <yipengw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487468}

This change renames |GetAffiliations| to |GetAffiliationsAndBranding|
and also changes the name of related methods.

Bug: 628988
Change-Id: Id3e38a209d63a8e07903eccf4005f70186b34a09
Reviewed-on: https://chromium-review.googlesource.com/574370
Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org>
Reviewed-by: Balazs Engedy <engedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487467}