Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2133)

Unified Diff: chrome/browser/extensions/notifications_apitest.cc

Issue 376253005: Migrate the notification permission to the new common permission classes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/notifications_apitest.cc
diff --git a/chrome/browser/extensions/notifications_apitest.cc b/chrome/browser/extensions/notifications_apitest.cc
index d7afc99370b5f7e68b728ae68d630a2edc6e964d..5eaa8676908e2d8916f0346318166556bab4668e 100644
--- a/chrome/browser/extensions/notifications_apitest.cc
+++ b/chrome/browser/extensions/notifications_apitest.cc
@@ -5,8 +5,7 @@
#include "chrome/browser/extensions/extension_apitest.h"
#include "chrome/browser/extensions/lazy_background_page_test_util.h"
-#include "chrome/browser/notifications/desktop_notification_service.h"
-#include "chrome/browser/notifications/desktop_notification_service_factory.h"
+#include "chrome/browser/notifications/desktop_notification_profile_util.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/ui/browser.h"
#include "extensions/browser/process_manager.h"
@@ -40,9 +39,9 @@ IN_PROC_BROWSER_TEST_F(ExtensionApiTest, NotificationsNoPermission) {
}
IN_PROC_BROWSER_TEST_F(ExtensionApiTest, NotificationsHasPermission) {
- DesktopNotificationServiceFactory::GetForProfile(browser()->profile())
- ->GrantPermission(GURL(
- "chrome-extension://peoadpeiejnhkmpaakpnompolbglelel"));
+ DesktopNotificationProfileUtil::GrantPermission(browser()->profile(),
+ GURL("chrome-extension://peoadpeiejnhkmpaakpnompolbglelel"));
+
ASSERT_TRUE(RunExtensionTest("notifications/has_permission_prefs"))
<< message_;
}

Powered by Google App Engine
This is Rietveld 408576698