[webkit-reviews] review requested: [Bug 66167] [Chromium] Add WebFrameClient::adoptLoadingResource() notification : [Attachment 103850] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Aug 12 18:25:15 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 103850: Patch
https://bugs.webkit.org/attachment.cgi?id=103850&action=review

------- Additional Comments from Dmitry Titov <dimich at chromium.org>
As discussed offline, created ResourceHandleInternal.h in chromium/src.

The impl of the class still lives in ResourceHandle.cpp, I can pull it out into
separate file if desired.
Data members of ResourceHandleInternal are made private, needed accessors are
added.

The complete fix consists of WebKit patches for bug 66165 and this one, and
also Chromium side here: http://codereview.chromium.org/7647003/


More information about the webkit-reviews mailing list