Unreviewed, rolling out r185284.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 6 Jun 2015 08:43:10 +0000 (08:43 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 6 Jun 2015 08:43:10 +0000 (08:43 +0000)
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

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@185288 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/rendering/RenderLayerCompositor.cpp

index 53d36ac..d590025 100644 (file)
@@ -1,3 +1,18 @@
+2015-06-06  Commit Queue  <commit-queue@webkit.org>
+
+        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  <rniwa@webkit.org>
 
         Typing is slow in Gmail on iPads
index a2e921e..f217e79 100644 (file)
@@ -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 @@ bool RenderLayerCompositor::requiresCompositingForTransform(RenderLayerModelObje
 
     // 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