GitLab

The handle conversion requires an extra cast on Windows.

TBR=ajwong@chromium.org

Change-Id: If7da57e0314496e3922cd0634b4ec8ab4e063f48
Reviewed-on: https://chromium-review.googlesource.com/580668Reviewed-by: Brett Wilson <brettw@chromium.org>
Commit-Queue: Brett Wilson <brettw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488511}

It is no longer used as a result of sending presentation connection
messages via the PresentationConnection interface.

Bug: 687011
Change-Id: Ic537619a08865d33a79f3dffa29d82473d1b4e5e
Reviewed-on: https://chromium-review.googlesource.com/580049Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Derek Cheng <imcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488510}

Adds Cast for Education as a possible MediaSink icon type. This will be a no-op until the component extension supports the icon type.

Bug: 681048
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ie61c80887956709ca88b52ae45894bf521ee938a
Reviewed-on: https://chromium-review.googlesource.com/577981
Commit-Queue: Takumi Fujimoto <takumif@chromium.org>
Reviewed-by: Jennifer Apacible <apacible@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488509}

BUG=715781

Change-Id: Id936a1c57a8d8d5796cec0bc013af4cce26f16b8
Reviewed-on: https://chromium-review.googlesource.com/580647Reviewed-by: Alexander Alekseev <alemate@chromium.org>
Commit-Queue: Toni Barzic <tbarzic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488508}

Currently the onPreDrawListener for native pages is called even after
VrShell exits. A fix is to unregister our onPreDrawListener during
VrShell shutdown.

VrShell destruction is ordered to remove VrShell from the view hierarchy,
then notify VrShell to do clean up.  Unfortunately removing VrShell
from the view hierarchy prevents cleaning up our onPreDrawListener, so
we need a new notification to clean up while still in the view
hierarchy.

The fix is to remove our OnPreDrawlistener in response to a new cleanup
method called while we are still hosted in the view.

BUG=743119

Change-Id: Ieacdfb6a08799d4347ea2b268070fd6543f6a2c7
Reviewed-on: https://chromium-review.googlesource.com/580107
Commit-Queue: Bill Orr <billorr@chromium.org>
Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488507}

-Adds logic to correctly resize on orientation changes.
-Adds logic to set clip bounds on resize
  (i.e. image loads or context menu shows/hides all of the link text).
-Sets rounded corners programmatically instead of in the context menu's xml file.

Note: this does not handle the image loading or link text view enlargement animations.

BUG=730329

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

The fix-it to remove the uses of To/FromInternalValue() is underway.
This change adds a PRESUBMIT warning (BANNED_CPP_FUNCTIONS) and
base/time/time.h header comments, to avoid having new code use these
deprecated functions.

The intention is for this change to be short-lived, and removed once
the deprecated functions are removed from the time classes.

One fault in the check: Since the script can only do regexp matching,
it's impossible to check that the ToInternalValue() method is being
called on one of the TimeXXX types. However, there are relatively few
non-TimeXXX types that have a ToInternalValue() method. Therefore, it
should be rare for the script to trigger a warning erroneously (and
this would be easy to bypass, anyway).

Bug: 634507
Change-Id: I48d75435703771107c87935cb3ea90a5a956989e
Reviewed-on: https://chromium-review.googlesource.com/577980Reviewed-by: Paweł Hajdan Jr. <phajdan.jr@chromium.org>
Commit-Queue: Yuri Wiitala <miu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488505}

Original CL: https://codereview.chromium.org/2418173002/

Revert CL: https://codereview.chromium.org/2463103002/
https://codereview.chromium.org/2474073004
https://codereview.chromium.org/2544893006

Since it was reverted long time ago, reimplement it from the scratch.

HW decoded video is blurry, because crbug.com/575587 shinked visual rect by 1
pixel, if coded rect is larger than visual rect. This fixed bleeding texture,
but caused blurry issue.

Meanwhile, crbug.com/429640 fixed bleeding texture for software YUV video in
the elegant way. It clamps all texture coordinates to this maximum value in the
fragment shader.

