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

Unified Diff: chrome/browser/ui/website_settings/mock_permission_prompt_factory.cc

Issue 2335623002: Use WebContents instead of Browser to construct PermissionPrompt (Closed)
Patch Set: address review comments Created 4 years, 3 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/ui/website_settings/mock_permission_prompt_factory.cc
diff --git a/chrome/browser/ui/website_settings/mock_permission_prompt_factory.cc b/chrome/browser/ui/website_settings/mock_permission_prompt_factory.cc
index cbdab35e08a26c90af24e96081be3bf44d78480b..ddf28bfc2b1951ee38e5ef5f4c5f21ca2d049790 100644
--- a/chrome/browser/ui/website_settings/mock_permission_prompt_factory.cc
+++ b/chrome/browser/ui/website_settings/mock_permission_prompt_factory.cc
@@ -10,6 +10,7 @@
#include "base/stl_util.h"
#include "chrome/browser/permissions/permission_request_manager.h"
#include "chrome/browser/ui/website_settings/mock_permission_prompt.h"
+#include "content/public/browser/web_contents.h"
MockPermissionPromptFactory::MockPermissionPromptFactory(
PermissionRequestManager* manager)
@@ -32,7 +33,7 @@ MockPermissionPromptFactory::~MockPermissionPromptFactory() {
}
std::unique_ptr<PermissionPrompt> MockPermissionPromptFactory::Create(
- Browser* browser) {
+ content::WebContents* web_contents) {
MockPermissionPrompt* prompt = new MockPermissionPrompt(this, manager_);
prompt->can_update_ui_ = can_update_ui_;
return base::WrapUnique(prompt);
@@ -74,7 +75,7 @@ void MockPermissionPromptFactory::WaitForPermissionBubble() {
// static
std::unique_ptr<PermissionPrompt> MockPermissionPromptFactory::DoNotCreate(
- Browser* browser) {
+ content::WebContents* web_contents) {
NOTREACHED();
return base::WrapUnique(new MockPermissionPrompt(nullptr, nullptr));
}

Powered by Google App Engine
This is Rietveld 408576698