GitLab

Test: manual
Bug: 839290
Change-Id: Ib72e5b43a1a8a0e7dd429dbef90de9b6b1b820c6
Reviewed-on: https://chromium-review.googlesource.com/1042809Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Sammie Quon <sammiequon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555839}

Bug: 824568, 752273
Change-Id: Ia5a814ef83e1869e4d3065f5c0afa402f1ec7d61
Reviewed-on: https://chromium-review.googlesource.com/1041222
Commit-Queue: Ryan Hansberry <hansberry@chromium.org>
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555838}

Add implementation to create OFFER message and handle ANSWER response.
Add the MirrorSettings which hold a set of default settings.

Bug: 734672
Change-Id: Ia6b9af5ac66e1f8f9496c0afaa335a22f40bb535
Reviewed-on: https://chromium-review.googlesource.com/1017235
Commit-Queue: Xiangjun Zhang <xjz@chromium.org>
Reviewed-by: David Benjamin <davidben@chromium.org>
Reviewed-by: Yuri Wiitala <miu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555837}

A function used to not remove from the secondary table when deleting.
This CL adds a one-time per device clean-up of the rows in the
secondary tables that were left behind.

Bug: 836737
Change-Id: Icfe70b8ee98deb5a3de1fc1f3b08b197e9760132
Reviewed-on: https://chromium-review.googlesource.com/1038582
Commit-Queue: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Reviewed-by: Marc Treib <treib@chromium.org>
Reviewed-by: Roger McFarlane <rogerm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555836}

This CL names the base::ScopedClosureRunner object in the XmlParsingDone
function. Previously, a temporary object was created, so the closure
would be called at the end of the expression. Naming the object ensures
it won't be called until the function ends.

Change-Id: I80805f85d15e75f287d169b381144c7fabbea1e7
Reviewed-on: https://chromium-review.googlesource.com/1037704Reviewed-by: Jay Civelli <jcivelli@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Andres Calderon Jaramillo <andrescj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555835}

The ExtensionFunction::GetAssociatedWebContentsDeprecated() method is
unpredictable and can return surprising values in certain circumstances.
It is deprecated and should be removed to ensure that callers are using
the WebContents they expect.

Update the webstore APIs (dashboardPrivate, inlineInstallPrivate,
webstorePrivate, and webstoreWidgetPrivate) to use
GetSenderWebContents() instead.

Bug: 461394

Change-Id: I0c42797004c62218ab8de139ce58a175f796a50a
Reviewed-on: https://chromium-review.googlesource.com/1019567Reviewed-by: Karan Bhatia <karandeepb@chromium.org>
Commit-Queue: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555834}

FeedHostServiceFactory will create FeedImageManager when creates
FeedHostService.

Bug: 807359
Change-Id: Ibe4718c382972d16de6081ce8b84309e5308557f
Reviewed-on: https://chromium-review.googlesource.com/1041074
Commit-Queue: Gang Wu <gangwu@chromium.org>
Reviewed-by: Patrick Noland <pnoland@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555833}

Changes related to libsync caused a build failure on MacOS when EGL is enabled.

Credit to zmo@ for this fix, which I cherry picked from:
https://chromium-review.googlesource.com/c/chromium/src/+/1005017/19..20

Bug: chromium:757974
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
Change-Id: Ie4b38b3d0c782fea4e12f5d99e2f50add94ebf84
Reviewed-on: https://chromium-review.googlesource.com/1042488Reviewed-by: Kenneth Russell <kbr@chromium.org>
Commit-Queue: Alexis Hétu <sugoi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555832}

Bug: b/77279478
Test: cast_media_unittests
Change-Id: I45ede6c40cbe7fca9dc7273e1ee1e179184a43d5
Reviewed-on: https://chromium-review.googlesource.com/1040662Reviewed-by: Stephen Lanham <slan@chromium.org>
Commit-Queue: Sergey Volk <servolk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555831}

