Title: [118840] trunk/Source/WebCore
- Revision
- 118840
- Author
- [email protected]
- Date
- 2012-05-29 15:24:47 -0700 (Tue, 29 May 2012)
Log Message
Fix transparent iframes with composited contents in WK1 on Mac
https://bugs.webkit.org/show_bug.cgi?id=76478
Reviewed by Simon Fraser.
Test: compositing/iframes/repaint-after-losing-scrollbars.html
* page/FrameView.cpp:
(WebCore::FrameView::useSlowRepaints):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (118839 => 118840)
--- trunk/Source/WebCore/ChangeLog 2012-05-29 22:22:42 UTC (rev 118839)
+++ trunk/Source/WebCore/ChangeLog 2012-05-29 22:24:47 UTC (rev 118840)
@@ -1,3 +1,15 @@
+2012-05-29 Adrienne Walker <[email protected]>
+
+ Fix transparent iframes with composited contents in WK1 on Mac
+ https://bugs.webkit.org/show_bug.cgi?id=76478
+
+ Reviewed by Simon Fraser.
+
+ Test: compositing/iframes/repaint-after-losing-scrollbars.html
+
+ * page/FrameView.cpp:
+ (WebCore::FrameView::useSlowRepaints):
+
2012-05-29 David Hyatt <[email protected]>
https://bugs.webkit.org/show_bug.cgi?id=87787
Modified: trunk/Source/WebCore/page/FrameView.cpp (118839 => 118840)
--- trunk/Source/WebCore/page/FrameView.cpp 2012-05-29 22:22:42 UTC (rev 118839)
+++ trunk/Source/WebCore/page/FrameView.cpp 2012-05-29 22:24:47 UTC (rev 118840)
@@ -1259,7 +1259,11 @@
{
bool mustBeSlow = m_slowRepaintObjectCount > 0 || (platformWidget() && m_fixedObjectCount > 0);
- if (contentsInCompositedLayer())
+ // FIXME: WidgetMac.mm makes the assumption that useSlowRepaints ==
+ // m_contentIsOpaque, so don't take the fast path for composited layers
+ // if they are a platform widget in order to get painting correctness
+ // for transparent layers. See the comment in WidgetMac::paint.
+ if (contentsInCompositedLayer() && !platformWidget())
return mustBeSlow;
#if PLATFORM(CHROMIUM)
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
