Bug 78249

Summary: [Chromium] TiledLayerChromium::protectVisibleTileTextures() should only protect the visible textures
Product: WebKit Reporter: Xianzhu Wang <wangxianzhu>
Component: Layout and RenderingAssignee: Xianzhu Wang <wangxianzhu>
Status: RESOLVED FIXED    
Severity: Normal CC: cc-bugs, enne, jamesr, vangelis, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
patch none

Description Xianzhu Wang 2012-02-09 10:26:30 PST
For now TiledLayerChromium::protectVisibleTileTextures() protects the whole layer:
    protectTileTextures(IntRect(IntPoint::zero(), contentBounds()));

Should this be changed to protectTileTextures(visibleLayerRect())?
Comment 1 Adrienne Walker 2012-02-09 10:57:58 PST
I think you're right.  I'm not sure why it wasn't this way to begin with.
Comment 2 Xianzhu Wang 2012-02-09 12:02:31 PST
Created attachment 126343 [details]
patch
Comment 3 WebKit Review Bot 2012-02-09 21:03:10 PST
Comment on attachment 126343 [details]
patch

Clearing flags on attachment: 126343

Committed r107352: <http://trac.webkit.org/changeset/107352>
Comment 4 WebKit Review Bot 2012-02-09 21:03:13 PST
All reviewed patches have been landed.  Closing bug.