R=mmoroz@chromium.org

Change-Id: Ia83a10d1e5c5cb2d81c4bcea2b6839b0e47e1012
Reviewed-on: https://chromium-review.googlesource.com/1042808Reviewed-by: Max Moroz <mmoroz@chromium.org>
Commit-Queue: Max Moroz <mmoroz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555830}

Fixes a regression in Linux handling of omnibox text like dict:word.
Regression caused omnibox to default to treating it as a url. This
CL makes Linux default to treating it as search text.

Regression introduced in https://chromium-review.googlesource.com/c/chromium/src/+/966896

Bug: 838570
Change-Id: I26be4279605ded6e3aed4e631f503e2b60551283
Reviewed-on: https://chromium-review.googlesource.com/1040881Reviewed-by: Thomas Anderson <thomasanderson@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Commit-Queue: David Bienvenu <davidbienvenu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555829}

This change includes:
1) Adding new isolate for webview
2) Updating presentation for luci configs
  a) adding new pixel bots
  b) removing Mac 10.12 Laptop Low End that no longer exists
3) Generating new json for fyi waterfall
4) updating script on bot to ignore reference build for android

Bug: 817872
Change-Id: I8376b50dd0fcd8595977609205e8fbd8cb2985b2
Reviewed-on: https://chromium-review.googlesource.com/1039769
Commit-Queue: Emily Hanley <eyaich@chromium.org>
Reviewed-by: Nodir Turakulov <nodir@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555828}

We prefer to paint background-attachment: fixed into the scrolling
contents layer if text quality is important (at the cost of repainting
on every scroll). This approach had a bug for the LayoutView if there
turned out to be no scrolling layer (e.g., in a non-composited iframe).
Without a scrolling layer, a fixed background will not be clipped. This
patch ensures the background is clipped in ViewPainter.

Bug: 835755
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I6c19406fdacb64c5319709f9495e4fc8d608ae33
Reviewed-on: https://chromium-review.googlesource.com/1042266Reviewed-by: Steve Kobes <skobes@chromium.org>
Commit-Queue: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555827}

Thanks to thakis for pointing these out.

Bug: 830659
Change-Id: I45f285d29c16c666292a7479248c97222b6ff39f
Tbr: thakis@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/1041565Reviewed-by: Kenneth Russell <kbr@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555826}

This is supported on OSX and was triggered when running with
--enable-oop-rasterization and --enable-raster-decoder. This CL is
necessary before we can land http://crrev.com/c/1031012

Notable changes:

- shuffled some code from compositor_util to gpu_memory_support so that
  we can use it in a lower layer

- add proper gpu memory buffer support to InProcCmdBuffer

- add a raster_in_process_context_tests to give us some basic code
  coverage

- copy-pasta from GLES2DecoderImpl to RasterDecoderImpl to implement
  TexStorage2DImage

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
Change-Id: Iee75adbfd922ab0c94f2e563fe4fbdbf4c84f231
Reviewed-on: https://chromium-review.googlesource.com/964894Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Commit-Queue: Jonathan Backer <backer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555825}

https://chromium.googlesource.com/angle/angle.git/+log/e708f004bce5..03bb2231a821

$ git log e708f004b..03bb2231a --date=short --no-merges --format='%ad %ae %s'
2018-05-03 oetuaho Add test for D3D11 nested floor() bug

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


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

Change-Id: Ieb3d226ee13b6f9f8595063974b44c341877d2b7
Reviewed-on: https://chromium-review.googlesource.com/1042530
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@{#555824}

Change-Id: I7a3928869f8b585f4dabc6eba57b7206f2f03bef
Reviewed-on: https://chromium-review.googlesource.com/1041325Reviewed-by: Philippe Hamel <hamelphi@chromium.org>
Commit-Queue: Philippe Hamel <hamelphi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555823}

