diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
index 1918f3e..10e88c7 100644
--- a/Source/WebCore/ChangeLog
+++ b/Source/WebCore/ChangeLog
@@ -1,3 +1,12 @@
+2019-10-11  Chris Dumez  <cdumez@apple.com>
+
+        Unreviewed, address post-landing feedback from Darin for r251019.
+
+        Added a new comment for clarity.
+
+        * loader/FrameLoader.cpp:
+        (WebCore::FrameLoader::stopForPageCache):
+
 2019-10-11  Alex Christensen  <achristensen@webkit.org>
 
         Only use CFNetwork SPI for metrics where needed
diff --git a/Source/WebCore/loader/FrameLoader.cpp b/Source/WebCore/loader/FrameLoader.cpp
index ed7409c..61a0aeb 100644
--- a/Source/WebCore/loader/FrameLoader.cpp
+++ b/Source/WebCore/loader/FrameLoader.cpp
@@ -1863,7 +1863,8 @@
     for (RefPtr<Frame> child = m_frame.tree().firstChild(); child; child = child->tree().nextSibling())
         child->loader().stopForPageCache();
 
-    // Make sure there are no scheduled loads or policy checks.
+    // We cancel pending navigations & policy checks *after* cancelling loads because cancelling loads might end up
+    // running script, which could schedule new navigations.
     policyChecker().stopCheck();
     m_frame.navigationScheduler().cancel();
 }