[Webkit-unassigned] [Bug 67229] ThreadableLoaderClient should support 'didDownloadData' updates

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Sep 2 11:34:42 PDT 2011


https://bugs.webkit.org/show_bug.cgi?id=67229


Darin Fisher (:fishd, Google) <fishd at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #105938|review?, commit-queue?      |review-
               Flag|                            |




--- Comment #19 from Darin Fisher (:fishd, Google) <fishd at chromium.org>  2011-09-02 11:34:42 PST ---
(From update of attachment 105938)
View in context: https://bugs.webkit.org/attachment.cgi?id=105938&action=review

> Source/WebCore/loader/ResourceLoader.cpp:577
> +#if PLATFORM(CHROMIUM)

As discussed in person, I think we should move these method implementations to
a separate file named ResourceLoaderChromium.cpp.  It should live in loader/chromium/.
See loader/mac/ResourceLoaderCFNet.cpp for example.

Same goes for the other loader/*.cpp files in this patch.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list