[webkit-reviews] review denied: [Bug 130958] [EFL][WK1] Apply std::unique_ptr<> to NavigatorContentUtilsClientEfl : [Attachment 228696] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Sun Apr 6 12:00:41 PDT 2014
Darin Adler <darin at apple.com> has denied Gyuyoung Kim
<gyuyoung.kim at samsung.com>'s request for review:
Bug 130958: [EFL][WK1] Apply std::unique_ptr<> to
NavigatorContentUtilsClientEfl
https://bugs.webkit.org/show_bug.cgi?id=130958
Attachment 228696: Patch
https://bugs.webkit.org/attachment.cgi?id=228696&action=review
------- Additional Comments from Darin Adler <darin at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=228696&action=review
> Source/WebKit/efl/ewk/ewk_view.cpp:-263
> - OwnPtr<WebCore::NavigatorContentUtilsClientEfl>
navigatorContentUtilsClient;
This data member is still needed. It can be a std::unique_ptr, but we can’t
just get rid of it.
> Source/WebKit/efl/ewk/ewk_view.cpp:687
> + WebCore::provideNavigatorContentUtilsTo(priv->page.get(),
std::make_unique<WebCore::NavigatorContentUtilsClientEfl>(smartData->self).get(
));
This is not going to work. It’s going to pass a pointer to
provideNavigatorContentUtilsTo after deleting the
NavigatorContentUtilsClientEfl object.
More information about the webkit-reviews
mailing list