[Webkit-unassigned] [Bug 73920] [Qt] Tiled backing store failing to update view under certain conditions

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=73920


Lamarque V. Souza <Lamarque.Souza at basyskom.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |Lamarque.Souza at basyskom.com




--- Comment #5 from Lamarque V. Souza <Lamarque.Souza at basyskom.com>  2012-10-30 14:00:01 PST ---
(In reply to comment #4)
> (From update of attachment 170646 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=170646&action=review
> 
> > Source/WebCore/page/FrameView.cpp:2026
> > +        frame()->tiledBackingStore()->removeAllTiles();
> 
> This feels quite wrong to drop all tiles just because of an issue edge tiles, the same tiles can be used between pages, and invalidate calls are responsible of triggering a refresh of the tile contents.
> There is a call to tiledBackingStoreContentsRect in TiledBackingStore::createTiles that should return a different contents size for the new page. A change of that rect should be responsible of triggering a call to resizeEdgeTiles. Please check around there to fix your issue.

I did some tests and WebKit calls resizeEdgeTiles() when loading the new page. resizeEdgeTiles() does not remove any tile, but TiledBackingStore::updateTileBuffers() detects some dirty tiles, but they are not being updated according to the new update when updateTileBuffers() calls dirtyTiles[n]->updateBackBuffer() for each of those dirty tiles.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.