Closed Bug 729534 Opened 9 years ago Closed 9 years ago

maple: Buffer rotation appears broken on engadget

Categories

(Core :: Graphics, defect, P1)

x86
macOS
defect

Tracking

()

VERIFIED FIXED
mozilla14
Tracking Status
firefox14 --- verified
firefox15 --- verified
blocking-fennec1.0 --- beta+

People

(Reporter: jrmuizel, Assigned: cwiiis)

References

Details

(Whiteboard: MAPLE mwc-demo)

Attachments

(1 file, 1 obsolete file)

Chris Lord says:
Buffer rotation doesn't appear to be working. At least on Engadget, I'm
constantly seeing huge uploads when scrolling a small amount.
Blocks: 729391
Assignee: nobody → chrislord.net
Whiteboard: maple → MAPLE mwc-demo
Priority: -- → P1
Just to note, it appears broken on pretty much every site to a certain degree, but Engadget is particularly badly affected.
blocking-fennec1.0: --- → beta+
Status: NEW → ASSIGNED
roc debugged this with me yesterday, and we(...he) discovered that buffer rotation isn't being used effectively on Engadget because we don't seem to be retaining layers as we scroll down the page.

This is because the frame used to retain a layer when there are multiple children of a scroll layer is the last one, which has a tendency to change. This patch makes it use the first child instead, which tends to be more stable.

The proper fix would be to associate it with a frame that definitely won't change, but this is a low-risk quick fix that I imagine helps in the majority of cases (and certainly fixes the engadget.com case).

This has a dramatic effect on performance for Maple.
Attachment #601563 - Flags: review?(roc)
For reference, a try run looks good: https://tbpl.mozilla.org/?tree=Try&rev=bf681c51d65e
Comment on attachment 601563 [details] [diff] [review]
Fix retaining layers when the displaylist children of a scroll layer change

Changing reviewer to mats, given that roc was the one that came up with the fix.
Attachment #601563 - Flags: review?(roc) → review?(matspal)
Comment on attachment 601563 [details] [diff] [review]
Fix retaining layers when the displaylist children of a scroll layer change

Seems a bit random, but if it works better, sure.  r=mats
Add a XXX comment and file a bug on making a proper fix?
Attachment #601563 - Flags: review?(matspal) → review+
Attaching final committed patch for reference.
Attachment #601563 - Attachment is obsolete: true
Attachment #601629 - Flags: review+
https://hg.mozilla.org/projects/maple/rev/d137bbd5fe5b

Chris, make sure you file a proper fix and link it from this bug.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
The proper fix is filed as bug 731641 (and the comment checked in mentions this also).
Target Milestone: --- → mozilla14
Verified fixed on:

Firefox 15.0a1 (2012-05-29)
Firefox 14.0a2 (2012-05-29)

Device: Galaxy Nexus
OS: Android 4.0.2
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.