Fix some tiled draw target bugs

RESOLVED FIXED in mozilla34

Status

()

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: mattwoodrow, Assigned: mattwoodrow)

Tracking

29 Branch
mozilla34
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
Created attachment 8479518 [details] [diff] [review]
tile-buffering-fixes

Just a few issues that cause us to fail reftests with tiling enabled.

We sometime end up with a backbuffer even when HasInternalBuffer is true because we're producing content faster than we can upload. Not discarding the back buffer helps for that, and we have TileExpiry to make sure it goes away 'soon'
Attachment #8479518 - Flags: review?(nical.bugzilla)
Comment on attachment 8479518 [details] [diff] [review]
tile-buffering-fixes

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

::: gfx/layers/client/TiledContentClient.cpp
@@ +1167,5 @@
>                              NS_roundf((dirtyRect->y - mSinglePaintBufferOffset.y) * mResolution),
>                              drawRect.width,
>                              drawRect.height);
>        gfx::IntPoint copyTarget(NS_roundf(drawRect.x), NS_roundf(drawRect.y));
>        // Mark the newly updated area as invalid in the front buffer

please update this comment too (front -> back)
Attachment #8479518 - Flags: review?(nical.bugzilla) → review+
https://hg.mozilla.org/mozilla-central/rev/fd00d686183f
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
You need to log in before you can comment on or make changes to this bug.