Summary: | Chromium needs a frameless loader for workers | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | David Levin <levin> | ||||
Component: | Platform | Assignee: | David Levin <levin> | ||||
Status: | RESOLVED INVALID | ||||||
Severity: | Normal | CC: | ap, fishd | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
David Levin
2009-05-26 15:09:07 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).
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 on attachment 30718 [details]
Proposed fix.
Withdrawing for now...
Will consider a dummy frame instead.
|