This CL removes the 1 pixel hack and applies the way to TextureDrawQuad and
StreamVideoDrawQuad.

In addition, Video overlay requires this fix because there is noticible
transition between GPU composition and hardware overlay.

TEST=cc_unittests PrecisionSamplerShadersCompile/PrecisionSamplerShaderPixelTest.ShadersCompile
chrome on http://browsertests.herokuapp.com/media/sharpness.html using amd64-generic
BUG=615325, 683347

Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;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: I63c8736a365f0024513bfe1e999bc7beb7321da5
Reviewed-on: https://chromium-review.googlesource.com/567809
Commit-Queue: Dongseong Hwang <dongseong.hwang@intel.com>
Reviewed-by: ccameron chromium <ccameron@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Reviewed-by: Antoine Labour <piman@chromium.org>
Reviewed-by: Dongseong Hwang <dongseong.hwang@intel.com>
Reviewed-by: Dan Sanders <sandersd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488504}

Ash in mash mode was reparenting system modal dialogs. Problem is the
code was using GetDefaultParent(), which makes some assumptions that
are not always right for Chrome code. This fixes the issue by making
it so that when the modal type changes ash doesn't change the parent.

I suspect we will have to revisit this at some point, but this is the
easy fix for now.

BUG=743064
TEST=covered by test

Change-Id: I21b097b5ad2c52e153b0880ec33c96e950fad624
Reviewed-on: https://chromium-review.googlesource.com/580631Reviewed-by: James Cook <jamescook@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488503}

In order to add serialization for PaintFlags internals, store the
SkPaint internals directly in PaintFlags instead of storing in
an SkPaint itself.

Reland of: https://chromium-review.googlesource.com/c/575511/
TBR=vmpstr@chromium.org,msw@chromium.org,junov@chromium.org

Bug: 737629
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: Ida37d539b3d5a5612acdf1466445f4d30b75d5ad
Reviewed-on: https://chromium-review.googlesource.com/580149
Commit-Queue: enne <enne@chromium.org>
Reviewed-by: Dan Erat <derat@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488502}

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/0e1c43e75abf..d8bc21bf34bf

$ git log 0e1c43e75..d8bc21bf3 --date=short --no-merges --format='%ad %ae %s'
2017-07-20 benjhayden Prevent findInputExpectations() from producing too many warnings.
2017-07-20 simonhatch Pinpoint - Plumb bug_id through to backend.

Created with:
  roll-dep src/third_party/catapult
BUG=743201


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: I580264c70084c7d95fcfd578ae56a0494b561747
Reviewed-on: https://chromium-review.googlesource.com/579801
Reviewed-by: <catapult-deps-roller@chromium.org>
Commit-Queue: <catapult-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488501}

This test verifies composited scrollers have consistent clip rect
regardless of current scroll offset. The four boxes in the test have
different fractional translations. The test body and reference are
identical except for scroll offset.

BUG=736052

Change-Id: I97257b29e5abf579aa028d0c04463f4c427f8a58
Reviewed-on: https://chromium-review.googlesource.com/578741
Commit-Queue: Tien-Ren Chen <trchen@chromium.org>
Reviewed-by: Peter Mayo <petermayo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488500}

As per UI review request to look the seperator for the tiles in the system tray bubble.

Test: manual
Bug: 747148
Change-Id: I2259bc498fb571d1c44d7e42a9970168c5872436
Reviewed-on: https://chromium-review.googlesource.com/580636Reviewed-by: James Cook <jamescook@chromium.org>
Commit-Queue: Sammie Quon <sammiequon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488499}

TBR=yzshen@chromium.org
NOTRY=true

Change-Id: I1ecb94c52f9accad18bd8da5986daf23bcb61066
Reviewed-on: https://chromium-review.googlesource.com/580308
Commit-Queue: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488498}

Bug: 746502
Change-Id: I883596553eb46ccb5b927cd5eb0d98a2beb93607
Reviewed-on: https://chromium-review.googlesource.com/576455Reviewed-by: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Reviewed-by: Mathieu Perreault <mathp@chromium.org>
Commit-Queue: Mathieu Perreault <mathp@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488497}

