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

Side by Side Diff: content/renderer/render_view_impl.cc

Issue 2929493003: Move handling of DraggableRegionsChanged notification from "view" to "frame". (Closed)
Patch Set: Addressed CR feedback from rdevlin.cronin@. Created 3 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/renderer/render_view_impl.h" 5 #include "content/renderer/render_view_impl.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <cmath> 8 #include <cmath>
9 #include <memory> 9 #include <memory>
10 10
(...skipping 2345 matching lines...) Expand 10 before | Expand all | Expand 10 after
2356 } 2356 }
2357 2357
2358 double RenderViewImpl::zoomLevelToZoomFactor(double zoom_level) const { 2358 double RenderViewImpl::zoomLevelToZoomFactor(double zoom_level) const {
2359 return ZoomLevelToZoomFactor(zoom_level); 2359 return ZoomLevelToZoomFactor(zoom_level);
2360 } 2360 }
2361 2361
2362 double RenderViewImpl::zoomFactorToZoomLevel(double factor) const { 2362 double RenderViewImpl::zoomFactorToZoomLevel(double factor) const {
2363 return ZoomFactorToZoomLevel(factor); 2363 return ZoomFactorToZoomLevel(factor);
2364 } 2364 }
2365 2365
2366 void RenderViewImpl::DraggableRegionsChanged() {
2367 for (auto& observer : observers_)
2368 observer.DraggableRegionsChanged(webview()->MainFrame());
2369 }
2370
2371 void RenderViewImpl::PageImportanceSignalsChanged() { 2366 void RenderViewImpl::PageImportanceSignalsChanged() {
2372 if (!webview() || !main_render_frame_) 2367 if (!webview() || !main_render_frame_)
2373 return; 2368 return;
2374 2369
2375 auto* web_signals = webview()->PageImportanceSignals(); 2370 auto* web_signals = webview()->PageImportanceSignals();
2376 2371
2377 PageImportanceSignals signals; 2372 PageImportanceSignals signals;
2378 signals.had_form_interaction = web_signals->HadFormInteraction(); 2373 signals.had_form_interaction = web_signals->HadFormInteraction();
2379 2374
2380 main_render_frame_->Send(new FrameHostMsg_UpdatePageImportanceSignals( 2375 main_render_frame_->Send(new FrameHostMsg_UpdatePageImportanceSignals(
(...skipping 275 matching lines...) Expand 10 before | Expand all | Expand 10 after
2656 input_event.GetCoalescedEventsPointers(), latency_info, 2651 input_event.GetCoalescedEventsPointers(), latency_info,
2657 dispatch_type); 2652 dispatch_type);
2658 return INPUT_EVENT_ACK_STATE_NOT_CONSUMED; 2653 return INPUT_EVENT_ACK_STATE_NOT_CONSUMED;
2659 } 2654 }
2660 idle_user_detector_->ActivityDetected(); 2655 idle_user_detector_->ActivityDetected();
2661 return RenderWidget::HandleInputEvent(input_event, latency_info, 2656 return RenderWidget::HandleInputEvent(input_event, latency_info,
2662 dispatch_type); 2657 dispatch_type);
2663 } 2658 }
2664 2659
2665 } // namespace content 2660 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/render_view_impl.h ('k') | extensions/browser/app_window/app_window_contents.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698