[Webkit-unassigned] [Bug 89639] [EFL] Change format of return value of navigator.language

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Jun 24 21:25:01 PDT 2012


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





--- Comment #12 from Kihong Kwon <kihong.kwon at samsung.com>  2012-06-24 21:25:00 PST ---
(In reply to comment #11)
> (From update of attachment 149154 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=149154&action=review
> 
> > Source/WebCore/ChangeLog:8
> > +        Change format of return value of navigator.language from xx-XX.UTF-8 to xx-XX.
> 
> You need to explain why you are doing this. Please also note that 'UTF-8' is not hardcoded, it just happens to be the encoding you are using on your system. A more generic description would be along the lines of "Remove the encoding from the language returned by navigator.language".
> 
> The `language' attribute of the navigator object does not seem to be part of any standard, so different browsers seem to return different values anyway. Do you know if "en-US" is currently more common than "en", for example?
> 
I think there is no correct answer for this now.
But there is no browser displaying encoding with language.
IMHO, It is better to remove encoding.
xx(en) : Chrome(win/mac), Opera(all).
xx-XX(en-US) : Firefox(all), Chrome(linux), safari(win).
xx-xx(en-us) : safari(mac).

> > Source/WebCore/ChangeLog:9
> > +        Need a short description and bug URL (OOPS!)
> 
> Wrong line.
OK.

> 
> > Source/WebCore/ChangeLog:11
> > +        Test: platform/efl/fast/js/navigator-language.html
> 
> I object to copying this test from Qt; it was moved to platform/qt in r113892 because it apparently tested something which was relevant only to that port.

Did you mean we don't need to add this test case to the EFL?

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



More information about the webkit-unassigned mailing list