[webkit-reviews] review requested: [Bug 26032] Chromium needs a frameless loader for workers : [Attachment 30718] Proposed fix.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed May 27 15:48:08 PDT 2009


David Levin <levin at chromium.org> has asked  for review:
Bug 26032: Chromium needs a frameless loader for workers
https://bugs.webkit.org/show_bug.cgi?id=26032

Attachment 30718: Proposed fix.
https://bugs.webkit.org/attachment.cgi?id=30718&action=review

------- Additional Comments from David Levin <levin at chromium.org>
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).


More information about the webkit-reviews mailing list