There are no existing tests (or any imported that I could find) that
verify captions' BFCs behave properly.

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: I56398bdd1ab6b993eaf8898e09b8aaa4b0ce0100
Bug: 635619
Reviewed-on: https://chromium-review.googlesource.com/1041219
Commit-Queue: David Grogan <dgrogan@chromium.org>
Reviewed-by: Morten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555822}

MojoAudioDecoderService, MojoVideoDecoderService and MojoRendererService
already hold a CdmContextRef to make sure the CDM attached to them will
always be valid. This CL brings this model to MojoDecryptorService to
make the code consistent. This will be used in the next CL when we host
MojoDecryptorService in MediaService.

Bug: 806018
Test: No functionality change.
Change-Id: I4f8f2fa519b00722252d2110a58fe2e6c2f8d56c
Reviewed-on: https://chromium-review.googlesource.com/1041658
Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
Reviewed-by: Frank Liberato <liberato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555821}

This ignores the computation error introduced when mapping to and from
the common planar of the layer and the chunk while the relative
transform between the layer and the chunk doesn't actually change.

Bug: 814711
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: Ie97b425eb3de601107328513b611b3f8676afa73
Reviewed-on: https://chromium-review.googlesource.com/1040537Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555820}

This Cl cleans up a TODO to remove an extra Layout call in Invalidate.
The extra call was a workaround for shifting text. I've identified some
unnecessary calls to SizeToPreferredSize() that was shifting the text.

This change should not cause any user visual changes.

This change may improve time to paint metrics.

Bug: None
Change-Id: I253b28e32c02d744a1e3d3c58b2522c04eefb323
Reviewed-on: https://chromium-review.googlesource.com/1041151Reviewed-by: Justin Donnelly <jdonnelly@chromium.org>
Commit-Queue: Dave Schuyler <dschuyler@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555819}

Remove an obsolete API that allows a crazy linker client to set
a library load offset. This was intended for loading libraries
directly from zip files, but was never used as is.

Besides, a future CL will refactor how libraries are located
and loaded from zip files in a completely different way (i.e.
adding the zip file path to the library search path).

BUG=802068
R=pasko@chromium.org, lizeb@chromium.org, agrieve@chromium.org

Change-Id: I33e2c1fe50a5a1accb42494f3a29a4d1128d5ea3
Reviewed-on: https://chromium-review.googlesource.com/1041963Reviewed-by: David Turner <digit@chromium.org>
Reviewed-by: Benoit L <lizeb@chromium.org>
Commit-Queue: David Turner <digit@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555818}

This is a precursor to having the loading parts of bookmarks create
the TitledUrlIndex. This should result in no change as the index is
only used once loaded.

BUG=680698
TEST=covered by existing tests

Change-Id: Ia059a26509f09472021bcfb31f24e9dcccd07d59
Reviewed-on: https://chromium-review.googlesource.com/1041528
Commit-Queue: Scott Violet <sky@chromium.org>
Reviewed-by: Jay Civelli <jcivelli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555817}

Also removes no longer needed guard flag for GrBackendObject.

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,linux_chromium_dbg_ng;master.tryserver.chromium.linux:linux_vr
Change-Id: I228f6bf5bcb554f6dcf31889c210ca2043f5a837
Reviewed-on: https://chromium-review.googlesource.com/1040425
Commit-Queue: Greg Daniel <egdaniel@chromium.org>
Reviewed-by: Brian Salomon <bsalomon@chromium.org>
Reviewed-by: Christopher Grant <cjgrant@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555816}

This CL removes a now-unneeded workaround for the old bug
https://crbug.com/476610 that would treat any proxy that responds with
the Data Reduction Proxy Via header as the Data Reduction Proxy, at
least for bypass/retry purposes. This workaround isn't needed anymore
since the root cause of that bug was found and there's code that fixes
the root cause of that bug at startup time (see
DataReductionProxyChromeSettings::
MigrateDataReductionProxyOffProxyPrefsHelper).

This CL also removes the associated histogram,
DataReductionProxy.ResponseProxyServerStatus.

