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

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

Issue 2531603003: Only scroll on main if the targeted frames need to scroll on main (Closed)
Patch Set: Bug fixed Created 4 years 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 0fb749217d8d06e6504136175415ac9bfb220106..9cc2e648370228d88bb3fad6463767e9386e5871 100644
--- a/third_party/WebKit/Source/core/page/Page.cpp
+++ b/third_party/WebKit/Source/core/page/Page.cpp
@@ -42,6 +42,7 @@
#include "core/html/HTMLMediaElement.h"
#include "core/inspector/ConsoleMessageStorage.h"
#include "core/inspector/InspectorInstrumentation.h"
+#include "core/layout/LayoutView.h"
pdr. 2016/12/13 05:51:16 Nit: Is this needed anymore?
yigu 2016/12/14 21:07:27 Done.
#include "core/layout/TextAutosizer.h"
#include "core/page/AutoscrollController.h"
#include "core/page/ChromeClient.h"
@@ -164,9 +165,9 @@ ScrollingCoordinator* Page::scrollingCoordinator() {
return m_scrollingCoordinator.get();
}
-String Page::mainThreadScrollingReasonsAsText() {
+String Page::mainThreadScrollingReasonsAsText(const Frame& frame) {
if (ScrollingCoordinator* scrollingCoordinator = this->scrollingCoordinator())
- return scrollingCoordinator->mainThreadScrollingReasonsAsText();
+ return scrollingCoordinator->mainThreadScrollingReasonsAsText(&frame);
return String();
}

Powered by Google App Engine
This is Rietveld 408576698