[Webkit-unassigned] [Bug 69455] Option.value should trim extra internal html spaces

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=69455


Kent Tamura <tkent at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #109848|review?                     |review-
               Flag|                            |




--- Comment #8 from Kent Tamura <tkent at chromium.org>  2011-10-13 23:35:36 PST ---
(From update of attachment 109848)
View in context: https://bugs.webkit.org/attachment.cgi?id=109848&action=review

> Source/WebCore/ChangeLog:13
> +        * dom/OptionElement.cpp: code restructure.

Please write what is changed and the reason.

> Source/WebCore/ChangeLog:16
> +        (WebCore::HTMLOptionElement::value): Fixed the issue

Please write what's changed.

> Source/WebCore/dom/OptionElement.h:44
> +    virtual String label() const = 0;

Why was this added?

> Source/WebCore/html/HTMLOptionElement.cpp:227
> -    String label = m_data.label();
> -    if (!label.isNull())
> -        return label;
> - 
> -    label = collectOptionInnerText(this).stripWhiteSpace(isHTMLSpace);
> -    label = label.simplifyWhiteSpace(isHTMLSpace);
> +    if (!m_data.label().isNull())
> +        return m_data.label();
>  
> -    return label;
> +    return collectOptionInnerText(this).stripWhiteSpace(isHTMLSpace).simplifyWhiteSpace(isHTMLSpace);

This part is not related to the bug.  Don't include this part in the patch.

> LayoutTests/fast/forms/option-value-trim-html-spaces.html:4
> +<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">

Why iso-8859-1? This files contains only ASCII characters.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.