From: Arnaud Renevier Date: Thu, 29 Oct 2015 22:03:11 +0000 (-0700) Subject: fixup! [M47_2526] Chromium upversion to m47_2526 branch X-Git-Tag: submit/tizen/20201118.160233~581 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=8573d442cec87199cc06c8841896ca632ca30588;p=platform%2Fframework%2Fweb%2Fchromium-efl.git fixup! [M47_2526] Chromium upversion to m47_2526 branch Since DisplayPersistentNotification and ClosePersistentNotification are NOTIMPLEMENTED(), GetDisplayedPersistentNotifications should be NOTIMPLEMENTED() too. Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14515 Reviewed by: a1.gomes, sns.park Change-Id: Ida2c3aacfdf0f55a91a74edb600ce33e6380f15f Signed-off-by: Arnaud Renevier --- diff --git a/tizen_src/ewk/efl_integration/browser/notification/notification_controller_efl.cc b/tizen_src/ewk/efl_integration/browser/notification/notification_controller_efl.cc index d540750..521beeb 100644 --- a/tizen_src/ewk/efl_integration/browser/notification/notification_controller_efl.cc +++ b/tizen_src/ewk/efl_integration/browser/notification/notification_controller_efl.cc @@ -281,13 +281,8 @@ void NotificationControllerEfl::SetNotificationCallbacks(Ewk_Notification_Show_C bool NotificationControllerEfl::GetDisplayedPersistentNotifications( BrowserContext* browser_context, std::set* displayed_notifications) { -#if !defined(EWK_BRINGUP) -// [M47_2526] Need to implement this API -// FIXME: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14515 -#else - // This line should be removed after resolving above issue + NOTIMPLEMENTED(); return false; -#endif } }//namespace