Bug 26032 - Chromium needs a frameless loader for workers
Summary: Chromium needs a frameless loader for workers
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: David Levin
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-05-26 15:09 PDT by David Levin
Modified: 2009-05-27 23:12 PDT (History)
2 users (show)

See Also:


Attachments
Proposed fix. (45.31 KB, patch)
2009-05-27 15:48 PDT, 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 2009-05-26 15:09:07 PDT
Since chromium loads workers in a separate process from the frame, it needs a loader which doesn't rely on frames in order to do the loading.  (This won't allow loaders to be used on other threads and is only needed for the chromium port.)
Comment 1 David Levin 2009-05-27 15:48:08 PDT
Created attachment 30718 [details]
Proposed fix.

Although these is no chromium specific code, it is a chromium specific directory for two reasons:
1. I expect that only chromium will use it.
2. It is so similar to ResourceLoader/SubresourceLoader that to land it in the main directory those classes should be refactored but I haven't been able to figure out how to do that without adding many more virtual function calls and more complexity to those classes (which is badness).
Comment 2 Darin Fisher (:fishd, Google) 2009-05-27 16:48:26 PDT
Hmm... duplication of complicated code vs. making complicated code more complex ;-)

How bad would the refactoring be?  It seems like that is the better long term solution.
Comment 3 David Levin 2009-05-27 23:11:59 PDT
Comment on attachment 30718 [details]
Proposed fix.

Withdrawing for now...

Will consider a dummy frame instead.