Bug: 735701
Change-Id: I99b4e0c075c8c4787ae84eaf45351e5e69d1fa20
Reviewed-on: https://chromium-review.googlesource.com/575308Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488496}

Currently there are some entries in W3CImportExpectations that do not closely
follow the syntax and hence cannot be picked up by DirectoryOwnersExtractor.
This CL fixes these entries. It is also in prepartion for converting the [Pass]
entries to OWNERS files, as the conversion script will be based on
DirectoryOwnersExtractor. (I could make these changes locally and then run the
script, but that would make the conversion process harder to trace and debug, in
case there is any need in the future.)

Bug: 713987
Change-Id: Iab1ed744e6fedbf9e805113badd4ff50e9e0347c
Reviewed-on: https://chromium-review.googlesource.com/580395Reviewed-by: Quinten Yearsley <qyearsley@chromium.org>
Commit-Queue: Robert Ma <robertma@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488495}

Lots of things want to anchor off the page info icon. E.g.,
permission prompts, the device chooser UI, the first run bubble
and the page info bubble itself.

Unfortunately, the page info bubble is not always present (in
fullscreen for example), and we want to anchor toolkit-views bubbles
off a BrowserView as well as off a BrowserWindowCocoa. So anchoring
is not simple. Currently the logic that deals with this is scattered
around the codebase, and some of it is broken (e.g. crbug/745291).

This CL consolidates the anchoring for permission prompts and the
device chooser UI, for both of their Cocoa and Views versions. The
first run bubble will also use this in a follow-up.

The PageInfo bubble anchoring is a bit simpler since it can't be
shown in fullscreen. It's consolidated partially, but it has some
unnecessary plumbing as well, so more cleanups there are left for
another follow-up).

Bug: 728160, 745291
Change-Id: Id8b3bb9284f8dcc83299aa92123d4ee6e58dddc4
Reviewed-on: https://chromium-review.googlesource.com/575791
Commit-Queue: Trent Apted <tapted@chromium.org>
Reviewed-by: Michael Wasserman <msw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488494}

Bug: 746653
Change-Id: I5ce0bfb68527968feff3da5e034dee78163f36b3
Reviewed-on: https://chromium-review.googlesource.com/578988
Commit-Queue: Victor Costan <pwnall@chromium.org>
Reviewed-by: Chris Mumford <cmumford@chromium.org>
Reviewed-by: Joshua Bell <jsbell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488493}

Using wpt-import in Chromium 8fc1f929.

Build: https://build.chromium.org/p/chromium.infra.cron/builders/wpt-importer/builds/349

Note to sheriffs: This CL imports external tests and adds
expectations for those tests; if this CL is large and causes
a few new failures, please fix the failures by adding new
lines to TestExpectations rather than reverting. See:
https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md

Directory owners for changes in this CL:
dom-dev@chromium.org:
  external/wpt/custom-elements

TBR=qyearsley@chromium.org

No-Export: true
Change-Id: I1cc6e148aba2ef921fa55c8ddb010da489b57cf9
Reviewed-on: https://chromium-review.googlesource.com/580394
Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org>
Reviewed-by: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org>
Reviewed-by: Quinten Yearsley <qyearsley@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488492}

Bug: 
Change-Id: I932892f3cc4c9f01d31139a0744f8049a1b7a332
Reviewed-on: https://chromium-review.googlesource.com/580545Reviewed-by: David Benjamin <davidben@chromium.org>
Commit-Queue: David Benjamin <davidben@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488491}

There are still tests that wraps NavigationBarDoneButton with a local
helper function, which is unnecessary, instead, we should call EarlGrey
explicitly like following:

[[EarlGrey selectElementWithMatcher:NavigationBarDoneButton()]
      performAction:grey_tap()];

So, this CL cleans them up.

