Introduced a ForwardingPolicyProvider.

chrome/browser/policy/schema_registry_unittest.cc

Index: chrome/browser/policy/schema_registry_unittest.cc
diff --git a/chrome/browser/policy/schema_registry_unittest.cc b/chrome/browser/policy/schema_registry_unittest.cc
index 62bed9e6b490e9c6c66f7773c7c80d123b4b6a87..29e2a91254d499303d61f0680b3341830e19af74 100644
--- a/chrome/browser/policy/schema_registry_unittest.cc
+++ b/chrome/browser/policy/schema_registry_unittest.cc
@@ -45,6 +45,7 @@ class MockSchemaRegistryObserver : public SchemaRegistry::Observer {
   virtual ~MockSchemaRegistryObserver() {}
 
   MOCK_METHOD1(OnSchemaRegistryUpdated, void(bool));
+  MOCK_METHOD0(OnSchemaRegistryReady, void());
 };
 
 }  // namespace
@@ -100,6 +101,22 @@ TEST(SchemaRegistryTest, Notifications) {
   EXPECT_FALSE(registry.HasObservers());
 }
 
+TEST(SchemaRegistryTest, IsReady) {
+  SchemaRegistry registry;
+  EXPECT_FALSE(registry.IsReady());
+  registry.SetReady(POLICY_DOMAIN_CHROME);
+#if defined(ENABLE_EXTENSIONS)
+  EXPECT_FALSE(registry.IsReady());
+  registry.SetReady(POLICY_DOMAIN_EXTENSIONS);
+  EXPECT_TRUE(registry.IsReady());
+  registry.SetReady(POLICY_DOMAIN_EXTENSIONS);

[bartfab (slow), 2013/11/11 14:40:08]

Nit 1: It would be useful to have an observer in this test, verifying not only
that IsReady() does not misbehave when the same domain becomes ready but also
OnSchemaRegistryReady() does not fire too often.

Nit 2: Why not use POLICY_DOMAIN_CHROME as the domain that you mark as ready
twice? That way, marking a component as ready twice would get tested even if
ENABLE_EXTENSIONS is not defined.

[Joao da Silva, 2013/11/12 15:26:33]

Done.

+#endif
+  EXPECT_TRUE(registry.IsReady());
+
+  CombinedSchemaRegistry combined;
+  EXPECT_TRUE(combined.IsReady());
+}
+
 TEST(SchemaRegistryTest, Combined) {
   std::string error;
   Schema schema = Schema::Parse(kTestSchema, &error);