Title: [127913] trunk/Source/WebCore
Revision
127913
Author
[email protected]
Date
2012-09-07 13:47:44 -0700 (Fri, 07 Sep 2012)

Log Message

[chromium] Remove bad assert in CCRenderPass::Id
https://bugs.webkit.org/show_bug.cgi?id=96138

Reviewed by James Robinson.

The assert prevents HashMap from creating "deleted" and "empty" ids. We
assert that RenderPasses have correct ids already, and don't need to
assert inside the Id structure itself.

* platform/graphics/chromium/cc/CCRenderPass.h:
(WebCore::CCRenderPass::Id::Id):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (127912 => 127913)


--- trunk/Source/WebCore/ChangeLog	2012-09-07 20:41:40 UTC (rev 127912)
+++ trunk/Source/WebCore/ChangeLog	2012-09-07 20:47:44 UTC (rev 127913)
@@ -1,3 +1,17 @@
+2012-09-07  Dana Jansens  <[email protected]>
+
+        [chromium] Remove bad assert in CCRenderPass::Id
+        https://bugs.webkit.org/show_bug.cgi?id=96138
+
+        Reviewed by James Robinson.
+
+        The assert prevents HashMap from creating "deleted" and "empty" ids. We
+        assert that RenderPasses have correct ids already, and don't need to
+        assert inside the Id structure itself.
+
+        * platform/graphics/chromium/cc/CCRenderPass.h:
+        (WebCore::CCRenderPass::Id::Id):
+
 2012-09-07  Arvid Nilsson  <[email protected]>
 
         [CSS Shaders] [BlackBerry] Refactor CustomFilterMesh and CustomFilter*Program

Modified: trunk/Source/WebCore/platform/graphics/chromium/cc/CCRenderPass.h (127912 => 127913)


--- trunk/Source/WebCore/platform/graphics/chromium/cc/CCRenderPass.h	2012-09-07 20:41:40 UTC (rev 127912)
+++ trunk/Source/WebCore/platform/graphics/chromium/cc/CCRenderPass.h	2012-09-07 20:47:44 UTC (rev 127913)
@@ -68,8 +68,6 @@
             : layerId(layerId)
             , index(index)
         {
-            ASSERT(layerId > 0);
-            ASSERT(index >= 0);
         }
 
         bool operator==(const Id& other) const { return layerId == other.layerId && index == other.index; }
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to