Summary: | Coordinated Graphics: CoordinatedTile::updateBackBuffer() returns dirty rects. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Dongseong Hwang <dongseong.hwang> | ||||||
Component: | Layout and Rendering | Assignee: | Dongseong Hwang <dongseong.hwang> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | kenneth, noam, webkit.review.bot, zeno | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 105787 | ||||||||
Attachments: |
|
Description
Dongseong Hwang
2013-01-17 16:09:00 PST
Created attachment 183304 [details]
Patch
Comment on attachment 183304 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=183304&action=review > Source/WebKit2/WebProcess/WebPage/CoordinatedGraphics/CoordinatedTile.cpp:103 > + Vector<IntRect> updatedRects; > + updatedRects.append(m_dirtyRect); > m_dirtyRect = IntRect(); > - return Vector<IntRect>(); > + return updatedRects; Why are we returning a vector? isnt m_dirtyRect not just one rect? Comment on attachment 183304 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=183304&action=review >> Source/WebKit2/WebProcess/WebPage/CoordinatedGraphics/CoordinatedTile.cpp:103 >> + return updatedRects; > > Why are we returning a vector? isnt m_dirtyRect not just one rect? CoordinatedTile::updateBackBuffer() overrides Tile::updateBackBuffer(), and Tile::updateBackBuffer() returns vector. We can refactor it later :) Comment on attachment 183304 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=183304&action=review > Source/WebKit2/ChangeLog:9 > + This patch makes CoordinatedTile::updateBackBuffer() return correct dirty rects. rect* - you could refrase Created attachment 183412 [details]
Patch for land
Comment on attachment 183412 [details] Patch for land Clearing flags on attachment: 183412 Committed r140118: <http://trac.webkit.org/changeset/140118> All reviewed patches have been landed. Closing bug. |