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

Unified Diff: chrome/browser/permissions/permission_request_manager_test_api.cc

Issue 2666523002: Allow permission bubbles to participate in key event dispatch as if they were a Browser. (Closed)
Patch Set: comments, fix permission->type missed in a refactor Created 3 years, 11 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/permissions/permission_request_manager_test_api.cc
diff --git a/chrome/browser/permissions/permission_request_manager_test_api.cc b/chrome/browser/permissions/permission_request_manager_test_api.cc
new file mode 100644
index 0000000000000000000000000000000000000000..d2e1bd47f2bde8c51ff547f7ceff15f77f77dff3
--- /dev/null
+++ b/chrome/browser/permissions/permission_request_manager_test_api.cc
@@ -0,0 +1,63 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/permissions/permission_request_manager_test_api.h"
+
+#include <memory>
+
+#include "chrome/browser/permissions/permission_request_impl.h"
+#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/tabs/tab_strip_model.h"
+
+namespace test {
+namespace {
+
+// Wraps a PermissionRequestImpl so that it can pass a closure to itself to the
+// PermissionRequestImpl constructor. Without this wrapper, there's no way to
+// handle all destruction paths.
+class TestPermisisonRequestOwner {
+ public:
+ TestPermisisonRequestOwner(Profile* profile, content::PermissionType type) {
+ bool user_gesture = true;
+ auto decided = [](bool, ContentSetting) {};
+ request_ = base::MakeUnique<PermissionRequestImpl>(
+ GURL("https://example.com"), type, profile, user_gesture,
+ base::Bind(decided), base::Bind(&TestPermisisonRequestOwner::DeleteThis,
+ base::Unretained(this)));
+ }
+
+ PermissionRequestImpl* request() { return request_.get(); }
+
+ private:
+ void DeleteThis() { delete this; }
+
+ std::unique_ptr<PermissionRequestImpl> request_;
+
+ DISALLOW_COPY_AND_ASSIGN(TestPermisisonRequestOwner);
+};
+
+} // namespace
+
+PermissionRequestManagerTestApi::PermissionRequestManagerTestApi(
+ PermissionRequestManager* manager)
+ : manager_(manager) {}
+
+PermissionRequestManagerTestApi::PermissionRequestManagerTestApi(
+ Browser* browser)
+ : PermissionRequestManagerTestApi(PermissionRequestManager::FromWebContents(
+ browser->tab_strip_model()->GetActiveWebContents())) {}
+
+void PermissionRequestManagerTestApi::AddSimpleRequest(
+ Profile* profile,
+ content::PermissionType type) {
+ TestPermisisonRequestOwner* request_owner =
+ new TestPermisisonRequestOwner(profile, type);
+ manager_->AddRequest(request_owner->request());
+}
+
+gfx::NativeWindow PermissionRequestManagerTestApi::GetPromptWindow() {
+ return manager_->view_ ? manager_->view_->GetNativeWindow() : nullptr;
+}
+
+} // namespace test

Powered by Google App Engine
This is Rietveld 408576698