Chromium Code Reviews

Issue 101573002: Remove TreatNullAs=NullString for HTMLSelectElement (Closed)

Created:
7 years ago by philipj_slow
Modified:
7 years ago
Reviewers:
jochen (gone - plz use gerrit)
CC:
blink-reviews, adamk+blink_chromium.org, dglazkov+blink, arv+blink, Inactive, watchdog-blink-watchlist_google.com
Visibility:
Public.

Description

Remove TreatNullAs=NullString for HTMLSelectElement http://whatwg.org/html#htmlselectelement The value attribute returns "" when there is no selected child <option> element, so add another test to verify that null stringifies to "null" on setting. The tests for the updated properties pass in Firefox Nightly, IE11 Release Preview and Opera 12.16. BUG=310298 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=163087

Patch Set 1 #

Unified diffs Side-by-side diffs Stats (+33 lines, -7 lines)
M LayoutTests/fast/dom/domstring-attribute-reflection.html View 1 chunk +1 line, -1 line 0 comments
M LayoutTests/fast/dom/domstring-attribute-reflection-expected.txt View 1 chunk +2 lines, -2 lines 0 comments
M LayoutTests/fast/dom/element-attribute-js-null.html View 1 chunk +1 line, -1 line 0 comments
M LayoutTests/fast/dom/element-attribute-js-null-expected.txt View 1 chunk +1 line, -1 line 0 comments
A LayoutTests/fast/forms/select-value-null.html View 1 chunk +22 lines, -0 lines 0 comments
A LayoutTests/fast/forms/select-value-null-expected.txt View 1 chunk +4 lines, -0 lines 0 comments
M Source/core/html/HTMLSelectElement.idl View 2 chunks +2 lines, -2 lines 0 comments

Messages

Total messages: 4 (0 generated)
philipj_slow
7 years ago (2013-12-03 11:56:16 UTC) #1
jochen (gone - plz use gerrit)
lgtm
7 years ago (2013-12-03 11:56:54 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/philipj@opera.com/101573002/1
7 years ago (2013-12-03 11:58:07 UTC) #3
commit-bot: I haz the power
7 years ago (2013-12-03 14:14:00 UTC) #4
Message was sent while issue was closed.
Change committed as 163087

Powered by Google App Engine