Bug 55590

Summary: Need web worker referer layout test for xhr requests.
Product: WebKit Reporter: David Levin <levin>
Component: XMLAssignee: David Levin <levin>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, commit-queue
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 24683    
Attachments:
Description Flags
Patch none

David Levin
Reported 2011-03-02 10:37:44 PST
See summary.
Attachments
Patch (6.41 KB, patch)
2011-03-02 10:39 PST, David Levin 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
David Levin
Comment 1 2011-03-02 10:39:25 PST
Created attachment 84437 [details] Patch
David Levin
Comment 2 2011-03-02 10:40:16 PST
Note the result say "FAIL" as this is currently broken (pending the overall fix in 24683).
Darin Adler
Comment 3 2011-03-02 17:21:57 PST
Comment on attachment 84437 [details] Patch Is there a reason you put the whole test in a separate file? I think Alexey would be way happier if you put the script in the HTML file and had one less file.
David Levin
Comment 4 2011-03-02 19:10:26 PST
(In reply to comment #3) > (From update of attachment 84437 [details]) > Is there a reason you put the whole test in a separate file? I think Alexey would be way happier if you put the script in the HTML file and had one less file. Both shared-worker-referer.html and referer.html use the js file, so we test this in both shared workers and workers using it. I suppose I could make the shared worker and worker test all happen in the same file. fwiw, nearly all of the tests in this directory are structured this way for the same reason.
WebKit Commit Bot
Comment 5 2011-03-03 19:41:48 PST
Comment on attachment 84437 [details] Patch Clearing flags on attachment: 84437 Committed r80311: <http://trac.webkit.org/changeset/80311>
WebKit Commit Bot
Comment 6 2011-03-03 19:41:53 PST
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.