Bug: 741898
Change-Id: I74ea738cb1e58b1ee44da57c1dfaf74b7c348405
Reviewed-on: https://chromium-review.googlesource.com/568271Reviewed-by: Eugene But <eugenebut@chromium.org>
Reviewed-by: Mihai Sardarescu (OOO) <msarda@chromium.org>
Reviewed-by: Jérôme Lebel <jlebel@chromium.org>
Reviewed-by: Mike Baxley <baxley@chromium.org>
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488490}

TBR=danakj@chromium.org

Bug: Documentation update
Change-Id: I47be9a2f109001643c9910ae4db49e056baccc08
Reviewed-on: https://chromium-review.googlesource.com/580544Reviewed-by: Gabriel Charette <gab@chromium.org>
Commit-Queue: Gabriel Charette <gab@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488489}

The notification can be shown during two point in time: when the user logs in,
and when the user disconnects from the Internet. In order for the notification
to be shown, Bluetooth must be disabled, the device must be disconnected from
the Internet, and there must be nothing besides Bluetooth preventing a Tether
scan from beginning.

This CL also changes the ownership of TetherNotificationPresenter. Now, one
instance is created and owned by TetherService and is passed through to the
Tether component when needed. Previously, the instance was owned by the Tether
component, but this no longer worked due to the fact that TetherService needed
to be able to display the "enable Bluetooth" notification when Tether was left
uninitialized (due to Bluetooth being disabled).

Lastly, now TetherService only sets Tether's TechnologyState to
UNINITIALIZED when Bluetooth is disabled, which fixes an issue in which
an "enable Bluetooth" message is shown in Quick Settings when Bluetooth
is on but Cellular is disabled.

Bug: 672263, 740635
Change-Id: Id19a847a9efb7a31d23d2ffb6308fad7d46ca948
Reviewed-on: https://chromium-review.googlesource.com/580349
Commit-Queue: Kyle Horimoto <khorimoto@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488488}

- The 'Learn More' button on the incognito NTP either opens a new
  activity or a new non-icognito tab. Both is not supported in VR.
  As a workaround for now, request to exit VR and if we succeed show the
  'Learn More' page in 2D.

Bug: 742668
Change-Id: I23b9a2447ab297468dd9612c3e3e351ef59b3e40
Reviewed-on: https://chromium-review.googlesource.com/578169
Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: Peter Conn <peconn@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488487}

Bug: 734150
Change-Id: I8b6aac965c5b627197f62f9f732f154452bfe3d4
Reviewed-on: https://chromium-review.googlesource.com/579607
Commit-Queue: Danyao Wang <danyao@chromium.org>
Reviewed-by: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488486}

Move all tablet mode related events from shell observer to a new tablet mode observer.

Test: all trybots especially ash tests
Bug: 743148
Change-Id: Idbc601cbf59476c3928fa7ba63ef1dd58fb83893
Reviewed-on: https://chromium-review.googlesource.com/572053
Commit-Queue: Sammie Quon <sammiequon@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Michael Wasserman <msw@chromium.org>
Reviewed-by: Jonathan Ross <jonross@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488485}

Hopefully, this can help to make your code reviews slightly less
bottleneck-y.

Bug: 
Change-Id: Ic56d9c213927e97d4e756e8e4cf52d8b8c38b412
Reviewed-on: https://chromium-review.googlesource.com/580504Reviewed-by: Mike Baxley <baxley@chromium.org>
Reviewed-by: Eugene But <eugenebut@chromium.org>
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488484}

Follow up to https://chromium-review.googlesource.com/c/579937/ which
removes //tools/auto_bisect. Reason: These are no longer used and will
no longer work after removing //tols/auto_bisect.

Change-Id: Ibe90a9ced87ee34a6673cd7f77645eb20b735f9f
Reviewed-on: https://chromium-review.googlesource.com/580492Reviewed-by: Roberto Carrillo <robertocn@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: Quinten Yearsley <qyearsley@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488483}

BUG=726972

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

Bug: 740070
Change-Id: I9d0a7371c22856a641ad88910d55ba844aca4191
Reviewed-on: https://chromium-review.googlesource.com/572332
Commit-Queue: Vincent Scheib <scheib@chromium.org>
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488481}

