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

Unified Diff: chrome/browser/notifications/platform_notification_service_browsertest.cc

Issue 1686903002: [Abandoned] Rename PermissionBubbleManager to PermissionRequestManager. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: tests compile Created 4 years, 8 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/notifications/platform_notification_service_browsertest.cc
diff --git a/chrome/browser/notifications/platform_notification_service_browsertest.cc b/chrome/browser/notifications/platform_notification_service_browsertest.cc
index 0b0997956018b08a46f52887036866fab9a1adcb..43420884271c00bc93fe9f69d8dedd866f89a88b 100644
--- a/chrome/browser/notifications/platform_notification_service_browsertest.cc
+++ b/chrome/browser/notifications/platform_notification_service_browsertest.cc
@@ -17,9 +17,9 @@
#include "chrome/browser/notifications/notification_test_util.h"
#include "chrome/browser/notifications/platform_notification_service_impl.h"
#include "chrome/browser/permissions/permission_manager.h"
+#include "chrome/browser/permissions/permission_request_manager.h"
#include "chrome/browser/ui/browser.h"
#include "chrome/browser/ui/tabs/tab_strip_model.h"
-#include "chrome/browser/ui/website_settings/permission_bubble_manager.h"
#include "chrome/grit/generated_resources.h"
#include "chrome/test/base/in_process_browser_test.h"
#include "chrome/test/base/ui_test_utils.h"
@@ -86,7 +86,7 @@ class PlatformNotificationServiceBrowserTest : public InProcessBrowserTest {
private:
std::string RequestAndRespondToPermission(
- PermissionBubbleManager::AutoResponseType bubble_response);
+ PermissionRequestManager::AutoResponseType response);
content::WebContents* GetActiveWebContents(Browser* browser) {
return browser->tab_strip_model()->GetActiveWebContents();
@@ -173,24 +173,24 @@ GURL PlatformNotificationServiceBrowserTest::TestPageUrl() const {
std::string
PlatformNotificationServiceBrowserTest::RequestAndRespondToPermission(
- PermissionBubbleManager::AutoResponseType bubble_response) {
+ PermissionRequestManager::AutoResponseType response) {
std::string result;
content::WebContents* web_contents = GetActiveWebContents(browser());
- PermissionBubbleManager::FromWebContents(web_contents)
- ->set_auto_response_for_test(bubble_response);
+ PermissionRequestManager::FromWebContents(web_contents)
+ ->set_auto_response_for_test(response);
EXPECT_TRUE(RunScript("RequestPermission();", &result));
return result;
}
bool PlatformNotificationServiceBrowserTest::RequestAndAcceptPermission() {
std::string result =
- RequestAndRespondToPermission(PermissionBubbleManager::ACCEPT_ALL);
+ RequestAndRespondToPermission(PermissionRequestManager::ACCEPT_ALL);
return "granted" == result;
}
bool PlatformNotificationServiceBrowserTest::RequestAndDenyPermission() {
std::string result =
- RequestAndRespondToPermission(PermissionBubbleManager::DENY_ALL);
+ RequestAndRespondToPermission(PermissionRequestManager::DENY_ALL);
return "denied" == result;
}
« no previous file with comments | « chrome/browser/media/webrtc_browsertest_base.cc ('k') | chrome/browser/permissions/permission_context_base.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698