Title: [185288] trunk/Source/WebCore
Revision
185288
Author
[email protected]
Date
2015-06-06 01:43:10 -0700 (Sat, 06 Jun 2015)

Log Message

Unreviewed, rolling out r185284.
https://bugs.webkit.org/show_bug.cgi?id=145723

it needs some discussion before landing (Requested by dino on
#webkit).

Reverted changeset:

"[iOS] Don't force compositing layers for no-op 3D transforms
on low-memory devices."
https://bugs.webkit.org/show_bug.cgi?id=145719
http://trac.webkit.org/changeset/185284

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (185287 => 185288)


--- trunk/Source/WebCore/ChangeLog	2015-06-06 08:21:20 UTC (rev 185287)
+++ trunk/Source/WebCore/ChangeLog	2015-06-06 08:43:10 UTC (rev 185288)
@@ -1,3 +1,18 @@
+2015-06-06  Commit Queue  <[email protected]>
+
+        Unreviewed, rolling out r185284.
+        https://bugs.webkit.org/show_bug.cgi?id=145723
+
+        it needs some discussion before landing (Requested by dino on
+        #webkit).
+
+        Reverted changeset:
+
+        "[iOS] Don't force compositing layers for no-op 3D transforms
+        on low-memory devices."
+        https://bugs.webkit.org/show_bug.cgi?id=145719
+        http://trac.webkit.org/changeset/185284
+
 2015-06-06  Ryosuke Niwa  <[email protected]>
 
         Typing is slow in Gmail on iPads

Modified: trunk/Source/WebCore/rendering/RenderLayerCompositor.cpp (185287 => 185288)


--- trunk/Source/WebCore/rendering/RenderLayerCompositor.cpp	2015-06-06 08:21:20 UTC (rev 185287)
+++ trunk/Source/WebCore/rendering/RenderLayerCompositor.cpp	2015-06-06 08:43:10 UTC (rev 185288)
@@ -62,7 +62,6 @@
 #include "TiledBacking.h"
 #include "TransformState.h"
 #include <wtf/CurrentTime.h>
-#include <wtf/RAMSize.h>
 #include <wtf/TemporaryChange.h>
 #include <wtf/text/CString.h>
 #include <wtf/text/StringBuilder.h>
@@ -2466,25 +2465,7 @@
 
     // Note that we ask the renderer if it has a transform, because the style may have transforms,
     // but the renderer may be an inline that doesn't suppport them.
-    if (!renderer.hasTransform())
-        return false;
-
-    if (!renderer.style().transform().has3DOperation())
-        return false;
-
-#if PLATFORM(IOS)
-    static uint64_t ramSizeInMB = ramSize() / 1024 / 1024;
-    if (ramSizeInMB <= 512) {
-        // Special policy for low-memory devices: Don't require compositing just because there's a no-op 3D transform.
-        for (auto& transform : renderer.style().transform().operations()) {
-            if (!transform->isIdentity())
-                return true;
-        }
-        return false;
-    }
-#endif
-
-    return true;
+    return renderer.hasTransform() && renderer.style().transform().has3DOperation();
 }
 
 bool RenderLayerCompositor::requiresCompositingForBackfaceVisibility(RenderLayerModelObject& renderer) const
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to