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

Unified Diff: content/renderer/render_view_impl.cc

Issue 325933003: Scroll offset pinning and content size notifications were only used for the Mac overlay navigation … (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 6 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
« no previous file with comments | « content/renderer/render_view_impl.h ('k') | content/renderer/render_widget.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index b6542ffbd6a27e3a63ea96e4858bb8f0a758a83b..b2b9997d1ef533f71f0f7600bdc91bac24cd048c 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -650,8 +650,6 @@ RenderViewImpl::RenderViewImpl(RenderViewImplParams* params)
#if defined(OS_ANDROID)
top_controls_constraints_(cc::BOTH),
#endif
- cached_is_main_frame_pinned_to_left_(false),
- cached_is_main_frame_pinned_to_right_(false),
has_scrolled_focused_editable_node_into_rect_(false),
push_messaging_dispatcher_(NULL),
speech_recognition_dispatcher_(NULL),
@@ -2226,22 +2224,6 @@ BrowserPluginManager* RenderViewImpl::GetBrowserPluginManager() {
}
void RenderViewImpl::UpdateScrollState(WebFrame* frame) {
- WebSize offset = frame->scrollOffset();
- WebSize minimum_offset = frame->minimumScrollOffset();
- WebSize maximum_offset = frame->maximumScrollOffset();
-
- bool is_pinned_to_left = offset.width <= minimum_offset.width;
- bool is_pinned_to_right = offset.width >= maximum_offset.width;
-
- if (is_pinned_to_left != cached_is_main_frame_pinned_to_left_ ||
- is_pinned_to_right != cached_is_main_frame_pinned_to_right_) {
- Send(new ViewHostMsg_DidChangeScrollOffsetPinningForMainFrame(
- routing_id_, is_pinned_to_left, is_pinned_to_right));
-
- cached_is_main_frame_pinned_to_left_ = is_pinned_to_left;
- cached_is_main_frame_pinned_to_right_ = is_pinned_to_right;
- }
-
Send(new ViewHostMsg_DidChangeScrollOffset(routing_id_));
}
« no previous file with comments | « content/renderer/render_view_impl.h ('k') | content/renderer/render_widget.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698