Chromium Code Reviews| Index: content/renderer/render_widget.cc |
| diff --git a/content/renderer/render_widget.cc b/content/renderer/render_widget.cc |
| index e4d75c0cd215d80c1b846121f808e397cb31bef6..1e608aff31d4781e3ab81d7cb17c684b69c11271 100644 |
| --- a/content/renderer/render_widget.cc |
| +++ b/content/renderer/render_widget.cc |
| @@ -1126,9 +1126,17 @@ void RenderWidget::OnImeSetComposition( |
| } |
| } |
| -void RenderWidget::OnImeConfirmComposition(const string16& text) { |
| - if (webwidget_) |
| +void RenderWidget::OnImeConfirmComposition( |
| + const string16& text, const ui::Range& replacement_range) { |
| + if (webwidget_) { |
| + // TODO(thakis): Uncomment this once |
| + // https://bugs.webkit.org/show_bug.cgi?id=69846 has landed. |
|
jamesr
2011/10/12 00:34:51
A better way to do this is to guard this code with
|
| + // if (replacement_range.IsValid()) { |
| + // webwidget_->setSelectionOnFocusedFrame(replacement_range.start(), |
| + // replacement_range.end()); |
| + // } |
| webwidget_->confirmComposition(text); |
| + } |
| // Send an updated IME range with just the caret range. |
| ui::Range range(ui::Range::InvalidRange()); |
| size_t location, length; |