Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(570)

Unified Diff: third_party/WebKit/Source/core/page/Page.cpp

Issue 2320303002: Reset VisualViewport position after same page navigation (Closed)
Patch Set: update comment Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..a5fa749cea263a6e18c4c61f819f0e9c28fef445 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 to reset visual viewport position here since before commit load we would update the previous history item,
+ // Page::didCommitLoad is called after a new history item is created in FrameLoader.
+ // fix for crbug.com/642279
+ frameHost().visualViewport().setScrollPosition(DoublePoint(), ProgrammaticScroll);
m_hostsUsingFeatures.updateMeasurementsAndClear();
UserGestureIndicator::clearProcessedUserGestureSinceLoad();
}

Powered by Google App Engine
This is Rietveld 408576698