724001 - Performance of touch events is worse than stock browser

Status: RESOLVED WORKSFORME

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

Description

[Martijn Wargers (dead)]

See url testcase, when I try to make a circle in the canvas by drawing a circle motion, then that works fine in stock browser, but on current native Fennec trunk, I get more something like a hexagon.
It seems that Fennec really see that many touch move events as stock browser.

Comment 1

[Wesley Johnston (:wesj)]

I have a feeling this has more to do with other factors outside the touch events code, but we need some profiling to find out.

Comment 2

[Doug Turner (:dougt)]

martijn, can you try the same demo on Maple and see if it is better or not?

Comment 3

[Martijn Wargers (dead)]

It seems that Maple fixes this (or mostly). I tested on the Samsung Galaxy Nexus and the HTC Desire HD. Compared it with the Aurora branch.

Comment 4

[Wesley Johnston (:wesj)]

if we're fine with just saying "fixed in Maple" I can live with that. I was hoping to rewrite the testcase so that we didn't attempt to draw anything until touchend to make sure it wasn't painting that was affecting this. I have a feeling that we are coalescing touch and paint events here:

http://mxr.mozilla.org/mozilla-central/source/widget/android/nsAppShell.cpp#269

which kills us on things like this.

Comment 5

[Doug Turner (:dougt)]

wes, please nom if you think this should block fennec native.

Comment 6

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

There have been many speedups done to touch events. I just tested this again on the GN and it seems pretty smooth. Going to mark as WFM.