Title: [129321] branches/chromium/1271/Source/WebCore/html/TimeInputType.cpp
Revision
129321
Author
[email protected]
Date
2012-09-23 19:29:08 -0700 (Sun, 23 Sep 2012)

Log Message

Merge 129095 - [Forms] multiple fields time input UI should call notifyFormStateChanged() when value of field is changed
https://bugs.webkit.org/show_bug.cgi?id=97169

Reviewed by Kent Tamura.

This patch makes multiple fields time input UI calls notifyFormStateChanged()
when field value is changed as other input types do.

This patch affects ports which enable both ENABLE_INPUT_TYPE_TIME and
ENABLE_INPUT_TYPE_TIME_MULTIPLE_FIELDS.

No new tests. We can't test this change in WebKit test tools. Test script
will be implemented in Chromium side.

* html/TimeInputType.cpp:
(WebCore::TimeInputType::DateTimeEditControlOwnerImpl::editControlValueChanged):

[email protected]
Review URL: https://codereview.chromium.org/10957058

Modified Paths

Diff

Modified: branches/chromium/1271/Source/WebCore/html/TimeInputType.cpp (129320 => 129321)


--- branches/chromium/1271/Source/WebCore/html/TimeInputType.cpp	2012-09-24 01:16:02 UTC (rev 129320)
+++ branches/chromium/1271/Source/WebCore/html/TimeInputType.cpp	2012-09-24 02:29:08 UTC (rev 129321)
@@ -152,6 +152,7 @@
     input->setNeedsStyleRecalc();
     input->dispatchFormControlInputEvent();
     input->dispatchFormControlChangeEvent();
+    input->notifyFormStateChanged();
 }
 
 bool TimeInputType::hasCustomFocusLogic() const
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to