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

Unified Diff: chrome/browser/renderer_host/render_widget_host_view_views.cc

Issue 7041003: Show composition text on IME panel when Pepper plugin is focused (Linux). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: A todo comment is reworded to be more correct. Created 9 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
Index: chrome/browser/renderer_host/render_widget_host_view_views.cc
diff --git a/chrome/browser/renderer_host/render_widget_host_view_views.cc b/chrome/browser/renderer_host/render_widget_host_view_views.cc
index b9ebf768e239fcf1454f88401e7ac4b2cc58a862..61a06e40023609360bfc68388f250dce241b8fd8 100644
--- a/chrome/browser/renderer_host/render_widget_host_view_views.cc
+++ b/chrome/browser/renderer_host/render_widget_host_view_views.cc
@@ -289,12 +289,16 @@ void RenderWidgetHostViewViews::SetIsLoading(bool is_loading) {
}
void RenderWidgetHostViewViews::ImeUpdateTextInputState(
- WebKit::WebTextInputType type,
+ ui::TextInputType type,
+ bool can_compose_inline,
const gfx::Rect& caret_rect) {
+ // TODO(kinaba): currently, can_compose_inline is ignored and always treated
+ // as true. We need to support "can_compose_inline=false" for PPAPI plugins
+ // that may want to avoid drawing composition-text by themselves and pass
+ // the responsibility to the browser.
DCHECK(GetInputMethod());
- ui::TextInputType new_type = static_cast<ui::TextInputType>(type);
- if (text_input_type_ != new_type) {
- text_input_type_ = new_type;
+ if (text_input_type_ != type) {
+ text_input_type_ = type;
GetInputMethod()->OnTextInputTypeChanged(this);
}
if (caret_bounds_ != caret_rect) {

Powered by Google App Engine
This is Rietveld 408576698