GitLab

Bug: 746640
Change-Id: Ie312a7756f5e6917a4d7c2bd107951f4928e97d0
Reviewed-on: https://chromium-review.googlesource.com/1024054Reviewed-by: Jesse Doherty <jwd@chromium.org>
Commit-Queue: Tarun Bansal <tbansal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552765}

The renderer now checks for empty as well as null strings.
Crashing if an empty string is presented to the browser-side is
technically correct, but it is now  caught as a SecurityError in the
renderer first.

Bug: 830855
Change-Id: Id904864b8f6d57e9e0eb342d287337ba96d368d4
Reviewed-on: https://chromium-review.googlesource.com/1020187
Commit-Queue: Kim Paulhamus <kpaulhamus@chromium.org>
Reviewed-by: Balazs Engedy <engedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552764}

Invalid Mojo shared memory handles can be "successfully" unwrapped to
invalid base shared memory handles. This may be a little weird, but
that API is going away in favor of unwrapping new base shared memory
types anyway.

So rather than changing the API behavior, this fixes the check in
ChildHistogramFetcherImpl so it doesn't attempt to use an invalid
base::SharedMemoryHandle.

Bug: 835837
Change-Id: If6f4fd5086547e671e4d86363ede9513adb2c054
Reviewed-on: https://chromium-review.googlesource.com/1024114Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Ken Rockot <rockot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552763}

... plus all the necessary code migration. A follow-up CL will be used
to update relevant comments.

Bug: 734095
Change-Id: I8516c750518e10a366bdc1496e75a3c2f4f0898b
Reviewed-on: https://chromium-review.googlesource.com/1022957Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Xi Cheng <chengx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552762}

Looks like mac has a different expected file, this CL changes it.

TBR=smcgruer@chromium.org
NOTRY=true

Bug: None
Change-Id: I0d669c8dd8f8b834c9197a38e7a4e82576dc73a8
Reviewed-on: https://chromium-review.googlesource.com/1024448Reviewed-by: Xida Chen <xidachen@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552761}

A cool off period of 600ms is added between the time page is scrolled to
the top and the time a pull-to-refresh can be initiated. If the user
tries overscroll before the cool off period is passed, the timer will
reset and the user has to wait 600ms more.

BUG=702404
TEST=OverscrollControllerTest.* in content_unittests

Change-Id: I151ec8ef2faab297e9a80251942040acd40af399
Reviewed-on: https://chromium-review.googlesource.com/1022512Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org>
Commit-Queue: Mohsen Izadi <mohsen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552760}

Replace SkColorSetARGBMacro with SkColorSetARGB.

Replace SkColorSetARGBInline with SkColorSetARGB.

SkColorSetARGBMacro and SkColorSetARGBInline
are macros which will be deleted.

R=sky@chromium.org,fmalita@chromium.org

Bug: 834984
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I08eb888f0848b9ebad93d2dc782c7a7f73bf7987
Reviewed-on: https://chromium-review.googlesource.com/1020043Reviewed-by: Florin Malita <fmalita@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Cary Clark <caryclark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552759}

These changes were scripted as part of the effort to restrict exposure of the
raw MessageLoop*.

Note that many static methods exposed on MessageLoopCurrent are also deprecated
so it is very well possible that this is migrating to an already deprecated
method.

The goal of this pass is to reduce usage of MessageLoop::current() (and
ultimately make it impossible to obtain the MessageLoop* statically).

As such I will not edit this CL unless the script did something logically wrong.
I defer to owners to fix highlighted usage of already deprecated APIs.

Possible script screw ups / things to look out for in this review:
 - Storing MessageLoopCurrent in a MessageLoop* variable or comparing it against
   one (I will go over that in a separate pass).

Includes should have been stripped if that was the last usage of message_loop.h
in that file.

Please CQ if LGTY

This CL was uploaded by git cl split.

R=scottmg@chromium.org

Bug: 825327
Change-Id: I3fe0959f4d7e62f3a05a0748782a44ae2306d496
Reviewed-on: https://chromium-review.googlesource.com/1024439
Commit-Queue: Gabriel Charette <gab@chromium.org>
Commit-Queue: Scott Graham <scottmg@chromium.org>
Reviewed-by: Scott Graham <scottmg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552758}

Update of feed library and build file.

Monochrome.apk_Dex
        +5 entries methods
      +225 bytes Java code size
    +1,080 bytes Compiled Android resources size
      +601 bytes Transfer size (deflate)
    +1,491 bytes normalized apk size
      +396 bytes main dex size
    +1,095 bytes APK size
 +2,694.84 bytes Estimated installed size
      +396 bytes Java code size
       +36 bytes DexCache
       +15 bytes Package metadata size
 +1,824.84 bytes Java code size
    +1,080 bytes Compiled Android resources size

