Title: [101701] trunk/Source
- Revision
- 101701
- Author
- [email protected]
- Date
- 2011-12-01 13:32:11 -0800 (Thu, 01 Dec 2011)
Log Message
[Chromium] Use contentBounds instead of bounds for invalidation.
https://bugs.webkit.org/show_bug.cgi?id=73525
Patch by David Reveman <[email protected]> on 2011-12-01
Reviewed by James Robinson.
Use setNeedsDisplay() instead of setNeedsDisplayRect() when possible.
Source/WebCore:
No new tests.
* platform/graphics/chromium/LayerChromium.cpp:
(WebCore::LayerChromium::setBounds):
Source/WebKit/chromium:
* src/WebMediaPlayerClientImpl.cpp:
(WebKit::WebMediaPlayerClientImpl::repaint):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (101700 => 101701)
--- trunk/Source/WebCore/ChangeLog 2011-12-01 21:13:06 UTC (rev 101700)
+++ trunk/Source/WebCore/ChangeLog 2011-12-01 21:32:11 UTC (rev 101701)
@@ -1,3 +1,17 @@
+2011-12-01 David Reveman <[email protected]>
+
+ [Chromium] Use contentBounds instead of bounds for invalidation.
+ https://bugs.webkit.org/show_bug.cgi?id=73525
+
+ Reviewed by James Robinson.
+
+ Use setNeedsDisplay() instead of setNeedsDisplayRect() when possible.
+
+ No new tests.
+
+ * platform/graphics/chromium/LayerChromium.cpp:
+ (WebCore::LayerChromium::setBounds):
+
2011-12-01 Andreas Kling <[email protected]>
CSSStyleSelector: Add missing fields to constructor initializer list.
Modified: trunk/Source/WebCore/platform/graphics/chromium/LayerChromium.cpp (101700 => 101701)
--- trunk/Source/WebCore/platform/graphics/chromium/LayerChromium.cpp 2011-12-01 21:13:06 UTC (rev 101700)
+++ trunk/Source/WebCore/platform/graphics/chromium/LayerChromium.cpp 2011-12-01 21:32:11 UTC (rev 101701)
@@ -215,7 +215,7 @@
m_bounds = size;
if (firstResize || m_pageScaleDirty)
- setNeedsDisplayRect(FloatRect(0, 0, bounds().width(), bounds().height()));
+ setNeedsDisplay();
else
setNeedsCommit();
Modified: trunk/Source/WebKit/chromium/ChangeLog (101700 => 101701)
--- trunk/Source/WebKit/chromium/ChangeLog 2011-12-01 21:13:06 UTC (rev 101700)
+++ trunk/Source/WebKit/chromium/ChangeLog 2011-12-01 21:32:11 UTC (rev 101701)
@@ -1,3 +1,15 @@
+2011-12-01 David Reveman <[email protected]>
+
+ [Chromium] Use contentBounds instead of bounds for invalidation.
+ https://bugs.webkit.org/show_bug.cgi?id=73525
+
+ Reviewed by James Robinson.
+
+ Use setNeedsDisplay() instead of setNeedsDisplayRect() when possible.
+
+ * src/WebMediaPlayerClientImpl.cpp:
+ (WebKit::WebMediaPlayerClientImpl::repaint):
+
2011-12-01 Elliot Poger <[email protected]>
webkit->chromium DEPS roll 111575->112463
Modified: trunk/Source/WebKit/chromium/src/WebMediaPlayerClientImpl.cpp (101700 => 101701)
--- trunk/Source/WebKit/chromium/src/WebMediaPlayerClientImpl.cpp 2011-12-01 21:13:06 UTC (rev 101700)
+++ trunk/Source/WebKit/chromium/src/WebMediaPlayerClientImpl.cpp 2011-12-01 21:32:11 UTC (rev 101701)
@@ -148,7 +148,7 @@
ASSERT(m_mediaPlayer);
#if USE(ACCELERATED_COMPOSITING)
if (m_videoLayer && supportsAcceleratedRendering())
- m_videoLayer->setNeedsDisplayRect(IntRect(0, 0, m_videoLayer->bounds().width(), m_videoLayer->bounds().height()));
+ m_videoLayer->setNeedsDisplay();
#endif
m_mediaPlayer->repaint();
}
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
