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

Side by Side Diff: content/renderer/render_widget.h

Issue 2522353002: Remove the CHECK in RenderWidget::GetInputMethodController (Closed)
Patch Set: Addressing creis@'s comments + Removing unused forward dec. 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 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_RENDERER_RENDER_WIDGET_H_ 5 #ifndef CONTENT_RENDERER_RENDER_WIDGET_H_
6 #define CONTENT_RENDERER_RENDER_WIDGET_H_ 6 #define CONTENT_RENDERER_RENDER_WIDGET_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 #include <stdint.h> 9 #include <stdint.h>
10 10
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
66 66
67 namespace blink { 67 namespace blink {
68 namespace scheduler { 68 namespace scheduler {
69 class RenderWidgetSchedulingState; 69 class RenderWidgetSchedulingState;
70 } 70 }
71 struct WebDeviceEmulationParams; 71 struct WebDeviceEmulationParams;
72 class WebDragData; 72 class WebDragData;
73 class WebFrameWidget; 73 class WebFrameWidget;
74 class WebGestureEvent; 74 class WebGestureEvent;
75 class WebImage; 75 class WebImage;
76 class WebInputMethodController;
77 class WebLocalFrame; 76 class WebLocalFrame;
78 class WebMouseEvent; 77 class WebMouseEvent;
79 class WebNode; 78 class WebNode;
80 struct WebPoint; 79 struct WebPoint;
81 } 80 }
82 81
83 namespace cc { 82 namespace cc {
84 class CompositorFrameSink; 83 class CompositorFrameSink;
85 class FrameSinkId; 84 class FrameSinkId;
86 class SwapPromise; 85 class SwapPromise;
(...skipping 75 matching lines...) Expand 10 before | Expand all | Expand 10 after
162 // Closes a RenderWidget that was created by |CreateForFrame|. 161 // Closes a RenderWidget that was created by |CreateForFrame|.
163 // TODO(avi): De-virtualize this once RenderViewImpl has-a RenderWidget. 162 // TODO(avi): De-virtualize this once RenderViewImpl has-a RenderWidget.
164 // https://crbug.com/545684 163 // https://crbug.com/545684
165 virtual void CloseForFrame(); 164 virtual void CloseForFrame();
166 165
167 int32_t routing_id() const { return routing_id_; } 166 int32_t routing_id() const { return routing_id_; }
168 167
169 CompositorDependencies* compositor_deps() const { return compositor_deps_; } 168 CompositorDependencies* compositor_deps() const { return compositor_deps_; }
170 virtual blink::WebWidget* GetWebWidget() const; 169 virtual blink::WebWidget* GetWebWidget() const;
171 170
172 // Returns the current instance of WebInputMethodController which is to be
173 // used for IME related tasks. This instance corresponds to the one from
174 // focused frame and can be nullptr.
175 blink::WebInputMethodController* GetInputMethodController() const;
176
177 const gfx::Size& size() const { return size_; } 171 const gfx::Size& size() const { return size_; }
178 bool is_fullscreen_granted() const { return is_fullscreen_granted_; } 172 bool is_fullscreen_granted() const { return is_fullscreen_granted_; }
179 blink::WebDisplayMode display_mode() const { return display_mode_; } 173 blink::WebDisplayMode display_mode() const { return display_mode_; }
180 bool is_hidden() const { return is_hidden_; } 174 bool is_hidden() const { return is_hidden_; }
181 // Temporary for debugging purposes... 175 // Temporary for debugging purposes...
182 bool closing() const { return closing_; } 176 bool closing() const { return closing_; }
183 bool has_host_context_menu_location() { 177 bool has_host_context_menu_location() {
184 return has_host_context_menu_location_; 178 return has_host_context_menu_location_;
185 } 179 }
186 gfx::Point host_context_menu_location() { 180 gfx::Point host_context_menu_location() {
(...skipping 672 matching lines...) Expand 10 before | Expand all | Expand 10 after
859 // being handled. If the current event results in starting a drag/drop 853 // being handled. If the current event results in starting a drag/drop
860 // session, this info is sent to the browser along with other drag/drop info. 854 // session, this info is sent to the browser along with other drag/drop info.
861 DragEventSourceInfo possible_drag_event_info_; 855 DragEventSourceInfo possible_drag_event_info_;
862 856
863 DISALLOW_COPY_AND_ASSIGN(RenderWidget); 857 DISALLOW_COPY_AND_ASSIGN(RenderWidget);
864 }; 858 };
865 859
866 } // namespace content 860 } // namespace content
867 861
868 #endif // CONTENT_RENDERER_RENDER_WIDGET_H_ 862 #endif // CONTENT_RENDERER_RENDER_WIDGET_H_
OLDNEW
« no previous file with comments | « no previous file | content/renderer/render_widget.cc » ('j') | third_party/WebKit/public/web/WebWidget.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698