Remove WebViewKey in rules registry.

chrome/browser/extensions/api/declarative/rules_registry_service_unittest.cc

 // Copyright (c) 2012 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 "extensions/browser/api/declarative/rules_registry_service.h"
 
 #include "base/bind.h"
 #include "base/message_loop/message_loop.h"
 #include "chrome/test/base/testing_profile.h"
 #include "content/public/test/test_browser_thread.h"
@@ skipping 37 matching lines @@
     message_loop_.RunUntilIdle();
   }
 
  protected:
   base::MessageLoop message_loop_;
   content::TestBrowserThread ui_;
   content::TestBrowserThread io_;
 };
 
 TEST_F(RulesRegistryServiceTest, TestConstructionAndMultiThreading) {
-  const RulesRegistry::WebViewKey key(0, 0);
+  RulesRegistryService registry_service(NULL);
+
+  int key = RulesRegistryService::kDefaultRulesRegistryID;
   TestRulesRegistry* ui_registry =
       new TestRulesRegistry(content::BrowserThread::UI, "ui", key);
 
   TestRulesRegistry* io_registry =
       new TestRulesRegistry(content::BrowserThread::IO, "io", key);
 
   // Test registration.
 
-  RulesRegistryService registry_service(NULL);
   registry_service.RegisterRulesRegistry(make_scoped_refptr(ui_registry));
   registry_service.RegisterRulesRegistry(make_scoped_refptr(io_registry));
 
   EXPECT_TRUE(registry_service.GetRulesRegistry(key, "ui").get());
   EXPECT_TRUE(registry_service.GetRulesRegistry(key, "io").get());
   EXPECT_FALSE(registry_service.GetRulesRegistry(key, "foo").get());
 
   content::BrowserThread::PostTask(
       content::BrowserThread::UI, FROM_HERE,
       base::Bind(&InsertRule, registry_service.GetRulesRegistry(key, "ui"),
                  "ui_task"));
 
   content::BrowserThread::PostTask(
-        content::BrowserThread::IO, FROM_HERE,
-        base::Bind(&InsertRule, registry_service.GetRulesRegistry(key, "io"),
-                   "io_task"));
+      content::BrowserThread::IO, FROM_HERE,
+      base::Bind(&InsertRule, registry_service.GetRulesRegistry(key, "io"),
+                 "io_task"));
 
   content::BrowserThread::PostTask(
-        content::BrowserThread::UI, FROM_HERE,
-        base::Bind(&VerifyNumberOfRules,
-                   registry_service.GetRulesRegistry(key, "ui"), 1));
+      content::BrowserThread::UI, FROM_HERE,
+      base::Bind(&VerifyNumberOfRules,
+                 registry_service.GetRulesRegistry(key, "ui"), 1));
 
   content::BrowserThread::PostTask(
-        content::BrowserThread::IO, FROM_HERE,
-        base::Bind(&VerifyNumberOfRules,
-                   registry_service.GetRulesRegistry(key, "io"), 1));
+      content::BrowserThread::IO, FROM_HERE,
+      base::Bind(&VerifyNumberOfRules,
+                 registry_service.GetRulesRegistry(key, "io"), 1));
 
   message_loop_.RunUntilIdle();
 
   // Test extension uninstalling.
 
   registry_service.SimulateExtensionUninstalled(kExtensionId);
 
   content::BrowserThread::PostTask(
-        content::BrowserThread::UI, FROM_HERE,
-        base::Bind(&VerifyNumberOfRules,
-                   registry_service.GetRulesRegistry(key, "ui"), 0));
+      content::BrowserThread::UI, FROM_HERE,
+      base::Bind(&VerifyNumberOfRules,
+                 registry_service.GetRulesRegistry(key, "ui"), 0));
 
   content::BrowserThread::PostTask(
-        content::BrowserThread::IO, FROM_HERE,
-        base::Bind(&VerifyNumberOfRules,
-                   registry_service.GetRulesRegistry(key, "io"), 0));
+      content::BrowserThread::IO, FROM_HERE,
+      base::Bind(&VerifyNumberOfRules,
+                 registry_service.GetRulesRegistry(key, "io"), 0));
 
   message_loop_.RunUntilIdle();
 }
 
-// This test verifies that removing rules registries by process ID works as
-// intended. This test ensures that removing registries associated with one
-// Webview embedder process does not remove registries associated with the
-// other.
-TEST_F(RulesRegistryServiceTest, TestWebViewKey) {
-  const int kEmbedderProcessID1 = 1;
-  const int kEmbedderProcessID2 = 2;
-  const int kWebViewInstanceID = 1;
-
-  const RulesRegistry::WebViewKey key1(kEmbedderProcessID1, kWebViewInstanceID);
-  const RulesRegistry::WebViewKey key2(kEmbedderProcessID2, kWebViewInstanceID);
-
-  TestRulesRegistry* ui_registry_key1 =
-      new TestRulesRegistry(content::BrowserThread::UI, "ui", key1);
-  TestRulesRegistry* ui_registry_key2 =
-      new TestRulesRegistry(content::BrowserThread::UI, "ui", key2);
-
-  RulesRegistryService registry_service(NULL);
-  registry_service.RegisterRulesRegistry(make_scoped_refptr(ui_registry_key1));
-  registry_service.RegisterRulesRegistry(make_scoped_refptr(ui_registry_key2));
-
-  content::BrowserThread::PostTask(
-      content::BrowserThread::UI, FROM_HERE,
-      base::Bind(&InsertRule, registry_service.GetRulesRegistry(key1, "ui"),
-                 "ui_task"));
-  content::BrowserThread::PostTask(
-      content::BrowserThread::UI, FROM_HERE,
-      base::Bind(&InsertRule, registry_service.GetRulesRegistry(key2, "ui"),
-                 "ui_task"));
-  message_loop_.RunUntilIdle();
-
-  registry_service.RemoveWebViewRulesRegistries(kEmbedderProcessID1);
-  EXPECT_FALSE(registry_service.GetRulesRegistry(key1, "ui").get());
-  EXPECT_TRUE(registry_service.GetRulesRegistry(key2, "ui").get());
-}
-
-TEST_F(RulesRegistryServiceTest, TestWebViewWebRequestRegistryHasNoCache) {
-  const int kEmbedderProcessID = 1;
-  const int kWebViewInstanceID = 1;
-  const RulesRegistry::WebViewKey key(kEmbedderProcessID, kWebViewInstanceID);
-  TestingProfile profile;
-  RulesRegistryService registry_service(&profile);
-  RulesRegistry* registry =
-      registry_service.GetRulesRegistry(
-          key,
-          declarative_webrequest_constants::kOnRequest).get();
-  EXPECT_TRUE(registry);
-  EXPECT_FALSE(registry->rules_cache_delegate_for_testing());
-}
-
 }  // namespace extensions