Bug: 819066, 791849, 708029
Change-Id: Ica4e707866e0a81c17c3f6d3a27e068734e26744
Reviewed-on: https://chromium-review.googlesource.com/1040827Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Reviewed-by: Brian White <bcwhite@chromium.org>
Commit-Queue: Scott Little <sclittle@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555815}

We prefer to paint background-attachment: fixed into the scrolling
contents layer if text quality is important (at the cost of repainting
on every scroll). This approach had a bug for the LayoutView if there
turned out to be no scrolling layer (e.g., in a non-composited iframe).
The counterscroll logic in FixedAttachmentPositioningArea should
only apply if actually painting into a scrolling layer. This patch
queries the background compositing state before applying the counter-
scroll. This is safe because it occurs during paint which is after the
compositing decisions have been made.

Bug: 835755
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: Ieebffb50579b28488ef802736dd1210b8b27b26d
Reviewed-on: https://chromium-review.googlesource.com/1036941Reviewed-by: Steve Kobes <skobes@chromium.org>
Commit-Queue: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555814}

This CL is a refactor with no intended behavior change.

Bug: 832879
Change-Id: I6d54051f50ed4b0c46f821872aafea1ba4b3fd32
Reviewed-on: https://chromium-review.googlesource.com/1040590Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Erik Chen <erikchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555813}

https://boringssl.googlesource.com/boringssl/+log/cece32610b89549386b42b2032dd5d8a423af6c8..8e75ae488047c519f14f2c08b02a55bf7712fa1d

BUG=none

Change-Id: I36edb8ad89f019bddea940ca7b6433ea3d8ef86e
Reviewed-on: https://chromium-review.googlesource.com/1040588
Commit-Queue: Steven Valdez <svaldez@chromium.org>
Reviewed-by: David Benjamin <davidben@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555812}

Change-Id: I4185507af93c77d0e2514769dfbd66376591302f
Reviewed-on: https://chromium-review.googlesource.com/1042791Reviewed-by: Stephen Martinis <martiniss@chromium.org>
Commit-Queue: John Budorick <jbudorick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555811}

Bug: 773295
Change-Id: Ibc2789d9d0202f4e36bbc88b472107a7d99e52bb
Reviewed-on: https://chromium-review.googlesource.com/1038244
Commit-Queue: Mark Pilgrim <pilgrim@chromium.org>
Reviewed-by: Yury Khmel <khmel@chromium.org>
Reviewed-by: Luis Hector Chavez <lhchavez@chromium.org>
Reviewed-by: Matt Menke <mmenke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555810}

NOTRY=true  # tests are covered by CQ

Bug: 836446
Change-Id: I65548078aa4c280fa0be6f29ed236e41b4194ee3
Reviewed-on: https://chromium-review.googlesource.com/1042275Reviewed-by: Emily Hanley <eyaich@chromium.org>
Commit-Queue: Ned Nguyen <nednguyen@google.com>
Cr-Commit-Position: refs/heads/master@{#555809}

TL;DR for non-mojo reviews: No functional changes in your code. All
mechanical aesthetic changes.

This cleans up all message pipe, data pipe, and shared buffer APIs to
reflect the intended stable ABI of the initial mojo_core release, adding
extensible options structs where appropriate and homogenizing argument
ordering conventions. Also renames some badly named types and begins
eliminating the split C/C++ flag constant definitions in favor of using
C macros everywhere.

Documentation formatting is also made more consistent, and the thunk
table ordering is mostly finalized here.

The only functional change is a small drive-by bugfix in
MojoGetBufferInfo where the implementation was not filling in the actual
size of the MojoSharedBufferInfo struct. This has no effect on any
current production uses of the API.

TBR=miu@chromium.org
TBR=jam@chromium.org
TBR=kinuko@chromium.org

Bug: 819046
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: I5c1019bf1fe95a9295a0d053544e867fdebadde8
Reviewed-on: https://chromium-review.googlesource.com/1041149
Commit-Queue: Ken Rockot <rockot@chromium.org>
Reviewed-by: Jay Civelli <jcivelli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555808}

