Prevent propagation of wheel events to dispatch event in parent iframe.

Prevent propagation of wheel events to dispatch event in parent iframe.

If the wheel event was unhandled in the iframe prevent it from being
dispatched in the parent's DOM. Do the parent's scroll action if necesary.
This code can be cleaned up when we enable wheel gestures by default
everywhere since it can just be an early return.

BUG=589406, 564042
Committed: https://crrev.com/4c40c8928c3e27c4fb09ac945b749669d3dbf8fc
Cr-Commit-Position: refs/heads/master@{#378243}

[dtapuska, 2016-02-25 14:45:32]

Description was changed from

==========
Prevent propagation of wheel events to dispatch event in parent iframe.

If the wheel event was unhandled in the iframe prevent it from being
dispatched in the parent's DOM. Do the parent's scroll action if necesary.
This code can be cleaned up when we enable wheel gestures by default
everywhere since it can just be an early return.

BUG=589406, 564042
==========

to

==========
Prevent propagation of wheel events to dispatch event in parent iframe.

If the wheel event was unhandled in the iframe prevent it from being
dispatched in the parent's DOM. Do the parent's scroll action if necesary.
This code can be cleaned up when we enable wheel gestures by default
everywhere since it can just be an early return.

BUG=589406, 564042
==========

[dtapuska, 2016-02-25 14:45:32]

dtapuska@chromium.org changed reviewers:
+ rbyers@chromium.org

[Rick Byers, 2016-02-25 16:34:16]

LGTM with nits

https://codereview.chromium.org/1731353003/diff/40001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/fast/events/iframe-mousewheel.html (right):

https://codereview.chromium.org/1731353003/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/events/iframe-mousewheel.html:13:
document.addEventListener("mousewheel", function() {
nit: new tests should test the standard 'wheel' event (just in case there's some
difference between the legacy 'mousewheel' behavior, though that seems very
unlikely).

https://codereview.chromium.org/1731353003/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/input/EventHandler.cpp (right):

https://codereview.chromium.org/1731353003/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/input/EventHandler.cpp:1809: // been enabled
everywhere; as we can just return early.
nit: reference the crbug link for that incase anyone reading this wants more
information.

[dtapuska, 2016-02-25 22:31:53]

The CQ bit was checked by dtapuska@chromium.org

[dtapuska, 2016-02-25 22:31:54]

The patchset sent to the CQ was uploaded after l-g-t-m from rbyers@chromium.org
Link to the patchset: https://codereview.chromium.org/1731353003/#ps60001
(title: "Fix nits")

[commit-bot: I haz the power, 2016-02-25 22:33:47]

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

[commit-bot: I haz the power, 2016-02-25 23:28:47]

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

[commit-bot: I haz the power, 2016-02-25 23:28:48]

Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  win8_chromium_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_ng/...)

[dtapuska, 2016-02-26 16:46:18]

On 2016/02/25 23:28:48, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   linux_chromium_chromeos_ozone_rel_ng on tryserver.chromium.linux
(JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
>   win8_chromium_ng on tryserver.chromium.win (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_ng/...)

Rick can you have a quick look at this? I apparently broke out of process
plugins in the past revision. The change has been modified (and inherently
cleaned up)

[Rick Byers, 2016-02-26 16:59:50]

Nice cleanup, thanks! 
LGTM

[dtapuska, 2016-02-26 20:09:48]

The CQ bit was checked by dtapuska@chromium.org

[dtapuska, 2016-02-26 20:09:49]

The patchset sent to the CQ was uploaded after l-g-t-m from rbyers@chromium.org
Link to the patchset: https://codereview.chromium.org/1731353003/#ps100001
(title: "Fix mac")

[commit-bot: I haz the power, 2016-02-26 20:11:58]

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

[commit-bot: I haz the power, 2016-02-26 20:52:20]

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

[commit-bot: I haz the power, 2016-02-26 20:52:21]

Try jobs failed on following builders:
  android_chromium_gn_compile_rel on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)

[dtapuska, 2016-02-29 14:49:38]

The CQ bit was checked by dtapuska@chromium.org

[commit-bot: I haz the power, 2016-02-29 14:49:48]

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

[commit-bot: I haz the power, 2016-02-29 14:52:36]

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

[commit-bot: I haz the power, 2016-02-29 14:52:37]

Try jobs failed on following builders:
  linux_chromium_compile_dbg_32_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[dtapuska, 2016-02-29 15:15:44]

The CQ bit was checked by dtapuska@chromium.org

[dtapuska, 2016-02-29 15:15:44]

The patchset sent to the CQ was uploaded after l-g-t-m from rbyers@chromium.org
Link to the patchset: https://codereview.chromium.org/1731353003/#ps120001
(title: "Rebase")

[commit-bot: I haz the power, 2016-02-29 15:15:57]

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

[commit-bot: I haz the power, 2016-02-29 17:06:06]

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

[commit-bot: I haz the power, 2016-02-29 17:06:08]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[dtapuska, 2016-02-29 17:48:27]

The CQ bit was checked by dtapuska@chromium.org

[commit-bot: I haz the power, 2016-02-29 17:48:43]

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

[commit-bot: I haz the power, 2016-02-29 18:58:50]

Description was changed from

==========
Prevent propagation of wheel events to dispatch event in parent iframe.

If the wheel event was unhandled in the iframe prevent it from being
dispatched in the parent's DOM. Do the parent's scroll action if necesary.
This code can be cleaned up when we enable wheel gestures by default
everywhere since it can just be an early return.

BUG=589406, 564042
==========

to

==========
Prevent propagation of wheel events to dispatch event in parent iframe.

If the wheel event was unhandled in the iframe prevent it from being
dispatched in the parent's DOM. Do the parent's scroll action if necesary.
This code can be cleaned up when we enable wheel gestures by default
everywhere since it can just be an early return.

BUG=589406, 564042
==========

[commit-bot: I haz the power, 2016-02-29 18:58:52]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2016-02-29 19:00:22]

Description was changed from

==========
Prevent propagation of wheel events to dispatch event in parent iframe.

If the wheel event was unhandled in the iframe prevent it from being
dispatched in the parent's DOM. Do the parent's scroll action if necesary.
This code can be cleaned up when we enable wheel gestures by default
everywhere since it can just be an early return.

BUG=589406, 564042
==========

to

==========
Prevent propagation of wheel events to dispatch event in parent iframe.

If the wheel event was unhandled in the iframe prevent it from being
dispatched in the parent's DOM. Do the parent's scroll action if necesary.
This code can be cleaned up when we enable wheel gestures by default
everywhere since it can just be an early return.

BUG=589406, 564042

Committed: https://crrev.com/4c40c8928c3e27c4fb09ac945b749669d3dbf8fc
Cr-Commit-Position: refs/heads/master@{#378243}
==========

[commit-bot: I haz the power, 2016-02-29 19:00:23]

Patchset 7 (id:??) landed as
https://crrev.com/4c40c8928c3e27c4fb09ac945b749669d3dbf8fc
Cr-Commit-Position: refs/heads/master@{#378243}