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

Side by Side Diff: content/browser/renderer_host/render_widget_host_impl.h

Issue 16114003: Don't send touch move to renderer while scrolling (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: fix the case touch event queue could be empty when GetLatestEvent() is called Created 7 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 #ifndef CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_IMPL_H_ 5 #ifndef CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_IMPL_H_
6 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_IMPL_H_ 6 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_IMPL_H_
7 7
8 #include <deque> 8 #include <deque>
9 #include <list> 9 #include <list>
10 #include <map> 10 #include <map>
(...skipping 326 matching lines...) Expand 10 before | Expand all | Expand 10 after
337 } 337 }
338 338
339 bool input_method_active() const { 339 bool input_method_active() const {
340 return input_method_active_; 340 return input_method_active_;
341 } 341 }
342 342
343 bool ShouldForwardTouchEvent() const; 343 bool ShouldForwardTouchEvent() const;
344 344
345 bool has_touch_handler() const { return has_touch_handler_; } 345 bool has_touch_handler() const { return has_touch_handler_; }
346 346
347 bool is_scroll_update_in_progress() const {
348 return scroll_update_in_progress_;
349 }
350
347 // Notification that the user has made some kind of input that could 351 // Notification that the user has made some kind of input that could
348 // perform an action. See OnUserGesture for more details. 352 // perform an action. See OnUserGesture for more details.
349 void StartUserGesture(); 353 void StartUserGesture();
350 354
351 // Set the RenderView background. 355 // Set the RenderView background.
352 void SetBackground(const SkBitmap& background); 356 void SetBackground(const SkBitmap& background);
353 357
354 // Notifies the renderer that the next key event is bound to one or more 358 // Notifies the renderer that the next key event is bound to one or more
355 // pre-defined edit commands 359 // pre-defined edit commands
356 void SetEditCommandsForNextKeyEvent( 360 void SetEditCommandsForNextKeyEvent(
(...skipping 549 matching lines...) Expand 10 before | Expand all | Expand 10 after
906 // switching back to the original tab, because the content may already be 910 // switching back to the original tab, because the content may already be
907 // changed. 911 // changed.
908 bool suppress_next_char_events_; 912 bool suppress_next_char_events_;
909 913
910 // The last scroll offset of the render widget. 914 // The last scroll offset of the render widget.
911 gfx::Vector2d last_scroll_offset_; 915 gfx::Vector2d last_scroll_offset_;
912 916
913 bool pending_mouse_lock_request_; 917 bool pending_mouse_lock_request_;
914 bool allow_privileged_mouse_lock_; 918 bool allow_privileged_mouse_lock_;
915 919
920 bool scroll_update_in_progress_;
921
916 // Keeps track of whether the webpage has any touch event handler. If it does, 922 // Keeps track of whether the webpage has any touch event handler. If it does,
917 // then touch events are sent to the renderer. Otherwise, the touch events are 923 // then touch events are sent to the renderer. Otherwise, the touch events are
918 // not sent to the renderer. 924 // not sent to the renderer.
919 bool has_touch_handler_; 925 bool has_touch_handler_;
920 926
921 base::WeakPtrFactory<RenderWidgetHostImpl> weak_factory_; 927 base::WeakPtrFactory<RenderWidgetHostImpl> weak_factory_;
922 928
923 SmoothScrollGestureController smooth_scroll_gesture_controller_; 929 SmoothScrollGestureController smooth_scroll_gesture_controller_;
924 scoped_ptr<TouchEventQueue> touch_event_queue_; 930 scoped_ptr<TouchEventQueue> touch_event_queue_;
925 scoped_ptr<GestureEventFilter> gesture_event_filter_; 931 scoped_ptr<GestureEventFilter> gesture_event_filter_;
926 scoped_ptr<OverscrollController> overscroll_controller_; 932 scoped_ptr<OverscrollController> overscroll_controller_;
927 933
928 #if defined(OS_WIN) 934 #if defined(OS_WIN)
929 std::list<HWND> dummy_windows_for_activation_; 935 std::list<HWND> dummy_windows_for_activation_;
930 #endif 936 #endif
931 937
932 // List of callbacks for pending snapshot requests to the renderer. 938 // List of callbacks for pending snapshot requests to the renderer.
933 std::queue<base::Callback<void(bool, const SkBitmap&)> > pending_snapshots_; 939 std::queue<base::Callback<void(bool, const SkBitmap&)> > pending_snapshots_;
934 940
935 int64 last_input_number_; 941 int64 last_input_number_;
936 942
937 BrowserRenderingStats rendering_stats_; 943 BrowserRenderingStats rendering_stats_;
938 944
939 DISALLOW_COPY_AND_ASSIGN(RenderWidgetHostImpl); 945 DISALLOW_COPY_AND_ASSIGN(RenderWidgetHostImpl);
940 }; 946 };
941 947
942 } // namespace content 948 } // namespace content
943 949
944 #endif // CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_IMPL_H_ 950 #endif // CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698