Adds the ability for WebContentsDelegate to decide if event should be updated

Adds the ability for WebContentsDelegate to decide if event should be updated

Currently the keyboard processing code for aura always marks the event
as handled. This means no other EventHandlers see the event if a page
has focus. For mus+ash we need to be able to control whether the
event should be marked handled or not. This patch changes
PreHandleKeyboardEvent() to return an enum indicating whether the
event should be marked handled or not.

BUG=701815
TEST=covered by tests
R=jam@chromium.org, sadrul@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2775553002
Cr-Commit-Position: refs/heads/master@{#459341}
Committed: https://chromium.googlesource.com/chromium/src/+/f65d9bb4555976b047385b392d40fc62e4df9f27

[sky, 2017-03-23 05:05:02]

Description was changed from

==========
Adds the ability for WebContentsDelegate to decide if event should be updated

Currently the keyboard processing code for aura always marks the event
as handled. This means no other EventHandlers see the event if a page
has focus. For mus+ash we need to be able to control whether the
event should be marked handled or not. This patch changes
PreHandleKeyboardEvent() to return an enum indicating whether the
event should be marked handled or not.

BUG=701815
TEST=covered by tests
R=jam@chromium.org, sadrul@chromium.org
==========

to

==========
Adds the ability for WebContentsDelegate to decide if event should be updated

Currently the keyboard processing code for aura always marks the event
as handled. This means no other EventHandlers see the event if a page
has focus. For mus+ash we need to be able to control whether the
event should be marked handled or not. This patch changes
PreHandleKeyboardEvent() to return an enum indicating whether the
event should be marked handled or not.

BUG=701815
TEST=covered by tests
R=jam@chromium.org, sadrul@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[sky, 2017-03-23 05:11:47]

sadrul: content-aura parts and chrome
jam: content non-aura parts

[sky, 2017-03-23 05:11:55]

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

[commit-bot: I haz the power, 2017-03-23 05:12:06]

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, 2017-03-23 05:22:02]

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

[commit-bot: I haz the power, 2017-03-23 05:22:02]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[sky, 2017-03-23 16:15:45]

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

[commit-bot: I haz the power, 2017-03-23 16:16: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, 2017-03-23 16:28:48]

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

[commit-bot: I haz the power, 2017-03-23 16:28:48]

Dry run: Try jobs failed on following builders:
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[sky, 2017-03-23 16:40:27]

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

[commit-bot: I haz the power, 2017-03-23 16:40:54]

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, 2017-03-23 17:14:52]

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

[commit-bot: I haz the power, 2017-03-23 17:14:54]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[sky, 2017-03-23 17:32:08]

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

[commit-bot: I haz the power, 2017-03-23 17:33:11]

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, 2017-03-23 18:30:37]

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

[commit-bot: I haz the power, 2017-03-23 18:30:37]

Dry run: This issue passed the CQ dry run.

[jam, 2017-03-23 23:01:47]

lgtm

[sadrul, 2017-03-24 00:17:02]

lgtm

[sky, 2017-03-24 02:21:08]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2017-03-24 02:21:32]

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, 2017-03-24 02:26:43]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1490322068866830,
"parent_rev": "5f783c1c0658b2bb3732d29d60f5fdce36e3bf45", "commit_rev":
"f65d9bb4555976b047385b392d40fc62e4df9f27"}

[commit-bot: I haz the power, 2017-03-24 02:27:35]

Description was changed from

==========
Adds the ability for WebContentsDelegate to decide if event should be updated

Currently the keyboard processing code for aura always marks the event
as handled. This means no other EventHandlers see the event if a page
has focus. For mus+ash we need to be able to control whether the
event should be marked handled or not. This patch changes
PreHandleKeyboardEvent() to return an enum indicating whether the
event should be marked handled or not.

BUG=701815
TEST=covered by tests
R=jam@chromium.org, sadrul@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Adds the ability for WebContentsDelegate to decide if event should be updated

Currently the keyboard processing code for aura always marks the event
as handled. This means no other EventHandlers see the event if a page
has focus. For mus+ash we need to be able to control whether the
event should be marked handled or not. This patch changes
PreHandleKeyboardEvent() to return an enum indicating whether the
event should be marked handled or not.

BUG=701815
TEST=covered by tests
R=jam@chromium.org, sadrul@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2775553002
Cr-Commit-Position: refs/heads/master@{#459341}
Committed:
https://chromium.googlesource.com/chromium/src/+/f65d9bb4555976b047385b392d40...
==========

[commit-bot: I haz the power, 2017-03-24 02:27:36]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/f65d9bb4555976b047385b392d40...