Reset VisualViewport position after same page navigation

third_party/WebKit/Source/core/page/Page.cpp

Index: third_party/WebKit/Source/core/page/Page.cpp
diff --git a/third_party/WebKit/Source/core/page/Page.cpp b/third_party/WebKit/Source/core/page/Page.cpp
index fe3e99ea8e0f9a28f94f496c9aba209c8ad98917..ea5b0c15824ec920aaf888c67505387995d6d29d 100644
--- a/third_party/WebKit/Source/core/page/Page.cpp
+++ b/third_party/WebKit/Source/core/page/Page.cpp
@@ -436,6 +436,11 @@ void Page::didCommitLoad(LocalFrame* frame)
         useCounter().didCommitLoad();
         deprecation().clearSuppression();
         frameHost().visualViewport().sendUMAMetrics();
+
+        // Need reset visual viewport position here since before this will change the history current item,

[bokan, 2016/09/15 18:27:32]

Nit: "Need reset visual" -> "Need to reset the visual"
Nit: "before this will change the history current item" -> "before commit load
we would update the previous history item"

+        // Page::didCommitLoad is after a new history item is created in FrameLoader.

[bokan, 2016/09/15 18:27:32]

Nit: "didCommitLoad is after" -> "didCommitLoad is called after"

+        // fix for crbug.com/642279
+        frameHost().visualViewport().setScrollPosition(DoublePoint(), ProgrammaticScroll);
         m_hostsUsingFeatures.updateMeasurementsAndClear();
         UserGestureIndicator::clearProcessedUserGestureSinceLoad();
     }