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

Issue 10957058: Merge 129095 - [Forms] multiple fields time input UI should call notifyFormStateChanged() when valu… (Closed)

Created:
8 years, 3 months ago by yosin_UTC9
Modified:
8 years, 3 months ago
Reviewers:
yosin_UTC9
CC:
chromium-reviews
Base URL:
http://svn.webkit.org/repository/webkit/branches/chromium/1271/
Visibility:
Public.

Description

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): TBR=yosin@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=129321

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M Source/WebCore/html/TimeInputType.cpp View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
yosin_UTC9
8 years, 3 months ago (2012-09-24 02:27:47 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698