Fix <webview> cursors with --site-per-process and --isolate-extensions.

Fix <webview> cursors with --site-per-process and --isolate-extensions.

When running BrowserPlugin without either flag, the guest sets the
cursor which is forwarded to the embedder. The embedder saves the
current guest cursor and will set the cursor to the guest value when
processing mouse events for the guest.

When running with either --site-per-process or --isolate-extensions, the
embedding renderer no longer receives the guest's mouse events which are
how the cursor ends up being set. In this case, directly set the cursor
with the owner view.

BUG=595446
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/6071fe20d749ff6ebde0da65c5e3f6fd83065cb9
Cr-Commit-Position: refs/heads/master@{#384193}

[avallee, 2016-03-30 22:13:25]

Description was changed from

==========
Fix <webview> cursors with --site-per-process and --isolate-extensions.

When running BrowserPlugin without either flag, the guest sets the
cursor which is forwarded to the embedder. The embedder saves the
current guest cursor and will set the cursor to the guest value when
processing mouse events for the guest.

When running with either --site-per-process or --isolate-extensions, the
embedding renderer no longer receives the guest's mouse events which are
how the cursor ends up being set. In this case, directly set the cursor
with the owner view.

BUG=595446
==========

to

==========
Fix <webview> cursors with --site-per-process and --isolate-extensions.

When running BrowserPlugin without either flag, the guest sets the
cursor which is forwarded to the embedder. The embedder saves the
current guest cursor and will set the cursor to the guest value when
processing mouse events for the guest.

When running with either --site-per-process or --isolate-extensions, the
embedding renderer no longer receives the guest's mouse events which are
how the cursor ends up being set. In this case, directly set the cursor
with the owner view.

BUG=595446
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

[avallee, 2016-03-30 22:38:00]

avallee@chromium.org changed reviewers:
+ piman@chromium.org

[avallee, 2016-03-30 22:38:01]

PTAL

[Charlie Reis, 2016-03-30 22:48:48]

Drive-by: Thanks for fixing!  Can you CC site-isolation-reviews@chromium.org on
future OOPIF-related CLs?  That helps us keep track of the work everyone on the
team is doing.

[piman, 2016-03-31 01:37:04]

lgtm

[avallee, 2016-03-31 02:21:55]

The CQ bit was checked by avallee@chromium.org

[commit-bot: I haz the power, 2016-03-31 02:22:10]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1845083002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1845083002/1

[commit-bot: I haz the power, 2016-03-31 03:27:19]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-31 03:28:49]

Description was changed from

==========
Fix <webview> cursors with --site-per-process and --isolate-extensions.

When running BrowserPlugin without either flag, the guest sets the
cursor which is forwarded to the embedder. The embedder saves the
current guest cursor and will set the cursor to the guest value when
processing mouse events for the guest.

When running with either --site-per-process or --isolate-extensions, the
embedding renderer no longer receives the guest's mouse events which are
how the cursor ends up being set. In this case, directly set the cursor
with the owner view.

BUG=595446
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Fix <webview> cursors with --site-per-process and --isolate-extensions.

When running BrowserPlugin without either flag, the guest sets the
cursor which is forwarded to the embedder. The embedder saves the
current guest cursor and will set the cursor to the guest value when
processing mouse events for the guest.

When running with either --site-per-process or --isolate-extensions, the
embedding renderer no longer receives the guest's mouse events which are
how the cursor ends up being set. In this case, directly set the cursor
with the owner view.

BUG=595446
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/6071fe20d749ff6ebde0da65c5e3f6fd83065cb9
Cr-Commit-Position: refs/heads/master@{#384193}
==========

[commit-bot: I haz the power, 2016-03-31 03:28:50]

Patchset 1 (id:??) landed as
https://crrev.com/6071fe20d749ff6ebde0da65c5e3f6fd83065cb9
Cr-Commit-Position: refs/heads/master@{#384193}