GitLab

https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Tools.git/+log/fe90a1d2dcbe..146eb3bdcf5b


git log fe90a1d2dcbe..146eb3bdcf5b --date=short --no-merges --format='%ad %ae %s'
2018-10-01 stevenperron@google.com Fix erroneous uses of the type manager in copy-prop-arrays. (#1942)


Created with:
  gclient setdep -r src/third_party/SPIRV-Tools/src@146eb3bdcf5b

The AutoRoll server is located here: https://autoroll.skia.org/r/spirv-tools-chromium-autoroll

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.


TBR=dsinclair@chromium.org

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

This allows Google Docs opened before Select-to-Speak was
enabled to still be read.

Bug: 882968
Change-Id: I723fcab46766983a8fe26477d2e9611e5bde1e9f
Reviewed-on: https://chromium-review.googlesource.com/1250601
Commit-Queue: Katie Dektar <katie@chromium.org>
Reviewed-by: David Tseng <dtseng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595553}

These tests are now failing on the bots due to the same reason
as other SpokenFeedbackTests.

BUG=889093
TEST=none

Change-Id: I1d3ac92f103e5aaf087fa0cb63ac88ceb5b8f631
Reviewed-on: https://chromium-review.googlesource.com/1252337Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Jun Mukai <mukai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595552}

In some cases we have a gesture on a previous navigation
but the `has_received_user_gesture` flag is not set so
we should also check the `has_received_user_gesture_before_nav`
flag too and allow propagation on the same eTLD+1.

BUG=884777

Change-Id: Iaabf03816bc4638f534321b1e9236bfa41d29938
Reviewed-on: https://chromium-review.googlesource.com/1255150Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Becca Hughes <beccahughes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595551}

Update browseraction.json docs for minor language and formatting

Bug: none
Change-Id: Id7f5f33bbd4193ed107d2e10da612d32f13a701b
Reviewed-on: https://chromium-review.googlesource.com/1217703
Commit-Queue: David Gash <dgash@google.com>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595550}

Bug: 850125
Change-Id: Ia8463b5edf15d2bf94a00ceab6286e8e20fc40ed
Reviewed-on: https://chromium-review.googlesource.com/1252861
Commit-Queue: Lindsay Pasricha <lindsayw@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595549}

This is similar to https://crrev.com/c/1178977

Bug: 882513
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: Ie3806648288b1da1f900f3ceb6385bd25a652f38
Reviewed-on: https://chromium-review.googlesource.com/1234320
Commit-Queue: Jonathan Backer <backer@chromium.org>
Reviewed-by: Antoine Labour <piman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595548}

This will guard https://skia-review.googlesource.com/c/skia/+/158344
It introduces a few pixel_test diffs due to the test framework's
processing of images after read-back.

Bug: skia:8382
Change-Id: Id0ca8c4205a5537665610086db348145855dd8c4
Reviewed-on: https://chromium-review.googlesource.com/1255206
Commit-Queue: Brian Osman <brianosman@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Reviewed-by: Mike Klein <mtklein@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595547}

This patch removes an unused function.

R=chrishtr@chromium.org, pdr@chromium.org, wangxianzhu@chromium.org

Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: Ib726be4ee83ce3032e1b1d44faac93d11086e879
Reviewed-on: https://chromium-review.googlesource.com/1255463Reviewed-by: Philip Rogers <pdr@chromium.org>
Commit-Queue: vmpstr <vmpstr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595546}

This will prevent keyboard navigation from reaching tiles that are not
visible.
Screencast: https://screencast.googleplex.com/cast/NTQ4MTM0NjE3MjQ1Mjg2NHxkYjlmMjExNy0zOQ

Bug: 889792
Change-Id: Id4c327417d97d9fa0b28f94fd3f98ddf97c4fc1d
Reviewed-on: https://chromium-review.googlesource.com/1252377
Commit-Queue: Kristi Park <kristipark@chromium.org>
Reviewed-by: Ramya Nagarajan <ramyan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595545}

