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

Unified Diff: chrome/browser/extensions/extension_process_manager_unittest.cc

Issue 62713003: Move ExtensionProcessManager to src/extensions, part 4 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase again 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/extension_process_manager_unittest.cc
diff --git a/chrome/browser/extensions/extension_process_manager_unittest.cc b/chrome/browser/extensions/extension_process_manager_unittest.cc
deleted file mode 100644
index c8ca541e46ff86935adc896e488bdb144cae3733..0000000000000000000000000000000000000000
--- a/chrome/browser/extensions/extension_process_manager_unittest.cc
+++ /dev/null
@@ -1,136 +0,0 @@
-// Copyright (c) 2011 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/extensions/extension_process_manager.h"
-
-#include "chrome/browser/chrome_notification_types.h"
-#include "chrome/browser/extensions/extension_error_reporter.h"
-#include "chrome/test/base/testing_profile.h"
-#include "content/public/browser/notification_service.h"
-#include "content/public/browser/render_process_host.h"
-#include "content/public/browser/site_instance.h"
-#include "testing/gtest/include/gtest/gtest.h"
-#include "testing/platform_test.h"
-
-using content::SiteInstance;
-
-// make the test a PlatformTest to setup autorelease pools properly on mac
-class ExtensionProcessManagerTest : public testing::Test {
- public:
- static void SetUpTestCase() {
- ExtensionErrorReporter::Init(false); // no noisy errors
- }
-
- virtual void SetUp() {
- ExtensionErrorReporter::GetInstance()->ClearErrors();
- }
-
- // Returns true if the notification |type| is registered for |manager| with
- // source |profile|. Pass NULL for |profile| for all sources.
- static bool IsRegistered(ExtensionProcessManager* manager,
- int type,
- TestingProfile* profile) {
- return manager->registrar_.IsRegistered(
- manager, type, content::Source<Profile>(profile));
- }
-};
-
-// Test that notification registration works properly.
-TEST_F(ExtensionProcessManagerTest, ExtensionNotificationRegistration) {
- // Test for a normal profile.
- scoped_ptr<TestingProfile> original_profile(new TestingProfile);
- scoped_ptr<ExtensionProcessManager> manager1(
- ExtensionProcessManager::Create(original_profile.get()));
-
- EXPECT_EQ(original_profile.get(), manager1->GetBrowserContext());
- EXPECT_EQ(0u, manager1->background_hosts().size());
-
- // It observes other notifications from this profile.
- EXPECT_TRUE(IsRegistered(manager1.get(),
- chrome::NOTIFICATION_EXTENSIONS_READY,
- original_profile.get()));
- EXPECT_TRUE(IsRegistered(manager1.get(),
- chrome::NOTIFICATION_EXTENSION_LOADED,
- original_profile.get()));
- EXPECT_TRUE(IsRegistered(manager1.get(),
- chrome::NOTIFICATION_EXTENSION_UNLOADED,
- original_profile.get()));
- EXPECT_TRUE(IsRegistered(manager1.get(),
- chrome::NOTIFICATION_EXTENSION_HOST_DESTROYED,
- original_profile.get()));
-
- // Now add an incognito profile associated with the master above.
- TestingProfile::Builder builder;
- builder.SetIncognito();
- scoped_ptr<TestingProfile> incognito_profile = builder.Build();
- incognito_profile->SetOriginalProfile(original_profile.get());
- scoped_ptr<ExtensionProcessManager> manager2(
- ExtensionProcessManager::Create(incognito_profile.get()));
-
- EXPECT_EQ(incognito_profile.get(), manager2->GetBrowserContext());
- EXPECT_EQ(0u, manager2->background_hosts().size());
-
- // Some notifications are observed for the original profile.
- EXPECT_TRUE(IsRegistered(manager2.get(),
- chrome::NOTIFICATION_EXTENSION_LOADED,
- original_profile.get()));
-
- // Some notifications are observed for the incognito profile.
- EXPECT_TRUE(IsRegistered(manager2.get(),
- chrome::NOTIFICATION_EXTENSION_HOST_DESTROYED,
- incognito_profile.get()));
-
- // Some notifications are observed for both incognito and original.
- EXPECT_TRUE(IsRegistered(manager2.get(),
- chrome::NOTIFICATION_PROFILE_DESTROYED,
- original_profile.get()));
- EXPECT_TRUE(IsRegistered(manager2.get(),
- chrome::NOTIFICATION_PROFILE_DESTROYED,
- incognito_profile.get()));
-
- // Some are not observed at all.
- EXPECT_FALSE(IsRegistered(manager2.get(),
- chrome::NOTIFICATION_EXTENSIONS_READY,
- original_profile.get()));
-
- // This notification is observed for incognito profiles only.
- EXPECT_TRUE(IsRegistered(manager2.get(),
- chrome::NOTIFICATION_PROFILE_DESTROYED,
- incognito_profile.get()));
-}
-
-// Test that extensions get grouped in the right SiteInstance (and therefore
-// process) based on their URLs.
-TEST_F(ExtensionProcessManagerTest, ProcessGrouping) {
- // Extensions in different profiles should always be different SiteInstances.
- // Note: we don't initialize these, since we're not testing that
- // functionality. This means we can get away with a NULL UserScriptMaster.
- TestingProfile profile1;
- scoped_ptr<ExtensionProcessManager> manager1(
- ExtensionProcessManager::Create(&profile1));
-
- TestingProfile profile2;
- scoped_ptr<ExtensionProcessManager> manager2(
- ExtensionProcessManager::Create(&profile2));
-
- // Extensions with common origins ("scheme://id/") should be grouped in the
- // same SiteInstance.
- GURL ext1_url1("chrome-extension://ext1_id/index.html");
- GURL ext1_url2("chrome-extension://ext1_id/monkey/monkey.html");
- GURL ext2_url1("chrome-extension://ext2_id/index.html");
-
- scoped_refptr<SiteInstance> site11 =
- manager1->GetSiteInstanceForURL(ext1_url1);
- scoped_refptr<SiteInstance> site12 =
- manager1->GetSiteInstanceForURL(ext1_url2);
- EXPECT_EQ(site11, site12);
-
- scoped_refptr<SiteInstance> site21 =
- manager1->GetSiteInstanceForURL(ext2_url1);
- EXPECT_NE(site11, site21);
-
- scoped_refptr<SiteInstance> other_profile_site =
- manager2->GetSiteInstanceForURL(ext1_url1);
- EXPECT_NE(site11, other_profile_site);
-}
« no previous file with comments | « chrome/browser/extensions/extension_process_manager_browsertest.cc ('k') | chrome/browser/extensions/extension_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698