Change-Id: I932caf40786751fa3ca0264534c1cbfb29e26434
Reviewed-on: https://chromium-review.googlesource.com/1023376Reviewed-by: Filip Gorski <fgorski@chromium.org>
Commit-Queue: Gang Wu <gangwu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552757}

Bug: 799482
Change-Id: I2479eb45e4764cc27ce3d56d86e54a8522b223bd
Reviewed-on: https://chromium-review.googlesource.com/1020225Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Erik Chen <erikchen@chromium.org>
Commit-Queue: Erik Chen <erikchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552756}

This is a deplumbing CL where we remove the unnecessary passing of
nullptr around. We don't use client side share groups now that we use
mailboxes.

Removing this preps for the eventual switch over to RasterDecoder
backend for RasterInterface because RasterImplementation is always in
it's own client side share group.

TBR=boliu@chromium.org,kenrb@chromium.org,haraken@chromium.org,rbyers@chromium.org,sky@chromium.org

Bug: 789238
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;master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I86bd2e2d9c42064da3d2d3bd1a7881d5392a1b38
Reviewed-on: https://chromium-review.googlesource.com/1015701
Commit-Queue: Jonathan Backer <backer@chromium.org>
Reviewed-by: danakj <danakj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552755}

Inserts the top History message using a TableViewTextItem. The UI will be updated on a followup
CL.

Bug: 805190
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I56ef6eece234eb1be50df0949f1400532a961ac6
Reviewed-on: https://chromium-review.googlesource.com/1018014
Commit-Queue: Sergio Collazos <sczs@chromium.org>
Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552754}

1) Prefer to keep the implementation of customized default wallpapers
   in //chrome.

2) There's no need to put the constants of small/large file suffixes
   to //ash/public, since they are implementation details, and //ash
   only needs to know the entire file path.

3) The original idea was to put
   |ash::WallpaperController::ResizeAndSaveWallpaper| to //ash/public
   but customized default wallpaper only needs a simplified version of
   it, so it's inlined.

Bug: 827062
Change-Id: I4c94adff973af697ea5f929ba751eb2fe8f019de
Reviewed-on: https://chromium-review.googlesource.com/1015744Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Wenzhao (Colin) Zang <wzang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552753}

Bug: 835572, 835573
Change-Id: I76a2d7407d376a92c0dea7649bc877349d862f2f
Reviewed-on: https://chromium-review.googlesource.com/1024233Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552752}

This CL is adding DebugID for modules present in chrome.
The intent of that CL is to help trace symbolisation by providing the
appropriate debug id.

R=erikchen@chromium.org

Change-Id: I6db22b1020dc3c00e20da781e63be8755f1fb913
Reviewed-on: https://chromium-review.googlesource.com/1014710
Commit-Queue: Etienne Bergeron <etienneb@chromium.org>
Reviewed-by: Primiano Tucci <primiano@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Erik Chen <erikchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552751}

This API matches ContentRendererClient::PrepareErrorPage from //content
and will be used to load errors into WKWebView as HTML.

Bug: 725241
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I656f2c892430de2840196a224a446111b44f4d9c
Reviewed-on: https://chromium-review.googlesource.com/1011383
Commit-Queue: Eugene But <eugenebut@chromium.org>
Reviewed-by: Danyao Wang <danyao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552750}

Change-Id: I7ac0ec87a2f27aba3cf2b93f8cf890d179369055
Reviewed-on: https://chromium-review.googlesource.com/1023911Reviewed-by: Egor Pasko <pasko@chromium.org>
Commit-Queue: agrieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552749}

Use the real FetchHelper, with the request delay removed, for tests.
This is a better end-to-end integration test as fewer things are
stubbed out.

BUG=827348

Change-Id: I34e588d032b0a48871b71a531328f87f705d3716
Reviewed-on: https://chromium-review.googlesource.com/1024055Reviewed-by: Filip Gorski <fgorski@chromium.org>
Commit-Queue: Theresa <twellington@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552748}

R=cmasso@chromium.org

Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation;master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Ibf6a5170c220727f537e78722a3555f2e3351f9f
Reviewed-on: https://chromium-review.googlesource.com/1024246Reviewed-by: Krishna Govind <govind@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552747}

The autofill code is calling IsFocused while pending stylesheets are
loading. That doesn't mean we should decide the focusability of the
form element at that point in time ignoring pending stylesheets as the
pending sheets are blocking rendering.

This fixes the incorrect triggering of transitions in 831590.

