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

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

Issue 2182883002: Permissions: Rename PermissionBubbleView to PermissionPromptInterface (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: thestig@ review Created 4 years, 4 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.cc
diff --git a/chrome/browser/ui/website_settings/mock_permission_bubble_view.cc b/chrome/browser/ui/website_settings/mock_permission_prompt.cc
similarity index 53%
rename from chrome/browser/ui/website_settings/mock_permission_bubble_view.cc
rename to chrome/browser/ui/website_settings/mock_permission_prompt.cc
index 3530ee411e8ed67da7bed0b156145c16cf86d9fa..d0e9cb06072b1ea5fdd7d0e73d8a37864f6a6f53 100644
--- a/chrome/browser/ui/website_settings/mock_permission_bubble_view.cc
+++ b/chrome/browser/ui/website_settings/mock_permission_prompt.cc
@@ -2,20 +2,19 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "chrome/browser/ui/website_settings/mock_permission_bubble_view.h"
+#include "chrome/browser/ui/website_settings/mock_permission_prompt.h"
#include "base/bind.h"
#include "base/run_loop.h"
#include "chrome/browser/permissions/permission_request_manager.h"
-#include "chrome/browser/ui/website_settings/mock_permission_bubble_factory.h"
+#include "chrome/browser/ui/website_settings/mock_permission_prompt_factory.h"
-MockPermissionBubbleView::~MockPermissionBubbleView() {
+MockPermissionPrompt::~MockPermissionPrompt() {
Hide();
}
-void MockPermissionBubbleView::Show(
- const std::vector<PermissionRequest*>& requests,
- const std::vector<bool>& accept_state) {
+void MockPermissionPrompt::Show(const std::vector<PermissionRequest*>& requests,
+ const std::vector<bool>& accept_state) {
factory_->ShowView(this);
factory_->show_count_++;
factory_->requests_count_ = manager_->requests_.size();
@@ -24,31 +23,30 @@ void MockPermissionBubbleView::Show(
is_visible_ = true;
}
-bool MockPermissionBubbleView::CanAcceptRequestUpdate() {
+bool MockPermissionPrompt::CanAcceptRequestUpdate() {
return can_update_ui_;
}
-void MockPermissionBubbleView::Hide() {
+void MockPermissionPrompt::Hide() {
if (is_visible_ && factory_)
factory_->HideView(this);
is_visible_ = false;
}
-bool MockPermissionBubbleView::IsVisible() {
+bool MockPermissionPrompt::IsVisible() {
return is_visible_;
}
-void MockPermissionBubbleView::UpdateAnchorPosition() {}
+void MockPermissionPrompt::UpdateAnchorPosition() {}
-gfx::NativeWindow MockPermissionBubbleView::GetNativeWindow() {
+gfx::NativeWindow MockPermissionPrompt::GetNativeWindow() {
// This class should only be used when the UI is not necessary.
NOTREACHED();
return nullptr;
}
-MockPermissionBubbleView::MockPermissionBubbleView(
- MockPermissionBubbleFactory* factory,
- PermissionRequestManager* manager)
+MockPermissionPrompt::MockPermissionPrompt(MockPermissionPromptFactory* factory,
+ PermissionRequestManager* manager)
: factory_(factory),
manager_(manager),
can_update_ui_(true),

Powered by Google App Engine
This is Rietveld 408576698