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

Unified Diff: Source/web/tests/WebViewTest.cpp

Issue 133443011: Add text field change handling for autofill preview in textarea element. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Update code as per tkent's comments Created 6 years, 10 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: Source/web/tests/WebViewTest.cpp
diff --git a/Source/web/tests/WebViewTest.cpp b/Source/web/tests/WebViewTest.cpp
index 17f20ee5eb6c2677056f97ff6d0531ef7e65ae74..ef865ae3e33d5302533c9ff74e0069cd01eb7f4e 100644
--- a/Source/web/tests/WebViewTest.cpp
+++ b/Source/web/tests/WebViewTest.cpp
@@ -1129,6 +1129,8 @@ public:
virtual ~MockAutofillClient() { }
virtual void setIgnoreTextChanges(bool ignore) OVERRIDE { m_ignoreTextChanges = ignore; }
+ // TODO(ziran.sun): This function is to be removed once both chromium and blink changes
tkent 2014/03/05 23:18:39 TODO() -> FIXME
+ // for BUG332557 are in.
virtual void textFieldDidChange(const WebInputElement&) OVERRIDE
{
if (m_ignoreTextChanges)
@@ -1136,6 +1138,13 @@ public:
else
++m_textChangesWhileNotIgnored;
}
+ virtual void textFieldDidChange(const WebFormControlElement&) OVERRIDE
+ {
+ if (m_ignoreTextChanges)
+ ++m_textChangesWhileIgnored;
+ else
+ ++m_textChangesWhileNotIgnored;
+ }
void clearChangeCounts()
{

Powered by Google App Engine
This is Rietveld 408576698