[webkit-reviews] review requested: [Bug 66167] [Chromium] Add WebFrameClient::adoptLoadingResource() notification : [Attachment 103821] Updated patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Aug 12 14:56:10 PDT 2011


Dmitry Titov <dimich at chromium.org> has asked  for review:
Bug 66167: [Chromium] Add WebFrameClient::adoptLoadingResource() notification
https://bugs.webkit.org/show_bug.cgi?id=66167

Attachment 103821: Updated patch
https://bugs.webkit.org/attachment.cgi?id=103821&action=review

------- Additional Comments from Dmitry Titov <dimich at chromium.org>
Renamed method to didAdoptURLLoader.

As for static method - I hope there is a better way, I'm using the fact that
ResourceHandleInternal is a 'friend' to ResourceHandle, to pull the 'd' member.
We currently don't have access to it otherwise. Do I miss something?


More information about the webkit-reviews mailing list