583610 - Value sanitizing algorithm form <input type='url'>

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Mounir Lamouri (:mounir)]

Value sanitizing algorithm came with bug 549475
input type url is going to come with bug 344615

Comment 1

[Mounir Lamouri (:mounir)]

Attachment: Patch v1

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 461907 [details] [diff] [review]
Patch v1


>-  const nsAString& for_str = nsContentUtils::TrimWhitespace(forAttr);
>+  const nsAString& for_str =
>+    nsContentUtils::TrimWhitespace<nsCRT::IsAsciiSpace>(forAttr);

Could you file a followup bug to check whether all the places where TrimWhitespace<nsCRT::IsAsciiSpace>
is used, really should use it and not TrimWhitespace<nsContentUtils::IsHTMLWhitespace>.

Comment 3

[Mounir Lamouri (:mounir)]

This is now in the tree:
rev: http://hg.mozilla.org/mozilla-central/rev/03367932ad41