Bug 55590 - Need web worker referer layout test for xhr requests.
Summary: Need web worker referer layout test for xhr requests.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: XML (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: David Levin
URL:
Keywords:
Depends on:
Blocks: 24683
  Show dependency treegraph
 
Reported: 2011-03-02 10:37 PST by David Levin
Modified: 2011-03-03 19:41 PST (History)
2 users (show)

See Also:


Attachments
Patch (6.41 KB, patch)
2011-03-02 10:39 PST, David Levin
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Levin 2011-03-02 10:37:44 PST
See summary.
Comment 1 David Levin 2011-03-02 10:39:25 PST
Created attachment 84437 [details]
Patch
Comment 2 David Levin 2011-03-02 10:40:16 PST
Note the result say "FAIL" as this is currently broken (pending the overall fix in 24683).
Comment 3 Darin Adler 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.
Comment 4 David Levin 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.
Comment 5 WebKit Commit Bot 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>
Comment 6 WebKit Commit Bot 2011-03-03 19:41:53 PST
All reviewed patches have been landed.  Closing bug.