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

Unified Diff: ui/views/controls/textfield/native_textfield_views.cc

Issue 15841009: Delays find-in-page until IME composition is committed. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed a Paste operation. Created 7 years, 7 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: 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..574c6b4a7d11ec5badf8f848de947b88b3ab2c0c 100644
--- a/ui/views/controls/textfield/native_textfield_views.cc
+++ b/ui/views/controls/textfield/native_textfield_views.cc
@@ -1397,12 +1397,13 @@ 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)
+ TextfieldController* controller = textfield_->GetController();
+ if (controller) {
+ // 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)
controller->ContentsChanged(textfield_, textfield_->text());
+ controller->OnAfterPaste();
}
}
return success;

Powered by Google App Engine
This is Rietveld 408576698