1011389 - [APZ] Click events are fired after a long press, even if the user has moved the finger

Status: RESOLVED FIXED

(Core :: Panning and Zooming, defect)

Description

[Cristian Rodriguez (:crdlc)]

See bug 1009729 comment 3, thx

Comment 1

[Kartikaya Gupta (email:[e-mail])]

Do you have STR that work on current gaia? Alternatively, if I write a gecko patch, can you verify that it's doing the right thing?

Comment 2

[Kartikaya Gupta (email:[e-mail])]

Attachment: Patch

Comment 3

[Kartikaya Gupta (email:[e-mail])]

Attachment: Patch

Slight update to the documentation as well.

And I tested this using the STR on bug 1009729, after reverting the gaia-side patch. Seems to work ok.

Comment 4

[Kartikaya Gupta (email:[e-mail])]

https://hg.mozilla.org/integration/mozilla-inbound/rev/3bcf99458c81

Comment 5

[Ryan VanderMeulen [:RyanVM]]

This patch had the misfortune of landing in the middle of a large bustage pileup on inbound that led to me having to revert to a last-good cset in lieu of waiting 3-4 hours for green instead. Unfortunately, that means this was backed out along with the others. If this was confirmed green on Try, it can re-land at your convenience.

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

Comment 6

[Kartikaya Gupta (email:[e-mail])]

https://tbpl.mozilla.org/?tree=Try&rev=046da541fd41

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

Comment 7

[Phil Ringnalda (:philor)]

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