Title: [107276] branches/chromium/1025/Source/WebCore/rendering
- Revision
- 107276
- Author
- [email protected]
- Date
- 2012-02-09 12:43:02 -0800 (Thu, 09 Feb 2012)
Log Message
Merge 107024 - Properly detect top level frames when propogating compositing
https://bugs.webkit.org/show_bug.cgi?id=78033
Reviewed by James Robinson.
There's no need to enumerate all tag names when searching for a
top-level frame. If a render view's document has a frame, then that
frame is not the top-level one.
* rendering/RenderLayerCompositor.cpp:
(WebCore::RenderLayerCompositor::shouldPropagateCompositingToEnclosingFrame):
* rendering/RenderLayerCompositor.h:
(RenderLayerCompositor):
[email protected]
Review URL: https://chromiumcodereview.appspot.com/9369039
Modified Paths
Diff
Modified: branches/chromium/1025/Source/WebCore/rendering/RenderLayerCompositor.cpp (107275 => 107276)
--- branches/chromium/1025/Source/WebCore/rendering/RenderLayerCompositor.cpp 2012-02-09 20:21:24 UTC (rev 107275)
+++ branches/chromium/1025/Source/WebCore/rendering/RenderLayerCompositor.cpp 2012-02-09 20:43:02 UTC (rev 107276)
@@ -1265,7 +1265,7 @@
// Parent document content needs to be able to render on top of a composited frame, so correct behavior
// is to have the parent document become composited too. However, this can cause problems on platforms that
// use native views for frames (like Mac), so disable that behavior on those platforms for now.
- HTMLFrameOwnerElement* ownerElement = enclosingFrameElement();
+ HTMLFrameOwnerElement* ownerElement = m_renderView->document()->ownerElement();
RenderObject* renderer = ownerElement ? ownerElement->renderer() : 0;
// If we are the top-level frame, don't propagate.
@@ -1296,14 +1296,6 @@
return false;
}
-HTMLFrameOwnerElement* RenderLayerCompositor::enclosingFrameElement() const
-{
- if (HTMLFrameOwnerElement* ownerElement = m_renderView->document()->ownerElement())
- return (ownerElement->hasTagName(iframeTag) || ownerElement->hasTagName(frameTag) || ownerElement->hasTagName(objectTag)) ? ownerElement : 0;
-
- return 0;
-}
-
bool RenderLayerCompositor::needsToBeComposited(const RenderLayer* layer) const
{
if (!canBeComposited(layer))
Modified: branches/chromium/1025/Source/WebCore/rendering/RenderLayerCompositor.h (107275 => 107276)
--- branches/chromium/1025/Source/WebCore/rendering/RenderLayerCompositor.h 2012-02-09 20:21:24 UTC (rev 107275)
+++ branches/chromium/1025/Source/WebCore/rendering/RenderLayerCompositor.h 2012-02-09 20:43:02 UTC (rev 107276)
@@ -178,8 +178,6 @@
static bool allowsIndependentlyCompositedFrames(const FrameView*);
bool shouldPropagateCompositingToEnclosingFrame() const;
- HTMLFrameOwnerElement* enclosingFrameElement() const;
-
static RenderLayerCompositor* frameContentsCompositor(RenderPart*);
// Return true if the layers changed.
static bool parentFrameContentLayers(RenderPart*);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
