Bug 207900 – Value sanitization for input[type=text] should not truncate a value at a control character

• Status: RESOLVED
• Resolution: FIXED
• Component: Forms

[Carlos Alberto Lopez Perez]

WebKit its truncating the value for input type text at the first control character that its non a tab.

This behaviour its not defined on the spec, neither match what other browsers do (Firefox, Chrome, etc).

This causes thousands of subtest failures on WPT tests encoding/legacy-mb-japanese/iso-2022-jp/iso2022jp-encode-form* <https://wpt.fyi/results/encoding/legacy-mb-japanese/iso-2022-jp>


Chrome corrected this behaviour years ago. For reference see: https://codereview.chromium.org/189843008

[Carlos Alberto Lopez Perez]

Created attachment 391070 [details]
Patch

[Carlos Alberto Lopez Perez]

(In reply to Carlos Alberto Lopez Perez from comment #0)
> WebKit its truncating the value for input type text at the first control
> character that its non a tab.
> 
> This behaviour its not defined on the spec, neither match what other
> browsers do (Firefox, Chrome, etc).
> 

Spec: https://html.spec.whatwg.org/multipage/input.html#text-(type=text)-state-and-search-state-(type=search):value-sanitization-algorithm

[Carlos Alberto Lopez Perez]

Created attachment 391152 [details]
Patch

Rebaseline also imported/w3c/web-platform-tests/html/semantics/forms/the-input-element/valueMode.html with more PASS lines

[Carlos Alberto Lopez Perez]

Created attachment 391196 [details]
Patch

Fix wrong baseline for ios in imported/w3c/web-platform-tests/html/semantics/forms/the-input-element/valueMode.html

[Carlos Alberto Lopez Perez]

r? All EWS are green now.

[Carlos Alberto Lopez Perez]

Comment on attachment 391196 [details]
Patch

Clearing flags on attachment: 391196

Committed r257132: <https://trac.webkit.org/changeset/257132>

[Carlos Alberto Lopez Perez]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/59664131>