Changeset 58222 in webkit

Timestamp:

Apr 24, 2010, 4:59:37 PM (15 years ago)

Author:

mitz@apple.com

Message:

Don't propagate compositing out of iframes on Mac
​https://bugs.webkit.org/show_bug.cgi?id=38072

Patch by Simon Fraser <Simon Fraser> on 2010-04-24
Reviewed by Dan Bernstein.

Propagating compositing out of iframes caused too many regressions on Mac,
so only do it for other platforms that may need to hook compositing layers
together across iframe boundaries.

• rendering/RenderLayerCompositor.cpp:

(WebCore::RenderLayerCompositor::requiresCompositingForIFrame):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• rendering/RenderLayerCompositor.cpp (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-04-24  Simon Fraser  <simon.fraser@apple.com>
+
+        Reviewed by Dan Bernstein.
+
+        Don't propagate compositing out of iframes on Mac
+        https://bugs.webkit.org/show_bug.cgi?id=38072
+
+        Propagating compositing out of iframes caused too many regressions on Mac,
+        so only do it for other platforms that may need to hook compositing layers
+        together across iframe boundaries.
+
+        * rendering/RenderLayerCompositor.cpp:
+        (WebCore::RenderLayerCompositor::requiresCompositingForIFrame):
+
 2010-04-24  Anton Muhin  <antonm@chromium.org>
 

trunk/WebCore/rendering/RenderLayerCompositor.cpp

@@ -1102 +1102 @@
 bool RenderLayerCompositor::requiresCompositingForIFrame(RenderObject* renderer) const
 {
+#if !PLATFORM(MAC)
     return renderer->isRenderIFrame() && toRenderIFrame(renderer)->requiresAcceleratedCompositing();
+#else
+    UNUSED_PARAM(renderer);
+    return false;
+#endif
 }