WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
56884
Add an option to new-run-webkit-httpd to allow overriding the location of LayoutTests directory
https://bugs.webkit.org/show_bug.cgi?id=56884
Summary
Add an option to new-run-webkit-httpd to allow overriding the location of Lay...
Adam Klein
Reported
2011-03-22 17:21:30 PDT
Add an option to new-run-webkit-httpd to allow overriding the location of LayoutTests directory
Attachments
Patch
(3.88 KB, patch)
2011-03-22 17:23 PDT
,
Adam Klein
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Adam Klein
Comment 1
2011-03-22 17:23:17 PDT
Created
attachment 86550
[details]
Patch
Mihai Parparita
Comment 2
2011-03-22 17:28:34 PDT
Comment on
attachment 86550
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=86550&action=review
> Tools/ChangeLog:9 > + referencing js-test-resources from a layout test over HTTP.
I'm not entirely sure I understand the use-case here, but if you'd like to remap what directories are available at what location when using new-run-webkit-tests's HTTP server, you can use aliases the way
http://trac.webkit.org/changeset/79630
works (see the Tools/Scripts changes).
Adam Klein
Comment 3
2011-03-22 17:33:07 PDT
Comment on
attachment 86550
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=86550&action=review
>> Tools/ChangeLog:9 >> + referencing js-test-resources from a layout test over HTTP. > > I'm not entirely sure I understand the use-case here, but if you'd like to remap what directories are available at what location when using new-run-webkit-tests's HTTP server, you can use aliases the way
http://trac.webkit.org/changeset/79630
works (see the Tools/Scripts changes).
So, the remapping I want is already there (/js-test-resources -> fast/js/resources), the trick is that these tests are running in Chromium's ui_tests against a limited set of LayoutTests under chrome/test/data/layout_tests. But port.layout_tests_dir() returns third_party/WebKit/LayoutTests, which on many folks' checkouts (including the try bots) is empty. See
http://codereview.chromium.org/6727001/diff/2005/chrome/test/ui/ui_test.cc
for how I plan to use this flag.
WebKit Commit Bot
Comment 4
2011-03-23 18:13:54 PDT
Comment on
attachment 86550
[details]
Patch Clearing flags on attachment: 86550 Committed
r81834
: <
http://trac.webkit.org/changeset/81834
>
WebKit Commit Bot
Comment 5
2011-03-23 18:13:59 PDT
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug