Last Comment Bug 718914 - Compress input events
: Compress input events
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Widget (show other bugs)
: Trunk
: ARM Gonk (Firefox OS)
: -- normal (vote)
: ---
Assigned To: Andreas Gal :gal
:
Mentors:
Depends on: 712973
Blocks: 718913 725773
  Show dependency treegraph
 
Reported: 2012-01-17 19:26 PST by Chris Jones [:cjones] inactive; ni?/f?/r? if you need me
Modified: 2012-02-09 12:17 PST (History)
2 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
patch (6.84 KB, patch)
2012-02-07 22:26 PST, Andreas Gal :gal
no flags Details | Diff | Splinter Review
patch (534 bytes, patch)
2012-02-07 22:32 PST, Andreas Gal :gal
no flags Details | Diff | Splinter Review
working patch (558 bytes, patch)
2012-02-07 22:40 PST, Andreas Gal :gal
cjones.bugs: review+
Details | Diff | Splinter Review

Description Chris Jones [:cjones] inactive; ni?/f?/r? if you need me 2012-01-17 19:26:54 PST
We're not compressing input events in the gonk widget backend, which causes the IME in bug 718913 to lag behind user input for a second or two after panning around the character-selection screen really quickly.

There's likely some other slowness we need to fix there, but staying permanently behind current user input by a few seconds isn't helping perceived perf.

ISTR InputReader or something else in that little group of classes did some compression already, so we might get this for free.
Comment 1 Chris Jones [:cjones] inactive; ni?/f?/r? if you need me 2012-02-07 10:23:28 PST
mwu, are you going to take this?
Comment 2 Andreas Gal :gal 2012-02-07 22:26:16 PST
Created attachment 595322 [details] [diff] [review]
patch
Comment 3 Andreas Gal :gal 2012-02-07 22:32:47 PST
Created attachment 595323 [details] [diff] [review]
patch
Comment 4 Andreas Gal :gal 2012-02-07 22:40:08 PST
Created attachment 595324 [details] [diff] [review]
working patch
Comment 5 Andreas Gal :gal 2012-02-07 22:44:23 PST
http://hg.mozilla.org/mozilla-central/rev/4b9608fd670c

Note You need to log in before you can comment on or make changes to this bug.