Title: [152103] trunk/Source/WebCore
Revision
152103
Author
commit-qu...@webkit.org
Date
2013-06-27 09:33:05 -0700 (Thu, 27 Jun 2013)

Log Message

Removing the redundant checks after r152078
https://bugs.webkit.org/show_bug.cgi?id=118117

Patch by Seokju Kwon <seokju.k...@gmail.com> on 2013-06-27
Reviewed by Simon Fraser.

Avoided redundant checks in RenderLayerCompositor::reasonsForCompositing() after r152078.

No new tests as no functionality change.

* inspector/InspectorLayerTreeAgent.cpp:
(WebCore::InspectorLayerTreeAgent::reasonsForCompositingLayer):
* rendering/RenderLayerCompositor.cpp:
(WebCore::RenderLayerCompositor::logReasonsForCompositing):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (152102 => 152103)


--- trunk/Source/WebCore/ChangeLog	2013-06-27 16:32:52 UTC (rev 152102)
+++ trunk/Source/WebCore/ChangeLog	2013-06-27 16:33:05 UTC (rev 152103)
@@ -1,3 +1,19 @@
+2013-06-27  Seokju Kwon  <seokju.k...@gmail.com>
+
+        Removing the redundant checks after r152078
+        https://bugs.webkit.org/show_bug.cgi?id=118117
+
+        Reviewed by Simon Fraser.
+
+        Avoided redundant checks in RenderLayerCompositor::reasonsForCompositing() after r152078.
+
+        No new tests as no functionality change.
+
+        * inspector/InspectorLayerTreeAgent.cpp:
+        (WebCore::InspectorLayerTreeAgent::reasonsForCompositingLayer):
+        * rendering/RenderLayerCompositor.cpp:
+        (WebCore::RenderLayerCompositor::logReasonsForCompositing):
+
 2013-06-27  Christophe Dumez  <ch.du...@sisa.samsung.com>
 
         Remove [NoInterfaceObject] from WorkerGlobalScope

Modified: trunk/Source/WebCore/inspector/InspectorLayerTreeAgent.cpp (152102 => 152103)


--- trunk/Source/WebCore/inspector/InspectorLayerTreeAgent.cpp	2013-06-27 16:32:52 UTC (rev 152102)
+++ trunk/Source/WebCore/inspector/InspectorLayerTreeAgent.cpp	2013-06-27 16:33:05 UTC (rev 152103)
@@ -253,14 +253,11 @@
 
     if (reasonsBitmask & CompositingReasonVideo)
         compositingReasons->setVideo(true);
-
-    if (reasonsBitmask & CompositingReasonCanvas)
+    else if (reasonsBitmask & CompositingReasonCanvas)
         compositingReasons->setCanvas(true);
-
-    if (reasonsBitmask & CompositingReasonPlugin)
+    else if (reasonsBitmask & CompositingReasonPlugin)
         compositingReasons->setPlugin(true);
-
-    if (reasonsBitmask & CompositingReasonIFrame)
+    else if (reasonsBitmask & CompositingReasonIFrame)
         compositingReasons->setIFrame(true);
     
     if (reasonsBitmask & CompositingReasonBackfaceVisibilityHidden)

Modified: trunk/Source/WebCore/rendering/RenderLayerCompositor.cpp (152102 => 152103)


--- trunk/Source/WebCore/rendering/RenderLayerCompositor.cpp	2013-06-27 16:32:52 UTC (rev 152102)
+++ trunk/Source/WebCore/rendering/RenderLayerCompositor.cpp	2013-06-27 16:33:05 UTC (rev 152103)
@@ -1911,14 +1911,11 @@
 
     if (reasons & CompositingReasonVideo)
         return "video";
-
-    if (reasons & CompositingReasonCanvas)
+    else if (reasons & CompositingReasonCanvas)
         return "canvas";
-
-    if (reasons & CompositingReasonPlugin)
+    else if (reasons & CompositingReasonPlugin)
         return "plugin";
-
-    if (reasons & CompositingReasonIFrame)
+    else if (reasons & CompositingReasonIFrame)
         return "iframe";
     
     if (reasons & CompositingReasonBackfaceVisibilityHidden)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to