16472 2012-03-21 20:45:38 +0000 Consider DOMString? for HTMLInputElement:value IDL attribute 2012-10-14 12:29:50 +0000 1 1 1 Unclassified HTML WG HTML5 spec unspecified PC Windows NT RESOLVED FIXED P2 normal --- 1 travil silviapfeiffer1 annevk bzbarsky cam jonas mike public-html-admin public-html-wg-issue-tracking robin silviapfeiffer1 public-html-bugzilla oldest_to_newest 65912 0 1105 travil 2012-03-21 20:45:38 +0000 Created attachment 1105 Test case for null assignment on common DOMString reflecting values I just ran into a compatibility issue in IE10 (on site 58.com), where the site ends up assigning the null literal value to the input element's value IDL attribute. As far as I can tell, IE10 is currently doing the "correct" behavior according to the current definition of the value IDL attribute and WebIDL conversion logic: <blockquote> The value IDL attribute allows scripts to manipulate the value of an input element. The attribute is in one of the following modes, which define its behavior: * value On getting, it must return the current value of the element. On setting, it must set the element's value to the new value, set the element's dirty value flag to true, invoke the value sanitization algorithm, if the element's type attribute's current state defines one, and then, if the element has a text entry cursor position, should move the text entry cursor position to the end of the text field, unselecting any selected text and resetting the selection direction to none. </blockquote> (From HTML5 4.10.7.4) On the site, the input is type=text, which means the value attribute is in the "value" mode. The value sanitization algorithm for that state just trims line breaks from the value. Since, the HTMLInputElement interface WebIDL is defined as following: interface HTMLInputElement : HTMLElement { [snip] attribute DOMString value; [snip] }; Then the conversion of a null literal value should result in the string "null" being assigned per 4.2.16. of WebIDL. IE10 places the string "null" in the value of the input type=text field. Unfortunately, the site does not want that to happen, instead, they expect the value to be the empty string so that the placeholder text is put in the input box instead. At a minimum, I'd recommending changing the value IDL attribute to either: [TreatNullAs=EmptyString] attribute DOMString value; or attribute DOMString? value; (and providing a description of how this IDL attribute handles null (e.g., by setting the value to the empty string) ... to ensure good interop and avoid site issues like the above-specified one. I did some testing of other reflecting DOM attributes (on DIV and on HTMLInputElement -- see attached) under this same scenario, and IE is pretty consistent about assigning the string "null" in all these cases (going as far back as IE8--likely back to IE6, but I didn't test that). While IE appears to have the correct behavior in these scenarios per the spec(and this behavior has not changed in some time), I wonder if this is the best behavior. According to the attached test, the other browsers aren't perfectly aligned on an alternate behavior, but none of them match IE at the moment. Interestingly, WebKit based browsers tend to clear the content attribute when null is assigned, whereas Firefox/Opera assign the empty string to the content attribute. This _does_ make a difference with respect to some attributes like "title" where the presence of the attribute (vs. an empty string value) make a difference. 65983 1 bzbarsky 2012-03-23 14:08:54 +0000 I'd much prefer the TreatNullAs approach to the prose approach, for what it's worth, as an implementor. 65984 2 zcorpan 2012-03-23 14:15:05 +0000 Since .value doesn't reflect the content attribute, I think it makes little sense to remove the content attribute when setting to null. Setting concept-value to the empty string seems OK. 68500 3 zcorpan 2012-06-01 11:43:04 +0000 (In reply to comment #2) > Since .value doesn't reflect the content attribute, I think it makes little > sense to remove the content attribute when setting to null. Setting > concept-value to the empty string seems OK. However, removing the content attribute when setting .defaultValue and other reflecting IDL attributes seems reasonable; https://www.w3.org/Bugs/Public/show_bug.cgi?id=17283 70139 4 contributor 2012-07-18 06:58:45 +0000 This bug was cloned to create bug 17834 as part of operation convergence. 76239 5 silviapfeiffer1 2012-10-14 12:29:50 +0000 EDITOR'S RESPONSE: This is an Editor's Response to your comment. If you are satisfied with this response, please change the state of this bug to CLOSED. If you have additional information and would like the Editor to reconsider, please reopen this bug. If you would like to escalate the issue to the full HTML Working Group, please add the TrackerRequest keyword to this bug, and suggest title and text for the Tracker Issue; or you may create a Tracker Issue yourself, if you are able to do so. For more details, see this document: http://dev.w3.org/html5/decision-policy/decision-policy-v2.html Status: Accepted Change Description: https://github.com/w3c/html/commit/827f02df82e56c91ec99f345f5dcbb5fcd088347 Rationale: accepted WHATWG change 1105 2012-03-21 20:45:38 +0000 2012-03-21 20:45:38 +0000 Test case for null assignment on common DOMString reflecting values assign_null_to_IDL_attributes.htm text/html 4800 travil 77u/PCFET0NUWVBFIGh0bWw+DQoNCjxodG1sIGxhbmc9ImVuIiB4bWxucz0iaHR0cDovL3d3dy53 My5vcmcvMTk5OS94aHRtbCI+DQo8aGVhZD4NCiAgICA8bWV0YSBjaGFyc2V0PSJ1dGYtOCIgLz4N CiAgICA8dGl0bGU+PC90aXRsZT4NCiAgICA8c3R5bGU+DQogICAgICAgIGJvZHkgew0KICAgICAg ICAgICAgZm9udC1mYW1pbHk6ICdTZWdvZSBVSSc7DQogICAgICAgIH0NCiAgICAgICAgdGggew0K ICAgICAgICAgICAgYmFja2dyb3VuZC1jb2xvcjogIzAwOTJmZjsNCiAgICAgICAgICAgIGNvbG9y OiB3aGl0ZTsNCiAgICAgICAgICAgIGZvbnQtc2l6ZTogbGFyZ2VyOw0KICAgICAgICB9DQogICAg ICAgIHRkLCB0aCB7DQogICAgICAgICAgICBwYWRkaW5nOiA1cHg7DQogICAgICAgIH0NCiAgICAg ICAgdHI6bnRoLWNoaWxkKG9kZCkgew0KICAgICAgICAgICAgYmFja2dyb3VuZC1jb2xvcjogI2Jk ZTNmZjsNCiAgICAgICAgfQ0KICAgICAgICB0ZDpudGgtY2hpbGQoMiksIHRoOm50aC1jaGlsZCgy KSB7DQogICAgICAgICAgICBib3JkZXItbGVmdDogIDRweCBkb3VibGUgYmxhY2s7DQogICAgICAg IH0NCiAgICAgICAgdGQ6bnRoLWNoaWxkKDQpLCB0aDpudGgtY2hpbGQoNCkgew0KICAgICAgICAg ICAgYm9yZGVyLWxlZnQ6IDJweCBzb2xpZCBibGFjazsNCiAgICAgICAgfQ0KICAgIDwvc3R5bGU+ DQo8L2hlYWQ+DQo8Ym9keT4NCiAgICA8ZGl2IGRhdGEtZmluZC1kaXY9IiIgaWQ9InRlc3QiIGNs YXNzPSJ0ZXN0IiB0aXRsZT0idGVzdCIgbGFuZz0idGVzdCIgZGlyPSJ0ZXN0Ij48L2Rpdj4NCiAg ICA8aW5wdXQgZGF0YS1maW5kLWlucHV0PSIiIGFjY2VwdD0idGVzdCIgYWx0PSJ0ZXN0IiBtYXg9 InRlc3QiIG1pbj0idGVzdCIgbXVsdGlwbGU9InRlc3QiIHBhdHRlcm49InRlc3QiIHBsYWNlaG9s ZGVyPSJ0ZXN0IiByZXF1aXJlZD0idGVzdCIgc2l6ZT0idGVzdCIgc3JjPSJ0ZXN0IiBzdGVwPSJ0 ZXN0IiBkaXJuYW1lPSJ0ZXN0IiByZWFkb25seT0idGVzdCIgY2hlY2tlZD0idGVzdCIgdmFsdWU9 InRlc3QiIC8+DQogICAgPGJ1dHRvbiBvbmNsaWNrPSJydW5UZXN0KCkiPlNldCBudWxsIHRvIElE TCBhdHRyaWJ1dGVzIG9uIHRoZSBkaXYgYW5kIGNoZWNrIHJlc3VsdHM8L2J1dHRvbj4NCiAgICA8 aDE+VGVzdGluZyByZWZsZWN0aW5nIERPTVN0cmluZyBJREwgYXR0cmlidXRlcyB3aGVuIGFzc2ln bmVkIHRoZSBudWxsIGxpdGVyYWwgdmFsdWU8L2gxPg0KICAgIDxwPlByZS10ZXN0aW5nIHNldHVw IGZhaWx1cmVzOjwvcD4NCiAgICA8b2wgaWQ9InByZVRlc3RGYWlsdXJlcyI+DQogICAgPC9vbD4N CiAgICA8cD5UZXN0IHJlc3VsdHMgKGV4Y2x1ZGluZyBmYWlsdXJlcyk6PC9wPg0KICAgIDx0YWJs ZSBzdHlsZT0iYm9yZGVyLWNvbGxhcHNlOiBjb2xsYXBzZSI+DQogICAgICAgIDx0cj4NCiAgICAg ICAgICAgIDx0aD5FbGVtZW50PC90aD4NCiAgICAgICAgICAgIDx0aD5JREwgYXR0cmlidXRlPC90 aD4NCiAgICAgICAgICAgIDx0aD5JREwgYXR0cmlidXRlIHZhbHVlIChhZnRlciBudWxsIGFzaWdu bWVudCk8L3RoPg0KICAgICAgICAgICAgPHRoPmNvbnRlbnQgYXR0cmlidXRlPC90aD4NCiAgICAg ICAgICAgIDx0aD5jb250ZW50IGF0dHJpYnV0ZSB2YWx1ZSAoYWZ0ZXIgbnVsbCBhc3NpZ25tZW50 KTwvdGg+DQogICAgICAgIDwvdHI+DQogICAgICAgIDx0ciBpZD0iY2xvbmUiIHN0eWxlPSJkaXNw bGF5Om5vbmUiPg0KICAgICAgICAgICAgPHRkPjwvdGQ+DQogICAgICAgICAgICA8dGQ+PC90ZD4N CiAgICAgICAgICAgIDx0ZD48L3RkPg0KICAgICAgICAgICAgPHRkPjwvdGQ+DQogICAgICAgICAg ICA8dGQ+PC90ZD4NCiAgICAgICAgPC90cj4NCiAgICA8L3RhYmxlPg0KICAgIDxzY3JpcHQ+DQoN CiAgICAgICAgdmFyIGlkbHMgPSBbImlkIiwgImNsYXNzTmFtZSIsICJ0aXRsZSIsICJsYW5nIiwg ImRpciIsICJhY2Nlc3NLZXkiXTsNCiAgICAgICAgdmFyIGNvbnQgPSBbImlkIiwgImNsYXNzIiwg InRpdGxlIiwgImxhbmciLCAiZGlyIiwgImFjY2Vza2V5Il07DQogICAgICAgIHZhciBza2lwcyA9 IHt9Ow0KDQogICAgICAgIHZhciBpbnB1dElETHMgPSBbImFjY2VwdCIsICJhbHQiLCAibWF4Iiwg Im1pbiIsICJwYXR0ZXJuIiwgInBsYWNlaG9sZGVyIiwgInNyYyIsICJzdGVwIiwgImRpck5hbWUi LCAiZGVmYXVsdFZhbHVlIl07DQogICAgICAgIHZhciBpbnB1dENvbnQgPSBbImFjY2VwdCIsICJh bHQiLCAibWF4IiwgIm1pbiIsICJwYXR0ZXJuIiwgInBsYWNlaG9sZGVyIiwgInNyYyIsICJzdGVw IiwgImRpcm5hbWUiLCAidmFsdWUiXTsNCiAgICAgICAgdmFyIGlucHV0U2tpcHMgPSB7fTsNCg0K ICAgICAgICBmdW5jdGlvbiBydW5UZXN0KCkgew0KICAgICAgICAgICAgdmFyIGRpdiA9IGRvY3Vt ZW50LnF1ZXJ5U2VsZWN0b3IoIltkYXRhLWZpbmQtZGl2XSIpOw0KICAgICAgICAgICAgdmFyIGlu cHV0ID0gZG9jdW1lbnQucXVlcnlTZWxlY3RvcigiW2RhdGEtZmluZC1pbnB1dF0iKTsNCg0KICAg ICAgICAgICAgYXNzaWduTnVsbFRvSURMQXR0cmlidXRlcyhpZGxzLCBkaXYsIHNraXBzKTsNCiAg ICAgICAgICAgIG91dHB1dFZhbHVlcyhpZGxzLCBjb250LCBkaXYsIHNraXBzKTsNCg0KICAgICAg ICAgICAgYXNzaWduTnVsbFRvSURMQXR0cmlidXRlcyhpbnB1dElETHMsIGlucHV0LCBpbnB1dFNr aXBzKTsNCiAgICAgICAgICAgIG91dHB1dFZhbHVlcyhpbnB1dElETHMsIGlucHV0Q29udCwgaW5w dXQsIGlucHV0U2tpcHMpOw0KICAgICAgICB9DQoNCiAgICAgICAgZnVuY3Rpb24gYXNzaWduTnVs bFRvSURMQXR0cmlidXRlcyhJRExBdHRycywgZWxlbWVudCwgc2tpcE9iKSB7DQogICAgICAgICAg ICBmb3IgKHZhciBpID0gMDsgaSA8IElETEF0dHJzLmxlbmd0aDsgaSsrKSB7DQogICAgICAgICAg ICAgICAgdHJ5IHsNCiAgICAgICAgICAgICAgICAgICAgaWYgKCEoSURMQXR0cnNbaV0gaW4gZWxl bWVudCkpIHsNCiAgICAgICAgICAgICAgICAgICAgICAgIG91dHB1dFNldHVwRmFpbHVyZXMoIlNr aXBwaW5nIG51bGwgYXNzaWdubWVudCB0byBwcm9wZXJ0eSAnIiArIElETEF0dHJzW2ldICsgIicg YmVjYXVzZSBpdCBpcyBub3Qgc3VwcG9ydGVkIG9uIHRoZSAiICsgZWxlbWVudCk7DQogICAgICAg ICAgICAgICAgICAgICAgICBza2lwT2JbaV0gPSB0cnVlOw0KICAgICAgICAgICAgICAgICAgICB9 DQogICAgICAgICAgICAgICAgICAgIGVsc2UNCiAgICAgICAgICAgICAgICAgICAgICAgIGVsZW1l bnRbSURMQXR0cnNbaV1dID0gbnVsbDsNCiAgICAgICAgICAgICAgICB9DQogICAgICAgICAgICAg ICAgY2F0Y2ggKGV4KSB7DQogICAgICAgICAgICAgICAgICAgIG91dHB1dFNldHVwRmFpbHVyZXMo IkV4Y2VwdGlvbiByYWlzZWQgb24gbnVsbCBhc2lnbm1lbnQgdG8gcHJvcGVydHkgJyIgKyBJRExB dHRyc1tpXSArICInIG9uIHRoZSAiICsgZWxlbWVudCk7DQogICAgICAgICAgICAgICAgICAgIHNr aXBPYltpXSA9IHRydWU7DQogICAgICAgICAgICAgICAgfQ0KICAgICAgICAgICAgfQ0KICAgICAg ICB9DQoNCiAgICAgICAgZnVuY3Rpb24gb3V0cHV0U2V0dXBGYWlsdXJlcyhtc2cpIHsNCiAgICAg ICAgICAgIHZhciB0ZXN0RmFpbCA9IGRvY3VtZW50LmNyZWF0ZUVsZW1lbnQoJ2xpJyk7DQogICAg ICAgICAgICB0ZXN0RmFpbC5pbm5lckhUTUwgPSBtc2c7DQogICAgICAgICAgICBkb2N1bWVudC5n ZXRFbGVtZW50QnlJZCgicHJlVGVzdEZhaWx1cmVzIikuYXBwZW5kQ2hpbGQodGVzdEZhaWwpOw0K ICAgICAgICB9DQoNCiAgICAgICAgZnVuY3Rpb24gb3V0cHV0VmFsdWVzKElETEF0dHJzLCBjb250 ZW50QXR0cnMsIGVsZW1lbnQsIHNraXBzKSB7DQogICAgICAgICAgICBmb3IgKHZhciBpID0gMDsg aSA8IElETEF0dHJzLmxlbmd0aDsgaSsrKSB7DQogICAgICAgICAgICAgICAgaWYgKHNraXBzW2ld KQ0KICAgICAgICAgICAgICAgICAgICBjb250aW51ZTsNCiAgICAgICAgICAgICAgICB2YXIgY2xv bmVUZW1wbGF0ZSA9IGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJjbG9uZSIpOw0KICAgICAgICAg ICAgICAgIHZhciBjbG9uZSA9IGNsb25lVGVtcGxhdGUuY2xvbmVOb2RlKHRydWUpOw0KICAgICAg ICAgICAgICAgIGNsb25lLnJlbW92ZUF0dHJpYnV0ZSgic3R5bGUiKTsNCiAgICAgICAgICAgICAg ICBjbG9uZS5yZW1vdmVBdHRyaWJ1dGUoImlkIik7DQogICAgICAgICAgICAgICAgY2xvbmUuY2hp bGRyZW5bMF0uaW5uZXJIVE1MID0gZWxlbWVudDsNCiAgICAgICAgICAgICAgICBjbG9uZS5jaGls ZHJlblsxXS5pbm5lckhUTUwgPSBJRExBdHRyc1tpXTsNCiAgICAgICAgICAgICAgICBjbG9uZS5j aGlsZHJlblsyXS5pbm5lckhUTUwgPSAoKGVsZW1lbnRbSURMQXR0cnNbaV1dID09PSBudWxsKSA/ ICItLW51bGwgbGl0ZXJhbC0tIiA6ICInIiArIGVsZW1lbnRbSURMQXR0cnNbaV1dKSArICInIjsN CiAgICAgICAgICAgICAgICBjbG9uZS5jaGlsZHJlblszXS5pbm5lckhUTUwgPSBjb250ZW50QXR0 cnNbaV07DQogICAgICAgICAgICAgICAgY2xvbmUuY2hpbGRyZW5bNF0uaW5uZXJIVE1MID0gKGVs ZW1lbnQuaGFzQXR0cmlidXRlKGNvbnRlbnRBdHRyc1tpXSkgPyAiJyIgKyBlbGVtZW50LmdldEF0 dHJpYnV0ZShjb250ZW50QXR0cnNbaV0pICsgIiciIDogIi0tYXR0cmlidXRlIG5vdCBzZXQtLSIp Ow0KICAgICAgICAgICAgICAgIGNsb25lVGVtcGxhdGUucGFyZW50Tm9kZS5hcHBlbmRDaGlsZChj bG9uZSk7DQogICAgICAgICAgICB9DQogICAgICAgIH0NCg0KICAgIDwvc2NyaXB0Pg0KDQo8L2Jv ZHk+DQo8L2h0bWw+