Bug: 831590
Change-Id: Ie041014bdcd4ba74f774919718e9fc3fc4540c6b
Reviewed-on: https://chromium-review.googlesource.com/1019401Reviewed-by: Mathieu Perreault <mathp@chromium.org>
Commit-Queue: Rune Lillesveen <futhark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552746}

When Mus is enabled, discardable memory manager server runs in ui
service. This CL adds support to access discardable memory manager server
in ui service from pdf compositor service.

BUG=834889

Change-Id: Ic667b04d3b9f588ace4accb9e6ecf2ad91a6819d
Reviewed-on: https://chromium-review.googlesource.com/1022810Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Wei Li <weili@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552745}

changes:
- Plumb note document text body, android side change: ag/3916628.

Bug: 818902
Test: device test and unit_tests ArcAppDataSearchProviderTest.*
Change-Id: I82b8bce0ac3117934a57018a299959698649b72d
Reviewed-on: https://chromium-review.googlesource.com/1016062Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Luis Hector Chavez <lhchavez@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Qiang Xu <warx@google.com>
Cr-Commit-Position: refs/heads/master@{#552744}

This CL removes usage of the last remaining system fonts.  With this, only GCS
fonts are used.

The two fonts removed are Comic Sans and Impact, which used to be the default
font families for cursive and fantasy, respectively.  Since there's no suitable
replacement for these fonts with an open license, fallback on the default font
(DejaVu Sans).

R=eae
TBR=dpranke
BUG=787020

Change-Id: Iee177b18e027370ba8181cb70edcfb3b3fdfcae3
Reviewed-on: https://chromium-review.googlesource.com/1010412Reviewed-by: Thomas Anderson <thomasanderson@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Commit-Queue: Thomas Anderson <thomasanderson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552743}

Change VirtualFidoDevice class to be abstract interface and create
VirtualU2fDevice class which implements the interface. This CL prepares
for implementation of VirtualCtap2Device.

Bug: 829413
Change-Id: I2cc618492c1626f8dfb50a821e4ed62533ad0975
Reviewed-on: https://chromium-review.googlesource.com/1009103
Commit-Queue: Jun Choi <hongjunchoi@chromium.org>
Reviewed-by: Jan Wilken Dörrie <jdoerrie@chromium.org>
Reviewed-by: Balazs Engedy <engedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552742}

We used to assume VP8 codec is always supported, but some newer devices
don't support it, so skip those tests instead of failing.

Bug: b/77279478
Test: CQ + unit tests on the device
Change-Id: Iccb4270333c7e6231fc6e8aaed86b65d14a480e2
Reviewed-on: https://chromium-review.googlesource.com/1023037
Commit-Queue: Sergey Volk <servolk@chromium.org>
Reviewed-by: Kenneth MacKay <kmackay@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552741}

Adds a voice search button to the steady location bar.

Bug: 825396
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I5f2a570ab09fb513586a3eb88f5fdcf9373d5c9b
Reviewed-on: https://chromium-review.googlesource.com/1021698Reviewed-by: Kurt Horimoto <kkhorimoto@chromium.org>
Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552740}

Change-Id: I3c649d738b551db7d9dc6e097356c91235087b7c
Reviewed-on: https://chromium-review.googlesource.com/1023717Reviewed-by: Ben Henry <benhenry@chromium.org>
Commit-Queue: Erik Chen <erikchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552739}

This method for performing periodic profiling has been superseded by the
one in ThreadProfiler.

Bug: 808588
Change-Id: I5554f2c2b46eebb4b08a490e36bdc64110013cc1
Reviewed-on: https://chromium-review.googlesource.com/1022721Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Commit-Queue: Mike Wittman <wittman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552738}

By setting this flag, all entry points for password-related tasks
become available (mainly the bottom sheet, tabs and generation).

As of now, generation requires enable-password-generation and
enable-manual-password-generation in addition.

Bug: 835234, 828832, 811747
Change-Id: Ie40f9368d1ac4940663e40bb8e061f85faef865c
Reviewed-on: https://chromium-review.googlesource.com/1021710
Commit-Queue: Friedrich Horschig <fhorschig@chromium.org>
Reviewed-by: Vadym Doroshenko <dvadym@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552737}

Change-Id: Iba992e41c496202cc8bc5c26fe271b6ea8fd3e44
Reviewed-on: https://chromium-review.googlesource.com/1022976Reviewed-by: Greg Thompson <grt@chromium.org>
Commit-Queue: Xi Cheng <chengx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552736}

Update the documentation regarding Certificate Transparency
to provide guidance for site operators and enterprises about
what CT is, as well as what options exist.

Further update the documentation regarding CT for Chromium
and //net embedders regarding best practices.

BUG=702062

