Fix runtime setting forcing passive event listeners to scroll blocking events.

Fix runtime setting forcing passive event listeners to scroll blocking events.

Previously the runtime settings would apply to all events; make it
restricted to scroll blocking events.

Also adjust the blockedEventsWarning threshold to use this same event
type (it was previously reporting touchend as well; which it shouldn't
have)

BUG=599611
Committed: https://crrev.com/b38d8cc6524ed76a371abb851f97e534aab04926
Cr-Commit-Position: refs/heads/master@{#396855}

[dtapuska, 2016-05-31 13:59:01]

Description was changed from

==========
Fix runtime setting forcing passive event listeners to scroll blocking events.

Previously the runtime settings would apply to all events; make it
restricted to scroll blocking events.

Also adjust the blockedEventsWarning threshold to use this same event
type (it was previously reporting touchend as well; which it shouldn't
have)

BUG=599611
==========

to

==========
Fix runtime setting forcing passive event listeners to scroll blocking events.

Previously the runtime settings would apply to all events; make it
restricted to scroll blocking events.

Also adjust the blockedEventsWarning threshold to use this same event
type (it was previously reporting touchend as well; which it shouldn't
have)

BUG=599611
==========

[dtapuska, 2016-05-31 13:59:01]

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

[dtapuska, 2016-05-31 13:59:41]

Sorry this was a brain fart. I definitely tested that I was assigning all event
handlers this fashion but clearly it is far better that it should only be the
scroll blocking ones.

[Rick Byers, 2016-05-31 14:03:51]

LGTM

No worries, I also can't believe I missed this.  When looking at the AMP case I
could have sworn I tested this - was racking my brain trying to think why the
behavior WASN'T just preventDefault on touchend getting ignored.

Please re-open the bug and request merge to 52 - I think it's important to avoid
confusion.

[Rick Byers, 2016-05-31 14:04:28]

On 2016/05/31 14:03:51, Rick Byers wrote:
> LGTM
> 
> No worries, I also can't believe I missed this.  When looking at the AMP case
I
> could have sworn I tested this - was racking my brain trying to think why the
> behavior WASN'T just preventDefault on touchend getting ignored.
> 
> Please re-open the bug and request merge to 52 - I think it's important to
avoid
> confusion.

It's convenient that the about:flags description is already correct - too late
for any string changes ;-)

[dtapuska, 2016-05-31 17:33:55]

The CQ bit was checked by dtapuska@chromium.org

[dtapuska, 2016-05-31 17:33:55]

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/2024913002/#ps20001
(title: "Fix layout test")

[commit-bot: I haz the power, 2016-05-31 17:34:13]

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

[commit-bot: I haz the power, 2016-05-31 17:38:24]

Description was changed from

==========
Fix runtime setting forcing passive event listeners to scroll blocking events.

Previously the runtime settings would apply to all events; make it
restricted to scroll blocking events.

Also adjust the blockedEventsWarning threshold to use this same event
type (it was previously reporting touchend as well; which it shouldn't
have)

BUG=599611
==========

to

==========
Fix runtime setting forcing passive event listeners to scroll blocking events.

Previously the runtime settings would apply to all events; make it
restricted to scroll blocking events.

Also adjust the blockedEventsWarning threshold to use this same event
type (it was previously reporting touchend as well; which it shouldn't
have)

BUG=599611
==========

[commit-bot: I haz the power, 2016-05-31 17:38:31]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-31 17:39:40]

Description was changed from

==========
Fix runtime setting forcing passive event listeners to scroll blocking events.

Previously the runtime settings would apply to all events; make it
restricted to scroll blocking events.

Also adjust the blockedEventsWarning threshold to use this same event
type (it was previously reporting touchend as well; which it shouldn't
have)

BUG=599611
==========

to

==========
Fix runtime setting forcing passive event listeners to scroll blocking events.

Previously the runtime settings would apply to all events; make it
restricted to scroll blocking events.

Also adjust the blockedEventsWarning threshold to use this same event
type (it was previously reporting touchend as well; which it shouldn't
have)

BUG=599611

Committed: https://crrev.com/b38d8cc6524ed76a371abb851f97e534aab04926
Cr-Commit-Position: refs/heads/master@{#396855}
==========

[commit-bot: I haz the power, 2016-05-31 17:39:41]

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