Title: [202139] trunk/Source/WebCore
Revision
202139
Author
[email protected]
Date
2016-06-16 14:47:06 -0700 (Thu, 16 Jun 2016)

Log 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):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (202138 => 202139)


--- trunk/Source/WebCore/ChangeLog	2016-06-16 21:35:48 UTC (rev 202138)
+++ trunk/Source/WebCore/ChangeLog	2016-06-16 21:47:06 UTC (rev 202139)
@@ -1,3 +1,16 @@
+2016-06-16  Andreas Kling  <[email protected]>
+
+        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  <[email protected]>
 
         WebRTC: Check type of this in RTCPeerConnection JS built-in functions

Modified: trunk/Source/WebCore/loader/FrameLoader.cpp (202138 => 202139)


--- trunk/Source/WebCore/loader/FrameLoader.cpp	2016-06-16 21:35:48 UTC (rev 202138)
+++ trunk/Source/WebCore/loader/FrameLoader.cpp	2016-06-16 21:47:06 UTC (rev 202139)
@@ -1786,12 +1786,6 @@
         // Check to see if we need to cache the page we are navigating away from into the back/forward cache.
         // 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
     }
 
     if (m_loadType != FrameLoadType::Replace)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to