Closed Bug 489389 Opened 11 years ago Closed 11 years ago

temporary image when doing zoomed tiling rendered too large [slow zoom scrolling]

Categories

(Core :: Graphics, defect)

defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: vlad, Assigned: roc)

References

Details

(Keywords: fixed1.9.1)

Attachments

(2 files)

Can't find a bug on this, but when we have a tiled image and we render when zoomed, we're creating a way too large temporary image in nsThebesImage::Draw .  We should at the very least be using the dirty rect to limit this.

For x/y repeat images we should be able to pad in just one direction and take advantage of native repeat.
Flags: blocking1.9.1+
the test page I saw that caused this problem was http://ajaxian.com/. Scroll up a couple of levels and scrolling becomes unusable.
OS: Mac OS X → All
Hardware: x86 → All
Assignee: vladimir → roc
Related to bug 480806?
Attached patch fixSplinter Review
Passes reftests on my machine. Most of the patch is just indentation changes and comments.
Attachment #375561 - Flags: review?(vladimir)
Whiteboard: [needs review]
Roc, could you also attach a whitespace-insensitive patch?
Comment on attachment 375561 [details] [diff] [review]
fix

I was close :)  Looks fine.
Attachment #375561 - Flags: review?(vladimir) → review+
Blocks: 491127
Whiteboard: [needs review] → [needs landing]
http://hg.mozilla.org/mozilla-central/rev/26c4c46fcaac
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [needs landing] → [needs 191 landing]
You need to log in before you can comment on or make changes to this bug.