Changeset 202139 in webkit

Timestamp:

Jun 16, 2016 2:47:06 PM (8 years ago)

Author:

akling@apple.com

Message:

REGRESSION(r196217): 3% JSBench regression on iPhone 5.
<​https://webkit.org/b/158848>
<rdar://problem/26609622>

Unreviewed rollout.

Don't jettison linked code on every top-level navigation as that was hurting JSBench on iPhone 5.

• loader/FrameLoader.cpp:

(WebCore::FrameLoader::commitProvisionalLoad):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• loader/FrameLoader.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2016-06-16  Andreas Kling  <akling@apple.com>
+
+        REGRESSION(r196217): 3% JSBench regression on iPhone 5.
+        <https://webkit.org/b/158848>
+        <rdar://problem/26609622>
+
+        Unreviewed rollout.
+
+        Don't jettison linked code on every top-level navigation as that was hurting JSBench on iPhone 5.
+
+        * loader/FrameLoader.cpp:
+        (WebCore::FrameLoader::commitProvisionalLoad):
+
 2016-06-16  Adam Bergkvist  <adam.bergkvist@ericsson.com>
 

trunk/Source/WebCore/loader/FrameLoader.cpp

@@ -1787 +1787 @@
         // We are doing this here because we know for sure that a new page is about to be loaded.
         PageCache::singleton().addIfCacheable(*history().currentItem(), m_frame.page());
-
-#if PLATFORM(IOS)
-        // For top-level navigations, have JSC throw away linked code. The immediate memory savings far
-        // outweigh the cost of recompiling in the case of a future backwards navigation.
-        GCController::singleton().deleteAllLinkedCode();
-#endif
     }