This CL adds support for specifying a short blacklist of hosts in a
field-trial param of the Client-side LoFi field trial, which will
prevent Client LoFi from being used on those hosts.

TBR=petewil@chromium.org

Bug: 745032
Change-Id: I55a7ae8b3ebcdb5a98c7229a9f714757754b7d29
Reviewed-on: https://chromium-review.googlesource.com/578588Reviewed-by: Scott Little <sclittle@chromium.org>
Reviewed-by: Ryan Sturm <ryansturm@chromium.org>
Reviewed-by: Matt Menke <mmenke@chromium.org>
Commit-Queue: Scott Little <sclittle@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488480}

This CL moves some hardcoded texts into remoting_string.grd so that
they can be localized.

Bug: 747091
Change-Id: Id9f57bd7d48338635d2c7d0567df1b8e24a83b0b
Reviewed-on: https://chromium-review.googlesource.com/578949
Commit-Queue: Yuwei Huang <yuweih@chromium.org>
Reviewed-by: Scott Nichols <nicholss@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488479}

Adds a call to hideKeyboard when the sheet is closed

Bug: 746803
Change-Id: Ib9d7a6a4696e7bf8f1864d0aaf3d6d3bb734f8d5
Reviewed-on: https://chromium-review.googlesource.com/579790
Commit-Queue: Daniel Park <danielpark@chromium.org>
Reviewed-by: Theresa <twellington@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488478}

Make our git diffs decent!
Preparation for:
https://chromium-review.googlesource.com/c/572164/

Bug=667560

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

When I added this method, I had added some extra logic for the case where we
have a collapsed selection to make it find spell check markers the caret is
touching an endpoint of. This was to support the spell check menu on Android,
where tapping the endpoint of a word brings up the menu (on other platforms you
need to right-click in the interior of the word for spell check options to
appear in the context menu).

Upon further reflection, we decided we should put this logic somewhere else.
We're going to keep this method around though because we're going to make use of
it in ContextMenuClient::SelectMisspellingAsync().

I'm renaming this method to GetSpellCheckMarkerUnderSelection() to try to make
the name less misleading.

BUG=736181

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

This adds UMA histograms for keeping track of how long it takes for
the Chrome Cleaner to scan and clean user's machines as well as the
time it takes for users to interact with the Chrome Cleaner prompt
dialog.

Bug: 746987
Change-Id: If4d70c196f68c462834cb0c1e8a82a8bfaa6ea05
Reviewed-on: https://chromium-review.googlesource.com/580087
Commit-Queue: Ali Tofigh <alito@chromium.org>
Reviewed-by: Robert Shield <robertshield@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488475}

Instead of transitioning to HALF or FULLSCREEN_SEARCH whenever the
search query is not empty, transition only when the query is not empty
and contains some non-whitespace characters.

This will be changed when Zero Query State is introduced, at a later
date.

for-Commit-Position: refs/heads/master@{#487254}
Bug: 746048
Change-Id: I78a2eb73589d3a13c2213d9f4f1d26f2c6162dc8
Reviewed-on: https://chromium-review.googlesource.com/578470Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Reviewed-by: Yury Khmel <khmel@chromium.org>
Commit-Queue: Alex Newcomer <newcomer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488474}

Change some of the strings on the palette tray buble.

Test: none
Bug: 747148
Change-Id: I22a2a2480f2884091bffb1f5b9ac4bd04581990c
Reviewed-on: https://chromium-review.googlesource.com/580365Reviewed-by: Sammie Quon <sammiequon@chromium.org>
Reviewed-by: Jacob Dufault <jdufault@chromium.org>
Commit-Queue: Sammie Quon <sammiequon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488473}

Bug: 742990
Change-Id: Ibe303f5e0f5a94015adad2b0d90de963b25d61fb
TBR: tsepez (removing params from ipc message)
Reviewed-on: https://chromium-review.googlesource.com/572222
Commit-Queue: Pavel Feldman <pfeldman@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488472}