[Webkit-unassigned] [Bug 90261] Move platform/qt/fast/js/navigator-language.html to fast/js.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Jun 29 18:45:26 PDT 2012


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





--- Comment #7 from Kihong Kwon <kihong.kwon at samsung.com>  2012-06-29 18:45:24 PST ---
(In reply to comment #5)
> (From update of attachment 150154 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=150154&action=review
> 
> > LayoutTests/fast/js/navigator-language.html:10
> > +    <head>
> > +        <script src="../../fast/js/resources/js-test-pre.js"></script>
> > +    </head>
> > +    <body>
> > +        <script>
> > +            description(
> > +                "<pre>Check return value of navigator.language."
> > +            );
> 
> Why are you indenting elements and scripts like this? We don't normally do this.
> Also, please use svn mv.
> 
> > LayoutTests/fast/js/navigator-language.html:18
> > +				shouldBe("language", "'en'");
> 
> We use 4-space indentation, not tabs.

It was my mistake.
They were fixed.

-- 
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