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

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

Issue 1275623004: [Extensions] Expand testing overrides for extension warning bubbles (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 5 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
« no previous file with comments | « chrome/browser/ui/website_settings/permission_bubble_browser_test_util.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/website_settings/permission_bubble_browser_test_util.cc
diff --git a/chrome/browser/ui/website_settings/permission_bubble_browser_test_util.cc b/chrome/browser/ui/website_settings/permission_bubble_browser_test_util.cc
index 0e7eefe52cd45587b2a3e14070fa7e3ad840871f..439897d8377ec49112b0305a5836a21fcfdf40a9 100644
--- a/chrome/browser/ui/website_settings/permission_bubble_browser_test_util.cc
+++ b/chrome/browser/ui/website_settings/permission_bubble_browser_test_util.cc
@@ -21,15 +21,14 @@ TestPermissionBubbleViewDelegate::TestPermissionBubbleViewDelegate()
: PermissionBubbleView::Delegate() {
}
-PermissionBubbleBrowserTest::PermissionBubbleBrowserTest()
- : InProcessBrowserTest() {
+PermissionBubbleBrowserTest::PermissionBubbleBrowserTest() {
}
PermissionBubbleBrowserTest::~PermissionBubbleBrowserTest() {
}
void PermissionBubbleBrowserTest::SetUpOnMainThread() {
- InProcessBrowserTest::SetUpOnMainThread();
+ ExtensionBrowserTest::SetUpOnMainThread();
// Add a single permission request.
MockPermissionBubbleRequest* request = new MockPermissionBubbleRequest(
@@ -38,57 +37,26 @@ void PermissionBubbleBrowserTest::SetUpOnMainThread() {
requests_.push_back(request);
}
-PermissionBubbleAppBrowserTest::PermissionBubbleAppBrowserTest()
- : InProcessBrowserTest(),
- PermissionBubbleBrowserTest(),
- ExtensionBrowserTest(),
- app_browser_(nullptr) {
-}
-
-PermissionBubbleAppBrowserTest::~PermissionBubbleAppBrowserTest() {
-}
-
-void PermissionBubbleAppBrowserTest::SetUpOnMainThread() {
- PermissionBubbleBrowserTest::SetUpOnMainThread();
- ExtensionBrowserTest::SetUpOnMainThread();
-
+Browser* PermissionBubbleBrowserTest::OpenExtensionAppWindow() {
auto extension =
LoadExtension(test_data_dir_.AppendASCII("app_with_panel_container/"));
- ASSERT_TRUE(extension);
-
- app_browser_ = OpenExtensionAppWindow(extension);
- ASSERT_TRUE(app_browser());
- ASSERT_TRUE(app_browser()->is_app());
-}
-
-void PermissionBubbleAppBrowserTest::SetUp() {
- ExtensionBrowserTest::SetUp();
-}
-
-void PermissionBubbleAppBrowserTest::SetUpCommandLine(
- base::CommandLine* command_line) {
- ExtensionBrowserTest::SetUpCommandLine(command_line);
-}
-
-Browser* PermissionBubbleAppBrowserTest::OpenExtensionAppWindow(
- const extensions::Extension* extension) {
- base::CommandLine command_line(base::CommandLine::NO_PROGRAM);
- command_line.AppendSwitchASCII(switches::kAppId, extension->id());
+ CHECK(extension);
AppLaunchParams params(browser()->profile(), extension,
extensions::LAUNCH_CONTAINER_PANEL, NEW_WINDOW,
- extensions::SOURCE_COMMAND_LINE);
- params.command_line = command_line;
- params.current_directory = base::FilePath();
+ extensions::SOURCE_TEST);
content::WebContents* app_window = OpenApplication(params);
- assert(app_window);
+ CHECK(app_window);
+
+ Browser* app_browser = chrome::FindBrowserWithWebContents(app_window);
+ CHECK(app_browser);
+ CHECK(app_browser->is_app());
- return chrome::FindBrowserWithWebContents(app_window);
+ return app_browser;
}
-PermissionBubbleKioskBrowserTest::PermissionBubbleKioskBrowserTest()
- : PermissionBubbleBrowserTest() {
+PermissionBubbleKioskBrowserTest::PermissionBubbleKioskBrowserTest() {
}
PermissionBubbleKioskBrowserTest::~PermissionBubbleKioskBrowserTest() {
« no previous file with comments | « chrome/browser/ui/website_settings/permission_bubble_browser_test_util.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698