Last Comment Bug 624449 - Don't set displayport to cover never-to-be-defined pixels
: Don't set displayport to cover never-to-be-defined pixels
Status: RESOLVED DUPLICATE of bug 624451
:
Product: Fennec Graveyard
Classification: Graveyard
Component: Panning/Zooming (show other bugs)
: Trunk
: All All
: -- normal (vote)
: ---
Assigned To: Benjamin Stover (:stechz)
:
Mentors:
Depends on:
Blocks: 624444
  Show dependency treegraph
 
Reported: 2011-01-10 10:56 PST by Chris Jones [:cjones] inactive; ni?/f?/r? if you need me
Modified: 2011-01-18 14:51 PST (History)
3 users (show)
See Also:
QA Whiteboard:
Iteration: ---
Points: ---


Attachments

Description Chris Jones [:cjones] inactive; ni?/f?/r? if you need me 2011-01-10 10:56:55 PST
In the current code (last time I looked anyway), if you load a long page like http://www.timecube.com and then pan down, the displayport starts out covering pixels above the top of the document and to the left.  As you pan down, the size of rendered area increases as the displayport moves into defined pixels.  A similar effect happens when panning horizontally.

Not using available displayport area in this manner is bad for several reasons
 - (obviously) checkerboard earlier than should
 - resizing gfx buffers is rather expensive
 - happens on every page load, not in edge cases

We shouldn't ever set the displayport above or to the left of document <0, 0>.
Comment 1 Matt Brubeck (:mbrubeck) 2011-01-18 14:36:23 PST
The WIP patch in bug 624451 fixes this.
Comment 2 Chris Jones [:cjones] inactive; ni?/f?/r? if you need me 2011-01-18 14:48:33 PST
Sorry, same here, and stechz is working on it already.
Comment 3 Mark Finkle (:mfinkle) (use needinfo?) 2011-01-18 14:50:23 PST
One bug is enough

*** This bug has been marked as a duplicate of bug 624451 ***

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