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

Side by Side Diff: content/browser/renderer_host/render_widget_host_view_base.cc

Issue 2864833002: Tapping handle shouldn't select misspelled word (Closed)
Patch Set: Mac test Created 3 years, 7 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/browser/renderer_host/render_widget_host_view_base.h" 5 #include "content/browser/renderer_host/render_widget_host_view_base.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "build/build_config.h" 8 #include "build/build_config.h"
9 #include "content/browser/accessibility/browser_accessibility_manager.h" 9 #include "content/browser/accessibility/browser_accessibility_manager.h"
10 #include "content/browser/gpu/gpu_data_manager_impl.h" 10 #include "content/browser/gpu/gpu_data_manager_impl.h"
11 #include "content/browser/renderer_host/input/synthetic_gesture_target_base.h" 11 #include "content/browser/renderer_host/input/synthetic_gesture_target_base.h"
12 #include "content/browser/renderer_host/render_process_host_impl.h" 12 #include "content/browser/renderer_host/render_process_host_impl.h"
13 #include "content/browser/renderer_host/render_widget_host_delegate.h" 13 #include "content/browser/renderer_host/render_widget_host_delegate.h"
14 #include "content/browser/renderer_host/render_widget_host_impl.h" 14 #include "content/browser/renderer_host/render_widget_host_impl.h"
15 #include "content/browser/renderer_host/render_widget_host_view_base_observer.h" 15 #include "content/browser/renderer_host/render_widget_host_view_base_observer.h"
16 #include "content/browser/renderer_host/render_widget_host_view_frame_subscriber .h" 16 #include "content/browser/renderer_host/render_widget_host_view_frame_subscriber .h"
17 #include "content/browser/renderer_host/text_input_manager.h" 17 #include "content/browser/renderer_host/text_input_manager.h"
18 #include "content/common/content_switches_internal.h" 18 #include "content/common/content_switches_internal.h"
19 #include "media/base/video_frame.h" 19 #include "media/base/video_frame.h"
20 #include "ui/base/layout.h" 20 #include "ui/base/layout.h"
21 #include "ui/base/ui_base_types.h"
21 #include "ui/display/screen.h" 22 #include "ui/display/screen.h"
22 #include "ui/gfx/geometry/point_conversions.h" 23 #include "ui/gfx/geometry/point_conversions.h"
23 #include "ui/gfx/geometry/size_conversions.h" 24 #include "ui/gfx/geometry/size_conversions.h"
24 #include "ui/gfx/geometry/size_f.h" 25 #include "ui/gfx/geometry/size_f.h"
25 26
26 namespace content { 27 namespace content {
27 28
28 namespace { 29 namespace {
29 30
30 } 31 }
(...skipping 177 matching lines...) Expand 10 before | Expand all | Expand 10 after
208 NOTREACHED(); 209 NOTREACHED();
209 return NULL; 210 return NULL;
210 } 211 }
211 212
212 void RenderWidgetHostViewBase::AccessibilityShowMenu(const gfx::Point& point) { 213 void RenderWidgetHostViewBase::AccessibilityShowMenu(const gfx::Point& point) {
213 RenderWidgetHostImpl* impl = NULL; 214 RenderWidgetHostImpl* impl = NULL;
214 if (GetRenderWidgetHost()) 215 if (GetRenderWidgetHost())
215 impl = RenderWidgetHostImpl::From(GetRenderWidgetHost()); 216 impl = RenderWidgetHostImpl::From(GetRenderWidgetHost());
216 217
217 if (impl) 218 if (impl)
218 impl->ShowContextMenuAtPoint(point); 219 impl->ShowContextMenuAtPoint(point, ui::MENU_SOURCE_MOUSE);
bokan 2017/05/23 14:06:02 I think this shouldn't be mouse - I'm assuming Acc
amaralp 2017/05/23 19:37:53 Done.
219 } 220 }
220 221
221 gfx::Point RenderWidgetHostViewBase::AccessibilityOriginInScreen( 222 gfx::Point RenderWidgetHostViewBase::AccessibilityOriginInScreen(
222 const gfx::Rect& bounds) { 223 const gfx::Rect& bounds) {
223 return bounds.origin(); 224 return bounds.origin();
224 } 225 }
225 226
226 gfx::AcceleratedWidget 227 gfx::AcceleratedWidget
227 RenderWidgetHostViewBase::AccessibilityGetAcceleratedWidget() { 228 RenderWidgetHostViewBase::AccessibilityGetAcceleratedWidget() {
228 return gfx::kNullAcceleratedWidget; 229 return gfx::kNullAcceleratedWidget;
(...skipping 251 matching lines...) Expand 10 before | Expand all | Expand 10 after
480 481
481 bool RenderWidgetHostViewBase::IsChildFrameForTesting() const { 482 bool RenderWidgetHostViewBase::IsChildFrameForTesting() const {
482 return false; 483 return false;
483 } 484 }
484 485
485 cc::SurfaceId RenderWidgetHostViewBase::SurfaceIdForTesting() const { 486 cc::SurfaceId RenderWidgetHostViewBase::SurfaceIdForTesting() const {
486 return cc::SurfaceId(); 487 return cc::SurfaceId();
487 } 488 }
488 489
489 } // namespace content 490 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698