Delays find-in-page until IME composition is committed.

ui/views/controls/textfield/native_textfield_views.cc

Index: ui/views/controls/textfield/native_textfield_views.cc
diff --git a/ui/views/controls/textfield/native_textfield_views.cc b/ui/views/controls/textfield/native_textfield_views.cc
index 5da334dcef5f317254de410d2139666b4a533ce1..28fdba588db3fcdb667ff87bdf7044d53066ae1a 100644
--- a/ui/views/controls/textfield/native_textfield_views.cc
+++ b/ui/views/controls/textfield/native_textfield_views.cc
@@ -1397,13 +1397,9 @@ bool NativeTextfieldViews::Paste() {
     string16 new_text = GetTextForDisplay(GetText());
     model_->SetText(new_text);
 
-    // Calls TextfieldController::ContentsChanged() explicitly if the paste
-    // action did not change the content at all. See http://crbug.com/79002
-    if (new_text == original_text) {
-      TextfieldController* controller = textfield_->GetController();
-      if (controller)
-        controller->ContentsChanged(textfield_, textfield_->text());
-    }
+    TextfieldController* controller = textfield_->GetController();

[msw, 2013/06/04 01:30:23]

nit: you can remove |controller|, or use assignment in the conditional if you
want; strive for consistency in your own code and with surrounding code.

[Yuki, 2013/06/04 03:35:04]

Thanks for the comment.
I'm not sure about our policy, and I wanted to keep consistency in each file,
not consistency on my side.  native_textfield_views.cc is written in this style
(separate assignment to |controller|), and native_textfield_win.cc is written in
the style of "assignment in conditional".  I thought it's better to keep the
consistency in each file.

+    if (controller)
+      controller->OnAfterPaste();
   }
   return success;
 }