Summary: | WebKitTestRunner needs layoutTestController.pathToLocalResource | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Maciej Stachowiak <mjs> | ||||
Component: | WebKit2 | Assignee: | Chang Shu <cshu> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | abarth, commit-queue, cshu, eric, sam | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | PC | ||||||
OS: | OS X 10.5 | ||||||
Bug Depends on: | 58665 | ||||||
Bug Blocks: | 58826, 197965 | ||||||
Attachments: |
|
Description
Maciej Stachowiak
2010-07-18 21:44:05 PDT
quite some tests still fail due to console message issue in 58665 even the pathToLocalResource method is fixed. Created attachment 90032 [details]
fix patch
Comment on attachment 90032 [details] fix patch View in context: https://bugs.webkit.org/attachment.cgi?id=90032&action=review OK. > Tools/WebKitTestRunner/InjectedBundle/mac/LayoutTestControllerMac.mm:61 > + return JSStringRetain(url); // Do nothing on mac. That's because mac doesn't disable access to file: urls? > Tools/WebKitTestRunner/InjectedBundle/win/LayoutTestControllerWin.cpp:64 > + return JSStringRetain(url); // TODO. I think we normally use FIXME. Thanks for the quick review. > > Tools/WebKitTestRunner/InjectedBundle/mac/LayoutTestControllerMac.mm:61 > > + return JSStringRetain(url); // Do nothing on mac. > > That's because mac doesn't disable access to file: urls? This is what DRT does. I guess there's no need to do any transform to native platform on Mac. > > > Tools/WebKitTestRunner/InjectedBundle/win/LayoutTestControllerWin.cpp:64 > > + return JSStringRetain(url); // TODO. > > I think we normally use FIXME. Sure. Will fix. I didn't know it's already cq+. will keep in mind.
> Sure. Will fix.
Comment on attachment 90032 [details] fix patch Clearing flags on attachment: 90032 Committed r84160: <http://trac.webkit.org/changeset/84160> All reviewed patches have been landed. Closing bug. The commit-queue encountered the following flaky tests while processing attachment 90032 [details]: java/lc3/JSObject/ToObject-001.html bug 53091 (author: ap@webkit.org) http/tests/xmlhttprequest/re-login.html bug 51987 (author: ap@webkit.org) The commit-queue is continuing to process your patch. *** Bug 42678 has been marked as a duplicate of this bug. *** |