In ContentViewRenderView, waiting an extra frame to remove the
background prevents flashes of black from Android's default Surface
content, and flashes of the compositor's default background.

Bug: internal b/73804884
Test: repro bug
Change-Id: Ib221403890200be6e842e0d85780d18db859d159
Reviewed-on: https://chromium-review.googlesource.com/1041185Reviewed-by: Bo <boliu@chromium.org>
Commit-Queue: Thoren Paulson <thoren@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555807}

PaintLayer::GetBackgroundPaintLocation returns a mask of where the
background should paint and is used for determining if composited
scrolling can occur at all through:
PaintLayerScrollableArea::ComputeNeedsCompositedScrolling.

If composited scrolling is not used, callsites handle not actually
painting onto the scrolling contents layer, see:
CompositedLayerMapping::UpdateBackgroundPaintsOntoScrollingContentsLayer

This patch breaks the dependency cycle introduced by [1]. Instead of
querying compositing state, this patch special-cases the root layer
so that kBackgroundPaintInScrollingContents can be returned even if
there is no overflow because the root layer creates scrolling contents
layers in this case.

[1] https://crrev.com/9e0775c12ad12

Bug: 835755
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I139f2f85c68d267de80c7b1e534a9fcef9af3983
Reviewed-on: https://chromium-review.googlesource.com/1042529Reviewed-by: Steve Kobes <skobes@chromium.org>
Commit-Queue: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555806}

|childWindowsDidChange| was removed from the delegate, so these methods
are now noops.

Change-Id: I5becdd5797c0adc14a9c2f53d0f78d5ce684ceea
Reviewed-on: https://chromium-review.googlesource.com/1040946Reviewed-by: Sidney San Martín <sdy@chromium.org>
Commit-Queue: Leonard Grey <lgrey@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555805}

It has another cert for OCSP tests not used by the EmbeddedTestServer.

Bug: 755309
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: Ic43314cf9f8e2978848a75764a4323ce34b0006a
Reviewed-on: https://chromium-review.googlesource.com/1033192
Commit-Queue: Matt Menke <mmenke@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Ryan Sleevi <rsleevi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555804}

Updates to the media_router WATCHLIST:
- Add c/b/ui/media_router/
- Fix the path for c/common/media_router/

Change-Id: I286bbae96da0f801997e69634642a19001e46ac6
Reviewed-on: https://chromium-review.googlesource.com/1042276Reviewed-by: Adam Parker <amp@chromium.org>
Commit-Queue: Takumi Fujimoto <takumif@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555803}

Since b66db7c4, WebView can now get the
channel. Variations uses this to filter experiments.

Stand-alone WebView will report "unknown" channel. Experiments which do
not specify any channel will apply to stand-alone WebView; see
CheckStudyChannel in study_filtering.cc.

BUG=758290

Change-Id: I6174b63c6995cb950e5ec349891ad579d24e7396
Reviewed-on: https://chromium-review.googlesource.com/1041206Reviewed-by: Bo <boliu@chromium.org>
Commit-Queue: Paul Miller <paulmiller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555802}

TBR=nednguyen@google.com
NOTRY=true

Bug: 839494
Change-Id: Idd671cbbf0cdd81f7a3417573d03e4e6cda6e4c6
Reviewed-on: https://chromium-review.googlesource.com/1042282Reviewed-by: Annie Sullivan <sullivan@chromium.org>
Commit-Queue: Annie Sullivan <sullivan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555801}

Bug: 598073

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: I59722f8aa6c1946c3f2cf00bbbad0d8c96e0d12f
Reviewed-on: https://chromium-review.googlesource.com/1041486Reviewed-by: Matt Menke <mmenke@chromium.org>
Commit-Queue: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555800}