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

Unified Diff: headless/lib/headless_browser_browsertest.cc

Issue 2771993003: Added headless permission manager. This permissions manager simply acks all permission requests wit… (Closed)
Patch Set: Added headless permission manager. This permissions manager simply acks all permission requests wit… Created 3 years, 9 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
« no previous file with comments | « headless/lib/browser/headless_permission_manager.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: headless/lib/headless_browser_browsertest.cc
diff --git a/headless/lib/headless_browser_browsertest.cc b/headless/lib/headless_browser_browsertest.cc
index 6b0b3b846f720e909b18a64d995745c9a471fc16..6561339d51eca2405d2c3a9a6a492d33b3668c42 100644
--- a/headless/lib/headless_browser_browsertest.cc
+++ b/headless/lib/headless_browser_browsertest.cc
@@ -11,8 +11,12 @@
#include "base/strings/stringprintf.h"
#include "base/strings/utf_string_conversions.h"
#include "base/threading/thread_restrictions.h"
+#include "content/public/browser/permission_manager.h"
+#include "content/public/browser/permission_type.h"
#include "content/public/common/url_constants.h"
#include "content/public/test/browser_test.h"
+#include "headless/lib/browser/headless_browser_context_impl.h"
+#include "headless/lib/browser/headless_web_contents_impl.h"
#include "headless/lib/headless_macros.h"
#include "headless/public/devtools/domains/inspector.h"
#include "headless/public/devtools/domains/network.h"
@@ -753,4 +757,27 @@ IN_PROC_BROWSER_TEST_F(CrashReporterTest, MAYBE_GenerateMinidump) {
browser_context_ = nullptr;
}
+IN_PROC_BROWSER_TEST_F(HeadlessBrowserTest, PermissionManagerAlwaysASK) {
+ GURL url("https://example.com");
+
+ HeadlessBrowserContext* browser_context =
+ browser()->CreateBrowserContextBuilder().Build();
+
+ HeadlessWebContents* headless_web_contents =
+ browser_context->CreateWebContentsBuilder().Build();
+ EXPECT_TRUE(headless_web_contents);
+
+ HeadlessWebContentsImpl* web_contents =
+ HeadlessWebContentsImpl::From(headless_web_contents);
+
+ content::PermissionManager* permission_manager =
+ web_contents->browser_context()->GetPermissionManager();
+ EXPECT_NE(nullptr, permission_manager);
+
+ // Check that the permission manager returns ASK for a given permission type.
+ EXPECT_EQ(blink::mojom::PermissionStatus::ASK,
+ permission_manager->GetPermissionStatus(
+ content::PermissionType::NOTIFICATIONS, url, url));
+}
+
} // namespace headless
« no previous file with comments | « headless/lib/browser/headless_permission_manager.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698