Cancel composition on RenderWidgetHostCocoaView after a child frame's RWHV receives the correspondi…

Cancel composition on RenderWidgetHostCocoaView after a child frame's RWHV receives the corresponding IPC.

This CL will add the required plumbing for Mac to cancel an ongoing
composition on the browser side when the renderer has canceled its
ongoing composition.

BUG=578168, 602723
Committed: https://crrev.com/7006d2f9c3804eb9a6aedbb328223b22df51eacc
Cr-Commit-Position: refs/heads/master@{#411151}

[EhsanK, 2016-08-09 20:51:39]

The CQ bit was checked by ekaramad@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-09 20:52:10]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-09 21:52:11]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-09 21:52:14]

Dry run: This issue passed the CQ dry run.

[EhsanK, 2016-08-09 22:11:27]

Description was changed from

==========
Cancel composition on RenderWidgetHostCocoaView after a child frame's RWHV
receives the corresponding IPC.

This CL will add the required plumbing for Mac to cancel an ongoing composition
on the browser side when the renderer has canceled its ongoing composition.

BUG=578168
==========

to

==========
Cancel composition on RenderWidgetHostCocoaView after a child frame's RWHV
receives the corresponding IPC.

This CL will add the required plumbing for Mac to cancel an ongoing
composition on the browser side when the renderer has canceled its
ongoing composition.

BUG=578168,602723
==========

[EhsanK, 2016-08-09 22:12:07]

The CQ bit was checked by ekaramad@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-09 22:12:52]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[EhsanK, 2016-08-09 23:48:21]

ekaramad@chromium.org changed reviewers:
+ creis@chromium.org, kenrb@chromium.org, nasko@chromium.org

[EhsanK, 2016-08-09 23:48:22]

PTAL.

This patch reminded me that we need a similar test for Aura. I will work on that
after I am finished with the rest of Mac.

[commit-bot: I haz the power, 2016-08-10 00:14:28]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-10 00:14:29]

Dry run: This issue passed the CQ dry run.

[kenrb, 2016-08-10 19:40:48]

lgtm

[Charlie Reis, 2016-08-10 19:43:52]

creis@chromium.org changed reviewers:
+ avi@chromium.org

[Charlie Reis, 2016-08-10 19:43:53]

I'm not a Cocoa reviewer; maybe Avi can do a quick sanity check on the .mm
files?

Otherwise content/ LGTM.

[Avi (use Gerrit), 2016-08-10 19:58:11]

LGTM

[EhsanK, 2016-08-10 19:59:21]

Thank you for the reviews!

[EhsanK, 2016-08-10 19:59:31]

The CQ bit was checked by ekaramad@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-10 20:00:16]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-10 20:03:46]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-10 20:03:48]

Dry run: This issue passed the CQ dry run.

[EhsanK, 2016-08-10 21:30:10]

The CQ bit was checked by ekaramad@chromium.org

[commit-bot: I haz the power, 2016-08-10 21:30:59]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-10 21:38:47]

Description was changed from

==========
Cancel composition on RenderWidgetHostCocoaView after a child frame's RWHV
receives the corresponding IPC.

This CL will add the required plumbing for Mac to cancel an ongoing
composition on the browser side when the renderer has canceled its
ongoing composition.

BUG=578168,602723
==========

to

==========
Cancel composition on RenderWidgetHostCocoaView after a child frame's RWHV
receives the corresponding IPC.

This CL will add the required plumbing for Mac to cancel an ongoing
composition on the browser side when the renderer has canceled its
ongoing composition.

BUG=578168,602723
==========

[commit-bot: I haz the power, 2016-08-10 21:38:49]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-10 21:40:45]

Description was changed from

==========
Cancel composition on RenderWidgetHostCocoaView after a child frame's RWHV
receives the corresponding IPC.

This CL will add the required plumbing for Mac to cancel an ongoing
composition on the browser side when the renderer has canceled its
ongoing composition.

BUG=578168,602723
==========

to

==========
Cancel composition on RenderWidgetHostCocoaView after a child frame's RWHV
receives the corresponding IPC.

This CL will add the required plumbing for Mac to cancel an ongoing
composition on the browser side when the renderer has canceled its
ongoing composition.

BUG=578168,602723

Committed: https://crrev.com/7006d2f9c3804eb9a6aedbb328223b22df51eacc
Cr-Commit-Position: refs/heads/master@{#411151}
==========

[commit-bot: I haz the power, 2016-08-10 21:40:47]

Patchset 2 (id:??) landed as
https://crrev.com/7006d2f9c3804eb9a6aedbb328223b22df51eacc
Cr-Commit-Position: refs/heads/master@{#411151}