Bug 72813 - Remove the dependence on jsonp from more of new-run-webkit-tests and the test results server
Summary: Remove the dependence on jsonp from more of new-run-webkit-tests and the test...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ojan Vafai
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-19 16:47 PST by Ojan Vafai
Modified: 2011-11-19 17:18 PST (History)
2 users (show)

See Also:

Attachments
Patch (6.25 KB, patch)
2011-11-19 16:50 PST, Ojan Vafai 		abarth: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ojan Vafai 2011-11-19 16:47:30 PST 
Remove the dependence on jsonp from more of new-run-webkit-tests and the test results server
Comment 1 Ojan Vafai 2011-11-19 16:50:47 PST 
Created attachment 115963 [details]
Patch
Comment 2 Adam Barth 2011-11-19 16:57:28 PST 
Comment on attachment 115963 [details]
Patch

Do we need to change anything in garden-o-matic?  It has some code to strip off the JSONP wrapper.  We might need to change it to detect whether the wrapper is there in the first place (or maybe it already does that).
Comment 3 Ojan Vafai 2011-11-19 17:00:15 PST 
Comment on attachment 115963 [details]
Patch

I'll just commit this now.
Comment 4 Ojan Vafai 2011-11-19 17:00:48 PST 
I think garden-o-matic is OK, but I'll double-check before commiting.
Comment 5 Ojan Vafai 2011-11-19 17:03:54 PST 
As best I can tell, garden-o-matic only reads full_results.json, which is the only one that's not changing (i.e. it's keeping the jsonp wrapper).
Comment 6 Adam Barth 2011-11-19 17:09:25 PST 
Ok.  Should we remove the wrapper from that one too?
Comment 7 Ojan Vafai 2011-11-19 17:13:34 PST 
Committed r100862: <http://trac.webkit.org/changeset/100862>
Comment 8 Ojan Vafai 2011-11-19 17:18:12 PST 
(In reply to comment #6)
> Ok.  Should we remove the wrapper from that one too?

There's a comment in the code about why we can't. After run-webkit-tests, we load results.html as a file URL in the browser. results.html pulls in full_results.json as a file URL. This doesn't work in chromium. So, I'm not really sure how we could fix this without running a local server, which I don't think people would be happy about for run-webkit-tests.