Change-Id: Ib80fa764ccfddd7f1643448c7d529d3fad2d7aeb
Reviewed-on: https://chromium-review.googlesource.com/1255043Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: agrieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595544}

Change-Id: I4c70b035c7be332633e5b238c8cd63ac220fef72
Reviewed-on: https://chromium-review.googlesource.com/1254758Reviewed-by: Eric Stevenson <estevenson@chromium.org>
Commit-Queue: agrieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595543}

TwoClientAutofillProfileSyncTest.DeleteAndUpdate can only expect a
deterministic outcome if the server operates with a strong consistency
model.

In order to prove that, we introduce a second test that enables such
mode and is correspondingly more strict about the final outcome (update
should override deletion).

Bug: 870333
Change-Id: Ie4e6f5ebd604c837103237168391ba05595d1b22
Reviewed-on: https://chromium-review.googlesource.com/1249206Reviewed-by: Marc Treib <treib@chromium.org>
Reviewed-by: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595542}

Was missing from https://chromium-review.googlesource.com/c/chromium/src/+/1221687.

Change-Id: Ia2b53df370ebdcf33fccf1e6ccf080905403d369
Reviewed-on: https://chromium-review.googlesource.com/1255204Reviewed-by: anthonyvd <anthonyvd@chromium.org>
Commit-Queue: anthonyvd <anthonyvd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595541}

This CL uses ScopedBlockingCall to mark blocking calls in /chromecast/browser/metrics.

This CL was created by replacing calls to AssertBlockingAllowed()
with instantiations of ScopedBlockingCall(MAY_BLOCK).
I kindly ask the reviewer to make sure of the following:
  - ScopedBlockingCall is instantiated in a scope with minimal CPU usage.
    If this is not the case, ScopedBlockingCall should be instantiated
    closer to the blocking call. See scoped_blocking_call.h for more
    info. Please let me know when/where the blocking call happens if this needs
    to be changed.
  - Parameter |blocking_type| matches expectation (MAY_BLOCK/WILL_BLOCK). See
    BlockingType for more info. While I assumed MAY_BLOCK by default, that might
    not be the best fit if we know that this callsite is guaranteed to block.
  - The ScopedBlockingCall's scope covers the entirety of the blocking operation
    previously asserted against by the AssertBlockingAllowed().

This CL was uploaded by git cl split.

R=bcwhite@chromium.org

Bug: 874080
Change-Id: If065cdbaa716f646caa8c4258053a2f4a089bc21
Reviewed-on: https://chromium-review.googlesource.com/1191198
Commit-Queue: Etienne Pierre-Doray <etiennep@chromium.org>
Reviewed-by: Brian White <bcwhite@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595540}

Bug: b/115843337
Test: Update test case in unit_tests
Change-Id: I29bb87c4a138e4a14f99f4a28b1be54afeb1dedb
Reviewed-on: https://chromium-review.googlesource.com/1244441
Commit-Queue: Yue Li <updowndota@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595539}

Not all GoogleServiceAuthErrors are possible on iOS.
See //ios/chrome/browser/sync/sync_setup_service.cc for comparison.

Bug: 867548
Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs
Change-Id: I3eab6314ddb10e9d3f68b018494846b1257226aa
Reviewed-on: https://chromium-review.googlesource.com/1252378Reviewed-by: Eugene But <eugenebut@chromium.org>
Reviewed-by: Hiroshi Ichikawa <ichikawa@chromium.org>
Commit-Queue: John Wu <jzw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595538}

This filed is unused and should not be used by embedder.
NavigationManager provides enough methods to get last committed, pending
or visible URLs.

Bug: None
Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs
Change-Id: I44bece041b41a784512cb9a738dcd346ba71ddbd
Reviewed-on: https://chromium-review.googlesource.com/1251982Reviewed-by: Mohammad Refaat <mrefaat@chromium.org>
Commit-Queue: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595537}

