Changeset 129095 in webkit

Timestamp:

Sep 19, 2012 11:13:37 PM (12 years ago)

Author:

yosin@chromium.org

Message:

[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):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• html/TimeInputType.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-09-19  Yoshifumi Inoue  <yosin@chromium.org>
+
+        [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):
+
 2012-09-19  Keishi Hattori  <keishi@webkit.org>
 

trunk/Source/WebCore/html/TimeInputType.cpp

@@ -153 +153 @@
     input->dispatchFormControlInputEvent();
     input->dispatchFormControlChangeEvent();
+    input->notifyFormStateChanged();
 }