| 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);
|
| -}
|
|
|