Change-Id: Ide4fbdc3958f4994cacd4355021b8808a951ab4c
Reviewed-on: https://chromium-review.googlesource.com/1017940Reviewed-by: Ryan Sleevi <rsleevi@chromium.org>
Reviewed-by: Emily Stark <estark@chromium.org>
Commit-Queue: Ryan Sleevi <rsleevi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552735}

Bug: 773382
Change-Id: Ibdaa19b8c9074e44f5ecd5984953d00d06d48f1a
Reviewed-on: https://chromium-review.googlesource.com/1022001
Commit-Queue: Sébastien Marchand <sebmarchand@chromium.org>
Reviewed-by: François Doray <fdoray@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552734}

This allows us to use the default favicon without having the initializer explicitly set it.

Bug: 825017
Change-Id: I86a8b2782bfe53c54ea0c937d1bbcfc94422601e
Reviewed-on: https://chromium-review.googlesource.com/1022181
Commit-Queue: Chris Lu <thegreenfrog@chromium.org>
Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Reviewed-by: Sergio Collazos <sczs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552733}

TBR=kbr@chromium.org

Bug: 829541
Change-Id: I6b6e481ec11abf895debe2cf3cfe44e812b2b436
Reviewed-on: https://chromium-review.googlesource.com/1023715Reviewed-by: Kai Ninomiya <kainino@chromium.org>
Commit-Queue: Kai Ninomiya <kainino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552732}

Bug: 734095
Change-Id: If429e9c97875e0a53242f2192f960e1def009667
Reviewed-on: https://chromium-review.googlesource.com/1024041
Commit-Queue: Xi Cheng <chengx@chromium.org>
Reviewed-by: Xi Cheng <chengx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552731}

TBR=thestig@chromium.org
NOTRY=true

Bug: 835869
Change-Id: I5a1030e9decb338c2ed8189fbe3500c57a0424c3
Reviewed-on: https://chromium-review.googlesource.com/1023023Reviewed-by: Xida Chen <xidachen@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552730}

Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I30b77b90e141b1c84a790eb2e8044fe7aa2b0eb0
Reviewed-on: https://chromium-review.googlesource.com/1022531Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Commit-Queue: Justin Cohen <justincohen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552729}

https://chromium.googlesource.com/angle/angle.git/+log/c6a0618fb02a..af2b33be5a3a

$ git log c6a0618fb..af2b33be5 --date=short --no-merges --format='%ad %ae %s'
2018-04-19 jiawei.shao Use ShaderMap in ProgramD3D - Part I
2018-04-23 oetuaho Add documentation for writing AST transformations
2018-03-27 geofflang GL: Fix the CPU readback path of CopyTextureCHROMIUM.
2018-04-23 lucferron Vulkan: Lift suppressions because of vulkan validation layer regression
2018-04-18 jiawei.shao ES31: Add validation on geometry shader interface blocks matching
2018-04-19 lucferron Vulkan: Implement Texture wrap modes and non-mipmapped filtering modes

Created with:
  roll-dep src/third_party/angle
BUG=chromium:693090


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

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

Change-Id: Id6d4968910d4b6d7ec1f601b950fb3dd8cf02ff0
Reviewed-on: https://chromium-review.googlesource.com/1023982
Commit-Queue: angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Reviewed-by: angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#552728}

TBR=thestig@chromium.org
NOTRY=true

Bug: 835865
Change-Id: Iaa027cb963cf6bd49bb4871bdedfc8ae0428020c
Reviewed-on: https://chromium-review.googlesource.com/1024202Reviewed-by: Xida Chen <xidachen@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552727}

This reverts commit 30b664a3.

Reason for revert: Introduced https://crbug.com/835541

Original change's description:
> macviews: handle user key equivalents
> 
> There is special logic in ChromeCommandDispatcherDelegate's
> prePerformKeyEquivalent: method that handles user key equivalents for builtin
> browser commands. This same logic needs to happen in the Views browser, so
> the Views-side logic needs to consult the BrowserView's accelerator table
> to see if the key is bound.
> 
> This involves an ungainly bridge, because it's not possible to include
> BrowserView and BrowserWindowController's full definitions at the same time -
> they have conflicting C++ and Objective-C class names in their dependencies.
> 
> Bug: 834754
> Change-Id: I67a62064d58cb7dba9c3bd532917ed1f90959340
> Reviewed-on: https://chromium-review.googlesource.com/1022530
> Reviewed-by: Robert Sesek <rsesek@chromium.org>
> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#552513}

TBR=ellyjones@chromium.org,rsesek@chromium.org

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

Bug: 834754
Change-Id: Ide55313a12460a254a90324d19591e2bf9c950db
Reviewed-on: https://chromium-review.googlesource.com/1023192Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552726}