Changeset 152078 in webkit

Timestamp:

Jun 26, 2013 11:02:51 PM (11 years ago)

Author:

commit-queue@webkit.org

Message:

Avoid redundant checks for Compositing reasons in RenderLayerCompositor
​https://bugs.webkit.org/show_bug.cgi?id=118114

Patch by Sanjoy Pal <​sanjoy.pal@samsung.com> on 2013-06-26
Reviewed by Anders Carlsson.

A RenderObject can be any one of the Video, Canvas, Plugin or Iframe.
Hence removing the redundant checks.

No new tests as no functionality change.

• rendering/RenderLayerCompositor.cpp:

(WebCore::RenderLayerCompositor::reasonsForCompositing):

Location:

trunk/Source/WebCore

Files:

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

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2013-06-26  Sanjoy Pal  <sanjoy.pal@samsung.com>
+
+        Avoid redundant checks for Compositing reasons in RenderLayerCompositor
+        https://bugs.webkit.org/show_bug.cgi?id=118114
+
+        Reviewed by Anders Carlsson.
+
+        A RenderObject can be any one of the Video, Canvas, Plugin or Iframe.
+        Hence removing the redundant checks.
+
+        No new tests as no functionality change.
+
+        * rendering/RenderLayerCompositor.cpp:
+        (WebCore::RenderLayerCompositor::reasonsForCompositing):
+
 2013-06-26  Gwang Yoon Hwang  <ryumiel@company100.net>
 

trunk/Source/WebCore/rendering/RenderLayerCompositor.cpp

@@ -1842 +1842 @@
     if (requiresCompositingForVideo(renderer))
         reasons |= CompositingReasonVideo;
-
-    if (requiresCompositingForCanvas(renderer))
+    else if (requiresCompositingForCanvas(renderer))
         reasons |= CompositingReasonCanvas;
-
-    if (requiresCompositingForPlugin(renderer))
+    else if (requiresCompositingForPlugin(renderer))
         reasons |= CompositingReasonPlugin;
-
-    if (requiresCompositingForFrame(renderer))
+    else if (requiresCompositingForFrame(renderer))
         reasons |= CompositingReasonIFrame;