[webkit-reviews] review requested: [Bug 125583] [SOUP] Port custom URI schemes implementation to CUSTOM PROTOCOLS : [Attachment 220296] Updated patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Jan 3 02:54:26 PST 2014
Carlos Garcia Campos <cgarcia at igalia.com> has asked for review:
Bug 125583: [SOUP] Port custom URI schemes implementation to CUSTOM PROTOCOLS
https://bugs.webkit.org/show_bug.cgi?id=125583
Attachment 220296: Updated patch
https://bugs.webkit.org/attachment.cgi?id=220296&action=review
------- Additional Comments from Carlos Garcia Campos <cgarcia at igalia.com>
Patch updated to not use any soup #ifdef in CustomProtocolManager header file.
In the end I've added a CustomProtocolManagerImpl with all the soup specific
implementation. This leaves CustomProtocolManagerSoup.cpp mostly
cross-platform, so if mac switches to use their own CustomProtocolManagerImpl
to remove also the mac #ifdefs, we could remove CustomProtocolManagerSoup.cpp
and CustomProtocolManagerMac.cpp an use a common implementation for the
cross-platform bits. I've also changed some uses of OwnPtr to use
std::unique_ptr instead.
More information about the webkit-reviews
mailing list