BUG=890467

Change-Id: Ie0e7fc413c068a83c9e225c63bc3e3abdebf3120
Reviewed-on: https://chromium-review.googlesource.com/1252636
Commit-Queue: Alexei Filippov <alph@chromium.org>
Reviewed-by: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
Reviewed-by: Paul Irish <paulirish@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595536}

This fixes an issue, where we didn't copy across any shape exclusion
data when inserting a new shelf between two other shelves.

Bug: 635619
Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng
Change-Id: Ib26359ab8ac677034ec4e51f2adf1987dadbd91c
Reviewed-on: https://chromium-review.googlesource.com/1255145Reviewed-by: Christian Biesinger <cbiesinger@chromium.org>
Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595535}

The bug they were meant to help track down is now fixed, so no
reason to keep them around.

(Previously added in https://chromium-review.googlesource.com/c/chromium/src/+/1224670)

Bug: 882246
Change-Id: I4ada70dd810e07f6fc7ecaaf873fa99d4cf6263b
Reviewed-on: https://chromium-review.googlesource.com/1254464Reviewed-by: Asanka Herath <asanka@chromium.org>
Commit-Queue: Maks Orlovich <morlovich@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595534}

TBR=haraken@chromium.org

Bug: 885297
Change-Id: Ic7a6df89b771062e3327736fec64aff3d124286f
Reviewed-on: https://chromium-review.googlesource.com/1254322Reviewed-by: Charlie Harrison <csharrison@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Charlie Harrison <csharrison@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595533}

We don't want to require the feature to be on when *forcing*
dark mode.

Bug: 850098

Change-Id: Ie6a7f8e7a34e1436136664cf08db2015e69a790e
Reviewed-on: https://chromium-review.googlesource.com/1254862Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Leonard Grey <lgrey@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595532}

Currently extensions can intercept browser initiated requests using the web
request API which potentially allows them to interfere with browser
functionality in various ways. This CL makes the following changes:
  - All browser initiated non-navigation requests are now hidden from
    extensions.
  - All webui requests are now hidden from extensions. (WebUI shouldn't be
    making network requests anyway).
  - Restructure code in WebRequestPermissions::HideRequest for better
    efficiency.

BUG=884932

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

This CL uses ScopedBlockingCall to mark blocking calls in /ios/chrome.

This CL was created by replacing calls to AssertBlockingAllowed()
with instantiations of ScopedBlockingCall(MAY_BLOCK).
I kindly ask the reviewer to make sure of the following:
  - ScopedBlockingCall is instantiated in a scope with minimal CPU usage.
    If this is not the case, ScopedBlockingCall should be instantiated
    closer to the blocking call. See scoped_blocking_call.h for more
    info. Please let me know when/where the blocking call happens if this needs
    to be changed.
  - Parameter |blocking_type| matches expectation (MAY_BLOCK/WILL_BLOCK). See
    BlockingType for more info. While I assumed MAY_BLOCK by default, that might
    not be the best fit if we know that this callsite is guaranteed to block.
  - The ScopedBlockingCall's scope covers the entirety of the blocking operation
    previously asserted against by the AssertBlockingAllowed().

This CL was uploaded by git cl split.

R=rohitrao@chromium.org

Bug: 874080
Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs
Change-Id: Id297ebc108dc8f8915402602ae12a63e5a3131d0
Reviewed-on: https://chromium-review.googlesource.com/1191186
Commit-Queue: Etienne Pierre-Doray <etiennep@chromium.org>
Reviewed-by: David Roger <droger@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595530}

The lifecycle API is the interaction point for Chrome-side application
lifecycle events that the Feed Library cares about for the purposes
of maintaining its internal state.

Dummy files have been added so that compilation succeeds when the
Feed is disabled via buildflag.
This CL also adds a presubmit whitelist for all android/feed/dummy files.

