Closed
Bug 872803
Opened 13 years ago
Closed 13 years ago
TScroll uses setInterval instead of requestAnimationFrame
Categories
(Testing :: Talos, defect)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 845943
People
(Reporter: mattwoodrow, Unassigned)
References
Details
Attachments
(1 file, 1 obsolete file)
|
1.58 KB,
patch
|
roc
:
review+
|
Details | Diff | Splinter Review |
This isn't tied to our painting rate, so we end up dropping frames.
Switching to requestAnimationFrame should guarantee that every frame of the scroll gets painted, and would more accurately measure how fast we can repaint scroll changes.
This will cause a regression in the numbers reported though, since we'll be painting more frames.
Attachment #750129 -
Flags: review?(roc)
Comment on attachment 750129 [details] [diff] [review]
Use requestAnimationFrame instead
Review of attachment 750129 [details] [diff] [review]:
-----------------------------------------------------------------
Notify dev.tree-management and dev.platform when this lands.
Attachment #750129 -
Flags: review?(roc) → review+
|
Reporter | |
Comment 2•13 years ago
|
||
https://hg.mozilla.org/build/talos/rev/7583b34ca6b1
create_talos_zip.py doesn't work because 'http://pyyaml.org/export/360/pyyaml/trunk/lib/yaml/composer.py' appears to be down.
|
|
Reporter | |
Comment 3•13 years ago
|
||
Attachment #750164 -
Flags: review?(jmaher)
|
|
Reporter | |
Comment 4•13 years ago
|
||
This is a duplicate of bug 845943, which fixes it for all scroll sub-tests (not just the tiled* ones).
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → DUPLICATE
|
|
Reporter | |
Updated•13 years ago
|
Attachment #750164 -
Attachment is obsolete: true
Attachment #750164 -
Flags: review?(jmaher)
You need to log in
before you can comment on or make changes to this bug.
Description
•