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

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

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.

More Reviews

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 #

Created: 8 years, 3 months ago

Download [raw] [tar.bz2]

		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)

Expand Messages | Collapse Messages