[Webkit-unassigned] [Bug 84820] [BlackBerry] [DRT] Some HTTP tests can't access relative css and/or javascript

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Apr 25 02:34:54 PDT 2012


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





--- Comment #4 from Xiaobo Wang <xiaobo.wang at torchmobile.com.cn>  2012-04-25 02:34:54 PST ---
(In reply to comment #2)
> (From update of attachment 138757 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=138757&action=review
> 
> > Tools/DumpRenderTree/blackberry/DumpRenderTree.cpp:345
> > +    return test.lower().substring(0, strlen(httpTestSyntax)) == httpTestSyntax
> > +            && test.lower().substring(strlen(httpTestSyntax), strlen(localTestSyntax)) != localTestSyntax;
> 
> You could cache test.lower(), and the results of the strlen() calls in local variables. Though as this is a DRT patch, I'll r+ it nevertheless, feel free to make the changes, and reupload a patch or set cq+ on this one.

Thanks Nikolas, changes were made according to your comment and new patch was uploaded.

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