1130256 - [CopyPaste] Prevent unwanted scrolling end event.

Status: RESOLVED FIXED

(Core :: DOM: Selection, defect, P2)

Description

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

Per bug 1105547 comment 15, there is 2 problem in existing code.

1. We should only send scroll start/end event only when carets are visible.
2. If we got blur event between scroll start and end. Scroll end event shouldn't be fired.

Comment 1

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

Attachment: Prevent unwanted scrolling event.

Comment 2

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

try https://treeherder.mozilla.org/#/jobs?repo=try&revision=b161cb78a4cf

Comment 3

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

new try with positive result: https://treeherder.mozilla.org/#/jobs?repo=try&revision=02d599d6ff56

Comment 4

[howie [:howie]]

blocking 2.2

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/d892c2a1b1db

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/d892c2a1b1db

Comment 7

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

Comment on attachment 8560259 [details] [diff] [review]
Prevent unwanted scrolling event.

NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings.

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
User impact if declined: Copy/Paste bubble would dispear/appear randomly when scrolling.
Testing completed: on master
Risk to taking this patch (and alternatives if risky): low risk
String or UUID changes made by this patch: none

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/f28591998c6c