[webkit-reviews] review requested: [Bug 16588] ResourceHandle::loadResourceSynchronously needs access to the FrameLoader : [Attachment 18311] add WebCore::Frame * as parameter to loadResourceSynchronously

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Jan 7 06:02:08 PST 2008


Simon Hausmann <hausmann at kde.org> has asked  for review:
Bug 16588: ResourceHandle::loadResourceSynchronously needs access to the
FrameLoader
http://bugs.webkit.org/show_bug.cgi?id=16588

Attachment 18311: add WebCore::Frame * as parameter to
loadResourceSynchronously
http://bugs.webkit.org/attachment.cgi?id=18311&action=edit

------- Additional Comments from Simon Hausmann <hausmann at kde.org>
Revised patch that uses Frame * instead of FrameLoader * as last argument.

I agree about the use of Frame * instead of FrameLoader. For some reason I
didn't see that the ResourceHandle constructor was using a Frame *, too.


More information about the webkit-reviews mailing list