(Google-internal design doc:
http://doc/1NyFLrqEAOoZe7MU2qy7bHYB6Rlp7c6lz22Zqzh-Cbo4)

Bug: 866950
Change-Id: Ie118c991aabb906e88344b5c96ab3b09e77c620f
Reviewed-on: https://chromium-review.googlesource.com/1231992
Commit-Queue: Patrick Noland <pnoland@chromium.org>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Reviewed-by: Theresa <twellington@chromium.org>
Reviewed-by: Filip Gorski <fgorski@chromium.org>
Reviewed-by: Sky Malice <skym@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595529}

SetupForInOutAnimation (in IconLabelBubbleView) sets
'EnableFlippingFoRTLUI' to true in all cases. Since only
ContentSettingImageView needs it set to true, removed it from the
method and it is now set separately in that subclass

Bug: 881798
Change-Id: I2d570cc84d9d4f9443b39ffa51d943d938624dbe
Reviewed-on: https://chromium-review.googlesource.com/1250055Reviewed-by: Evan Stade <estade@chromium.org>
Commit-Queue: Carlos IL <carlosil@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595528}

isHTML(), isScript(), isScriptURL() and isURL() added to
TrustedTypePolicyFactory class as part of Trusted Types API update to
current JS polyfill.
The methods require additional code to be fully matched to polyfill
expected behaviour.

Added a helper private method GetWrapperTypeInfoFromScriptValue.

Bug: 739170
Change-Id: I027e43ab6432405c686255a4d0ce24248c59a4dc
Reviewed-on: https://chromium-review.googlesource.com/1238433
Commit-Queue: Daniel Vogelheim <vogelheim@chromium.org>
Reviewed-by: Daniel Vogelheim <vogelheim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595527}

Mash implementation is a TODO.

Test: put in tablet mode, go to [1], press Lock in Current Orienation

[1] https://whatwebcando.today/screen-orientation.html

Bug: 889981
Change-Id: I19192a92ba927eba76c63089fa57c1288b34ff1c
Reviewed-on: https://chromium-review.googlesource.com/1252525
Commit-Queue: Evan Stade <estade@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595526}

Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs
Change-Id: Icc149b76bb5dc5abe436eac8e10ae9935f66fa69
Reviewed-on: https://chromium-review.googlesource.com/1251528Reviewed-by: Moe Ahmadi <mahmadi@chromium.org>
Reviewed-by: Olivier Robin <olivierrobin@chromium.org>
Commit-Queue: Olivier Robin <olivierrobin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595525}

Bug: 832676
Change-Id: I3088c0fa54d5b549f27dd158bbe46ec7834360a8
Reviewed-on: https://chromium-review.googlesource.com/1255207
Commit-Queue: Nico Weber <thakis@chromium.org>
Reviewed-by: Wei Li <weili@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595524}

This CL does the following:

1. Use VpxVideoDecoder in utility process to do software decoding for
vp8, vp9 video files.

2. Simplify the class to render media::VideoFrame into bitmap, since
PaintCanvasVideoRenderer works for all kind of VideoFrames.

3. Do not retrieve poster image from metadata. The mime type of the
poster image is inferred from mime sniffer, and needs to be
processed in sandbox process.


Bug: 826021
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: I91c9cf72a75be4bc3a95203a40633647e079a40a
Reviewed-on: https://chromium-review.googlesource.com/1234859
Commit-Queue: Xing Liu <xingliu@chromium.org>
Reviewed-by: David Trainor <dtrainor@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Reviewed-by: Tommy Li <tommycli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595523}

The function is somewhat slow; just get it from the constraint space
where we have one.

R=eae@chromium.org,mstensho@chromium.org

Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng
Change-Id: I08e8ba0a5d15d90f9a3ae4545302e0af618e73f4
Reviewed-on: https://chromium-review.googlesource.com/1254141
Commit-Queue: Christian Biesinger <cbiesinger@chromium.org>
Reviewed-by: Morten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595522}

