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

Unified Diff: Source/core/html/forms/TextFieldInputType.cpp

Issue 128133002: Update of change event for input type number (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 years, 11 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/core/html/forms/TextFieldInputType.cpp
diff --git a/Source/core/html/forms/TextFieldInputType.cpp b/Source/core/html/forms/TextFieldInputType.cpp
index 807f2623d1cd28b3bfbb431cd43d1060fbc4538a..0550c6eb73fafc0a50f659c1f247a9887a098a79 100644
--- a/Source/core/html/forms/TextFieldInputType.cpp
+++ b/Source/core/html/forms/TextFieldInputType.cpp
@@ -178,6 +178,7 @@ void TextFieldInputType::setValue(const String& sanitizedValue, bool valueChange
switch (eventBehavior) {
case DispatchChangeEvent:
+ case DispatchInputAndChangeEvent:
tkent 2014/01/14 01:06:46 This is not acceptable as I already wrote.
// If the user is still editing this field, dispatch an input event rather than a change event.
// The change event will be dispatched when editing finishes.
if (input->focused())
@@ -185,13 +186,6 @@ void TextFieldInputType::setValue(const String& sanitizedValue, bool valueChange
else
input->dispatchFormControlChangeEvent();
break;
-
- case DispatchInputAndChangeEvent: {
- input->dispatchFormControlInputEvent();
- input->dispatchFormControlChangeEvent();
- break;
- }
-
case DispatchNoEvent:
break;
}
« no previous file with comments | « LayoutTests/fast/forms/number/number-type-update-by-change-event.html ('k') | Source/core/html/shadow/SpinButtonElement.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698