711071 - Fix velocity calculation

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect, P3)

Description

[Lawrence Mandel [:lmandel] (use needinfo)]

Attachment: catlog while being unable to scroll

Build:20111214
Device: Samsung Galaxy S II

Occasionally when I open the browser to a page I cannot scroll at all. Firefox does not respond to any panning/scrolling touch events. Note that at this time I can still click buttons, links, and the awesome bar. I can also zoom. Restarting the browser allows me to pan/scroll again.

Unfortunately I have not been able to determine a reliable path to reproduce the issue. I was able to catch a catlog with some interesting messages that I hope will shed some light on the source of this issue.

12-15 10:36:30.900: I/GeckoSoftwareLayerClient(10290): Adjusting viewport
12-15 10:36:31.130: I/InputReader(2715): dispatchTouch::touch event's action is 0, pending(waiting finished signal)=0
12-15 10:36:31.140: I/InputDispatcher(2715): Delivering touch to current input target: action: 0, channel '40767260 org.mozilla.fennec/org.mozilla.fennec.App (server)'
12-15 10:36:31.145: E/lights(2715): write_int: path /sys/devices/virtual/misc/melfas_touchkey/brightness, value 1
12-15 10:36:31.145: I/PowerManagerService(2715): Ulight 3->7|0
12-15 10:36:31.145: D/PowerManagerService(2715): setLightBrightness : mButtonLight : 69
12-15 10:36:31.175: E/GeckoPanZoomController(10290): Received impossible touch move while in NOTHING
12-15 10:36:31.185: E/GeckoPanZoomController(10290): Received impossible touch move while in NOTHING
12-15 10:36:31.230: E/GeckoPanZoomController(10290): Received impossible touch move while in NOTHING
...

Comment 1

[Aaron Train [:aaronmt]]

recent regression? dupe of bug 710694?

Comment 2

[Lawrence Mandel [:lmandel] (use needinfo)]

The video in bug 710694 shows different behaviour than what I'm seeing here. There is no jumping or random resizing as shown. For me, the page loads fine but is stuck from a pan/scroll position. I can zoom in or out as expected but still can't move the page up/down/left/right.

Comment 3

[Wesley Johnston (:wesj)]

Attachment: Patch

This fixes some issues with the fling controller. I'm also sometimes seeing finishAnimation called over and over and over. Apparently we're killing the timertask but not cancelling? Still digging on that bit.

Looking into this because I'm running into issues while working on the touchevents stuff, and want to see if there faults with those patches or something else.

Comment 4

[Doug Turner (:dougt)]

is your patch needed?  has this problem been addressed already.

Comment 5

[Wesley Johnston (:wesj)]

Attachment: Patch

Not fixed yet.

Comment 6

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

(In reply to Wesley Johnston (:wesj) from comment #3)
> This fixes some issues with the fling controller. I'm also sometimes seeing
> finishAnimation called over and over and over. Apparently we're killing the
> timertask but not cancelling? Still digging on that bit.

If you figure out anything to do with this, let me know. I have bug 713011 which is tracking this issue.

Comment 7

[Mark Finkle (:mfinkle) (use needinfo?)]

Changed bug into something appropriate for the patch. Bug 713011 will handle the panning issue

Comment 8

[Patrick Walton (:pcwalton)]

Comment on attachment 591226 [details] [diff] [review]
Patch

Review of attachment 591226 [details] [diff] [review]:
-----------------------------------------------------------------

r=me

Comment 9

[Wesley Johnston (:wesj)]

http://hg.mozilla.org/integration/mozilla-inbound/rev/f8282085e225

Comment 10

[Wesley Johnston (:wesj)]

Comment on attachment 591226 [details] [diff] [review]
Patch

Nom for aurora. Fixes a bug in how we calculate the velocity. Can lead to issues with panning. Mobile only.

Comment 11

[Joe Drew (not getting mail)]

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

Comment 12

[Brad Lassey [:blassey] (use needinfo?)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/02093039f454