Log to the History Channel in a few more places
authorsimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 13 Feb 2017 22:53:46 +0000 (22:53 +0000)
committersimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 13 Feb 2017 22:53:46 +0000 (22:53 +0000)
https://bugs.webkit.org/show_bug.cgi?id=168252

Reviewed by Brady Eidson.

*** Aliens ***

* loader/HistoryController.cpp:
(WebCore::HistoryController::restoreScrollPositionAndViewState):
(WebCore::HistoryController::updateForReload):
* loader/NavigationScheduler.cpp:
(WebCore::NavigationScheduler::scheduleHistoryNavigation):
(WebCore::NavigationScheduler::timerFired):
(WebCore::NavigationScheduler::cancel):
* page/History.cpp:
(WebCore::History::go):

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

Source/WebCore/ChangeLog
Source/WebCore/loader/HistoryController.cpp
Source/WebCore/loader/NavigationScheduler.cpp
Source/WebCore/page/History.cpp

index dd11203..6c5028c 100644 (file)
@@ -1,3 +1,22 @@
+2017-02-13  Simon Fraser  <simon.fraser@apple.com>
+
+        Log to the History Channel in a few more places
+        https://bugs.webkit.org/show_bug.cgi?id=168252
+
+        Reviewed by Brady Eidson.
+
+        *** Aliens ***
+
+        * loader/HistoryController.cpp:
+        (WebCore::HistoryController::restoreScrollPositionAndViewState):
+        (WebCore::HistoryController::updateForReload):
+        * loader/NavigationScheduler.cpp:
+        (WebCore::NavigationScheduler::scheduleHistoryNavigation):
+        (WebCore::NavigationScheduler::timerFired):
+        (WebCore::NavigationScheduler::cancel):
+        * page/History.cpp:
+        (WebCore::History::go):
+
 2017-02-13  Alex Christensen  <achristensen@webkit.org>
 
         Percent should be allowed in non-special URL hosts
index 09b8d66..fd1b516 100644 (file)
@@ -159,7 +159,7 @@ void HistoryController::restoreScrollPositionAndViewState()
     if (view && !view->wasScrolledByUser()) {
         Page* page = m_frame.page();
         auto desiredScrollPosition = m_currentItem->scrollPosition();
-
+        LOG(Scrolling, "HistoryController::restoreScrollPositionAndViewState scrolling to %d,%d", desiredScrollPosition.x(), desiredScrollPosition.y());
         if (page && m_frame.isMainFrame() && m_currentItem->pageScaleFactor())
             page->setPageScaleFactor(m_currentItem->pageScaleFactor() * page->viewScaleFactor(), desiredScrollPosition);
         else
@@ -349,7 +349,7 @@ void HistoryController::updateForBackForwardNavigation()
 
 void HistoryController::updateForReload()
 {
-    LOG(History, "HistoryController %p updateForBackForwardNavigation: Updating History for reload in frame %p (main frame %d) %s", this, &m_frame, m_frame.isMainFrame(), m_frame.loader().documentLoader() ? m_frame.loader().documentLoader()->url().string().utf8().data() : "");
+    LOG(History, "HistoryController %p updateForReload: Updating History for reload in frame %p (main frame %d) %s", this, &m_frame, m_frame.isMainFrame(), m_frame.loader().documentLoader() ? m_frame.loader().documentLoader()->url().string().utf8().data() : "");
 
     if (m_currentItem) {
         PageCache::singleton().remove(*m_currentItem);
index 0e5b64d..6e5adef 100644 (file)
@@ -46,6 +46,7 @@
 #include "HTMLFrameOwnerElement.h"
 #include "HistoryItem.h"
 #include "InspectorInstrumentation.h"
+#include "Logging.h"
 #include "Page.h"
 #include "ScriptController.h"
 #include "UserGestureIndicator.h"
@@ -462,6 +463,7 @@ void NavigationScheduler::scheduleRefresh(Document& initiatingDocument)
 
 void NavigationScheduler::scheduleHistoryNavigation(int steps)
 {
+    LOG(History, "NavigationScheduler %p scheduleHistoryNavigation(%d) - shouldSchedule %d", this, steps, shouldScheduleNavigation());
     if (!shouldScheduleNavigation())
         return;
 
@@ -495,6 +497,8 @@ void NavigationScheduler::timerFired()
     Ref<Frame> protect(m_frame);
 
     std::unique_ptr<ScheduledNavigation> redirect = WTFMove(m_redirect);
+    LOG(History, "NavigationScheduler %p timerFired - firing redirect %p", this, redirect.get());
+
     redirect->fire(m_frame);
     InspectorInstrumentation::frameClearedScheduledNavigation(m_frame);
 }
@@ -545,6 +549,8 @@ void NavigationScheduler::startTimer()
 
 void NavigationScheduler::cancel(bool newLoadInProgress)
 {
+    LOG(History, "NavigationScheduler %p cancel(newLoadInProgress=%d)", this, newLoadInProgress);
+
     if (m_timer.isActive())
         InspectorInstrumentation::frameClearedScheduledNavigation(m_frame);
     m_timer.stop();
index 0daedd4..fbc52f3 100644 (file)
@@ -34,6 +34,7 @@
 #include "FrameLoaderClient.h"
 #include "HistoryController.h"
 #include "HistoryItem.h"
+#include "Logging.h"
 #include "MainFrame.h"
 #include "Page.h"
 #include "ScriptController.h"
@@ -106,6 +107,8 @@ void History::forward(Document& document)
 
 void History::go(int distance)
 {
+    LOG(History, "History %p go(%d) frame %p (main frame %d)", this, distance, m_frame, m_frame ? m_frame->isMainFrame() : false);
+
     if (!m_frame)
         return;
 
@@ -114,6 +117,8 @@ void History::go(int distance)
 
 void History::go(Document& document, int distance)
 {
+    LOG(History, "History %p go(%d) in document %p frame %p (main frame %d)", this, distance, &document, m_frame, m_frame ? m_frame->isMainFrame() : false);
+
     if (!m_frame)
         return;