Support IMM32 reconversion on Windows

views/controls/textfield/native_textfield_views.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "views/controls/textfield/native_textfield_views.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/command_line.h"
@@ skipping 773 matching lines @@
   if (model_->HasSelection()) {
     model_->DeleteSelection();
     UpdateAfterChange(true, true);
   }
   OnAfterUserAction();
   return true;
 }
 
 bool NativeTextfieldViews::GetTextFromRange(
     const ui::Range& range,
-    const base::Callback<void(const string16&)>& callback) {
-  if (GetTextInputType() != ui::TEXT_INPUT_TYPE_TEXT || range.is_empty())
+    string16* text) {
+  if (GetTextInputType() != ui::TEXT_INPUT_TYPE_TEXT || !range.IsValid())
     return false;
 
-  callback.Run(model_->GetTextFromRange(range));
+  *text = model_->GetTextFromRange(range);

[James Su, 2011/10/28 04:20:45]

You need to make sure the range is inside the content range and return false if
it's not.

[Peng, 2011/10/28 15:42:19]

Done.

   return true;
 }
 
 void NativeTextfieldViews::OnInputMethodChanged() {
   NOTIMPLEMENTED();
 }
 
 bool NativeTextfieldViews::ChangeTextDirectionAndLayoutAlignment(
     base::i18n::TextDirection direction) {
   NOTIMPLEMENTED();
@@ skipping 265 matching lines @@
 
 #if defined(USE_AURA)
 // static
 NativeTextfieldWrapper* NativeTextfieldWrapper::CreateWrapper(
     Textfield* field) {
   return new NativeTextfieldViews(field);
 }
 #endif
 
 }  // namespace views