Closed Bug 587697 Opened 9 years ago Closed 9 years ago
Sharing context menu appears when panning
When I start panning, and I begin on a link, I often get the "Share Link" menu unintentionally. I think it'd be best if this timer lived in the chrome side, since the cancel message may not get there as fast as we'd like. Basically, immediately on link tap we'd send a message with all the data we are right now. On chrome side, we'd start the timer and if panning starts we cancel the timer right there.
Summary: Long taps often misfire → Sharing context menu appears when panning
This does what is suggested. I'm having trouble reliably reproducing this right now though, so its tough to test.
Comment on attachment 469598 [details] [diff] [review] Move timer to chrome process > case "Browser:MouseDown": > this._overlayTimeout.clear(); >- this._overlayTimeout.clear(); > > let element = elementFromPoint(x, y); Maybe this was the problem. The removed line should be: this._contextTimeout.clear(); Stover - how easy can you repro this?
Well this is a boring patch...
Comment on attachment 470104 [details] [diff] [review] Change one line LOL!
Attachment #470104 - Flags: review?(mark.finkle) → review+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
verified FIXED on builds: Mozilla/5.0 (maemol Linux armv7l; rv:2.0b7pre) Gecko/20101029 Firefox/4.0b8pre Fennec/4.0b2 Mozilla/5.0 (android Linux armv7l; rv:2.0b7pre) Gecko/20101029 Firefox/4.0b8pre Fennec/4.0b2
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.