There exist two interface through which a views::View is stitched to
a content::WebContentsView
- the ui::Layer is attached via a cr_setParentUiLayer method
- accessibility is attached via the accesibilityHostable protocol

Lifetimes of the embedding are slightly unclear because the attached
WebContentsView may be destroyed the NativeViewHost detaches it.

Replace the AccessibilityHostable protocol with a ViewsHostable
protocol, and add a C++ interface, ui::ViewsHostableView, which provides
a single place through which a views::View and a content::WebContentsView
are stitched together
- have content::WebContentsViewMac implement ui::ViewsHostableView
- have views::NativeWidgetHost implement ui::ViewsHostableView::Host

Use this explicit interface to ensure that the views::NativeWidgetHost
detaches from the content::WebContentsViewMac before it is destroyed.

Additional parameters and method will be added to the
ui::ViewsHostableView interface to enable stitching together
out-of-process NSViews.

R=lgrey
TBR=avi (content/ OWNERship)

Bug: 821561
Change-Id: Iddc46278cd06b9e27d3f12aa5a9e830273f3b562
Reviewed-on: https://chromium-review.googlesource.com/1252641
Commit-Queue: ccameron <ccameron@chromium.org>
Reviewed-by: Leonard Grey <lgrey@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595521}

Only DetachTimeFg16x is added because it is the most
different variation from control, while other variations
test values in between.


Bug: 847501
Change-Id: I12da837873f86345e36cfb9577a7a803015baa40
Reviewed-on: https://chromium-review.googlesource.com/1249805Reviewed-by: Robert Kaplow (sloooow) <rkaplow@chromium.org>
Commit-Queue: Etienne Pierre-Doray <etiennep@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595520}

There were two possible users of this logic:

1. RenderFrameImpl::LoadDataURL. This one is done during a commit time,
and it seems strange to reuse previous request for data urls, while
not reusing it for all other requests.

2. NetErrorHelper::LoadErrorPage. This one is called from
NetErrorHelper::DidFinishLoad through a chain of callbacks, so
there should not be a provisional document loader at the time of
commit and the logic is wasted.

Note that error page loads did switch to CommitDataNavigationWithRequest
recently, so are not using this logic anymore.

Bug: none
Change-Id: I3b00921c2c6441a007e8391ba3a4a8119d08c5bb
Reviewed-on: https://chromium-review.googlesource.com/1246599Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Camille Lamy <clamy@chromium.org>
Reviewed-by: Łukasz Anforowicz <lukasza@chromium.org>
Commit-Queue: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595519}

Also document GetCatalogTask and update the version checking logic to
be more accurate and clearer.

Bug: 889104
Change-Id: I2467dd767c7a19e5454d4b31412061ea884b2b7e
Reviewed-on: https://chromium-review.googlesource.com/1254741Reviewed-by: Peter Williamson <petewil@chromium.org>
Commit-Queue: Justin DeWitt <dewittj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595518}

Bug: 878390
Change-Id: I163448cb47ebd21e0c078a6b372466d163336186
Reviewed-on: https://chromium-review.googlesource.com/1228113Reviewed-by: Juan Antonio Navarro Pérez <perezju@chromium.org>
Commit-Queue: Camillo Bruni <cbruni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595517}

We are not ready for flatten in frames - the TargetRegistry is null.

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

Adding myself there to be able to track things easier.

Change-Id: Ia1228e69de4da77630208d12016754086f28b1be
Reviewed-on: https://chromium-review.googlesource.com/1253622Reviewed-by: Maksim Sisov <msisov@igalia.com>
Reviewed-by: Robert Kroeger <rjkroege@chromium.org>
Commit-Queue: Maksim Sisov <msisov@igalia.com>
Cr-Commit-Position: refs/heads/master@{#595515}