Summary: | Need web worker referer layout test for xhr requests. | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | David Levin <levin> | ||||
Component: | XML | Assignee: | 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
David Levin
2011-03-02 10:37:44 PST
Created attachment 84437 [details]
Patch
Note the result say "FAIL" as this is currently broken (pending the overall fix in 24683). 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.
(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. Comment on attachment 84437 [details] Patch Clearing flags on attachment: 84437 Committed r80311: <http://trac.webkit.org/changeset/80311> All reviewed patches have been landed. Closing bug. |