Title: [109739] trunk/Source/WebCore
- Revision
- 109739
- Author
- [email protected]
- Date
- 2012-03-05 06:54:54 -0800 (Mon, 05 Mar 2012)
Log Message
Improve comments in the tiled backing store https://bugs.webkit.org/show_bug.cgi?id=80279
Patch by Kenneth Rohde Christiansen and Alexander Færøy.
Reviewed by Simon Hausmann.
* platform/graphics/TiledBackingStore.cpp:
(WebCore::TiledBackingStore::createTiles):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (109738 => 109739)
--- trunk/Source/WebCore/ChangeLog 2012-03-05 14:26:00 UTC (rev 109738)
+++ trunk/Source/WebCore/ChangeLog 2012-03-05 14:54:54 UTC (rev 109739)
@@ -1,5 +1,17 @@
2012-03-05 Alexander Færøy <[email protected]>
+ Improve comments in the tiled backing store
+ https://bugs.webkit.org/show_bug.cgi?id=80279
+
+ Patch by Kenneth Rohde Christiansen and Alexander Færøy.
+
+ Reviewed by Simon Hausmann.
+
+ * platform/graphics/TiledBackingStore.cpp:
+ (WebCore::TiledBackingStore::createTiles):
+
+2012-03-05 Alexander Færøy <[email protected]>
+
TiledBackingStore.{h,cpp} contains unclear variable names
https://bugs.webkit.org/show_bug.cgi?id=80276
Modified: trunk/Source/WebCore/platform/graphics/TiledBackingStore.cpp (109738 => 109739)
--- trunk/Source/WebCore/platform/graphics/TiledBackingStore.cpp 2012-03-05 14:26:00 UTC (rev 109738)
+++ trunk/Source/WebCore/platform/graphics/TiledBackingStore.cpp 2012-03-05 14:54:54 UTC (rev 109739)
@@ -260,6 +260,9 @@
Vector<Tile::Coordinate> tilesToCreate;
unsigned requiredTileCount = 0;
+ // Cover areas (in tiles) with minimum distance from the visible rect. If the visible rect is
+ // not covered already it will be covered first in one go, due to the distance being 0 for tiles
+ // inside the visible rect.
Tile::Coordinate topLeft = tileCoordinateForPoint(coverRect.location());
Tile::Coordinate bottomRight = tileCoordinateForPoint(innerBottomRight(coverRect));
for (unsigned yCoordinate = topLeft.y(); yCoordinate <= bottomRight.y(); ++yCoordinate) {
@@ -268,7 +271,6 @@
if (tileAt(currentCoordinate))
continue;
++requiredTileCount;
- // Distance is 0 for all tiles inside the visibleRect.
double distance = tileDistance(visibleRect, currentCoordinate);
if (distance > shortestDistance)
continue;
@@ -288,7 +290,7 @@
}
requiredTileCount -= tilesToCreateCount;
- // Paint the content of the newly created tiles.
+ // Paint the content of the newly created tiles or resized tiles.
if (tilesToCreateCount || didResizeTiles)
updateTileBuffers();
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
