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

Side by Side Diff: chrome/browser/extensions/extension_process_manager_browsertest.cc

Issue 52983004: Convert ExtensionProcessManager to BrowserContext, part 1 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix chromeos Created 7 years, 1 month 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
(Empty)
1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "chrome/browser/extensions/extension_process_manager.h"
6
7 #include "chrome/browser/extensions/browser_action_test_util.h"
8 #include "chrome/browser/extensions/extension_browsertest.h"
9 #include "chrome/browser/extensions/extension_service.h"
10 #include "chrome/browser/extensions/extension_system.h"
11 #include "chrome/test/base/in_process_browser_test.h"
12 #include "content/public/browser/notification_service.h"
13 #include "content/public/test/test_utils.h"
14
15 using extensions::Extension;
16 using extensions::ExtensionSystem;
17
18 // Exists as a browser test because ExtensionHosts are hard to create without
19 // a real browser.
20 typedef ExtensionBrowserTest ExtensionProcessManagerBrowserTest;
21
22 // Test that basic extension loading creates the appropriate ExtensionHosts
23 // and background pages.
24 IN_PROC_BROWSER_TEST_F(ExtensionProcessManagerBrowserTest,
25 ExtensionHostCreation) {
26 ExtensionProcessManager* pm =
27 ExtensionSystem::Get(profile())->process_manager();
28
29 // Load an extension with a background page.
30 scoped_refptr<const Extension> extension =
31 LoadExtension(test_data_dir_.AppendASCII("api_test")
32 .AppendASCII("browser_action")
33 .AppendASCII("none"));
34 ASSERT_TRUE(extension.get());
35
36 // Process manager gains a background host.
37 EXPECT_EQ(1u, pm->background_hosts().size());
38 EXPECT_EQ(1u, pm->GetAllViews().size());
39 EXPECT_TRUE(pm->GetBackgroundHostForExtension(extension->id()));
40 EXPECT_TRUE(pm->GetSiteInstanceForURL(extension->url()));
41 EXPECT_EQ(1u, pm->GetRenderViewHostsForExtension(extension->id()).size());
42 EXPECT_FALSE(pm->IsBackgroundHostClosing(extension->id()));
43 EXPECT_EQ(0, pm->GetLazyKeepaliveCount(extension.get()));
44
45 // Unload the extension.
46 UnloadExtension(extension->id());
47
48 // Background host disappears.
49 EXPECT_EQ(0u, pm->background_hosts().size());
50 EXPECT_EQ(0u, pm->GetAllViews().size());
51 EXPECT_FALSE(pm->GetBackgroundHostForExtension(extension->id()));
52 EXPECT_TRUE(pm->GetSiteInstanceForURL(extension->url()));
53 EXPECT_EQ(0u, pm->GetRenderViewHostsForExtension(extension->id()).size());
54 EXPECT_FALSE(pm->IsBackgroundHostClosing(extension->id()));
55 EXPECT_EQ(0, pm->GetLazyKeepaliveCount(extension.get()));
56
57 // Load an extension with the ability to open a popup but no background
58 // page.
59 scoped_refptr<const Extension> popup =
60 LoadExtension(test_data_dir_.AppendASCII("api_test")
61 .AppendASCII("browser_action")
62 .AppendASCII("popup"));
63 ASSERT_TRUE(popup);
64
65 // No background host was added.
66 EXPECT_EQ(0u, pm->background_hosts().size());
67 EXPECT_EQ(0u, pm->GetAllViews().size());
68 EXPECT_FALSE(pm->GetBackgroundHostForExtension(popup->id()));
69 EXPECT_EQ(0u, pm->GetRenderViewHostsForExtension(popup->id()).size());
70 EXPECT_TRUE(pm->GetSiteInstanceForURL(popup->url()));
71 EXPECT_FALSE(pm->IsBackgroundHostClosing(popup->id()));
72 EXPECT_EQ(0, pm->GetLazyKeepaliveCount(popup.get()));
73
74 // Simulate clicking on the action to open a popup.
75 BrowserActionTestUtil test_util(browser());
76 content::WindowedNotificationObserver frame_observer(
77 content::NOTIFICATION_LOAD_COMPLETED_MAIN_FRAME,
78 content::NotificationService::AllSources());
79 // Open popup in the first extension.
80 test_util.Press(0);
81 frame_observer.Wait();
82 ASSERT_TRUE(test_util.HasPopup());
83
84 // We now have a view, but still no background hosts.
85 EXPECT_EQ(0u, pm->background_hosts().size());
86 EXPECT_EQ(1u, pm->GetAllViews().size());
87 EXPECT_FALSE(pm->GetBackgroundHostForExtension(popup->id()));
88 EXPECT_EQ(1u, pm->GetRenderViewHostsForExtension(popup->id()).size());
89 EXPECT_TRUE(pm->GetSiteInstanceForURL(popup->url()));
90 EXPECT_FALSE(pm->IsBackgroundHostClosing(popup->id()));
91 EXPECT_EQ(0, pm->GetLazyKeepaliveCount(popup.get()));
92 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698