Decoupled the SchemaRegistryService from SchemaRegistry.

chrome/browser/policy/schema_registry_service_factory.cc

Index: chrome/browser/policy/schema_registry_service_factory.cc
diff --git a/chrome/browser/policy/schema_registry_service_factory.cc b/chrome/browser/policy/schema_registry_service_factory.cc
index 71758ffc9076cbb1247912cd29ca8f1166599b20..c937500f0e682139b358b54b554f7cda8e1d0f90 100644
--- a/chrome/browser/policy/schema_registry_service_factory.cc
+++ b/chrome/browser/policy/schema_registry_service_factory.cc
@@ -60,10 +60,11 @@ SchemaRegistryServiceFactory::CreateForContextInternal(
     CombinedSchemaRegistry* global_registry) {
   DCHECK(!context->IsOffTheRecord());
   DCHECK(registries_.find(context) == registries_.end());
-  SchemaRegistryService* registry =
-      new SchemaRegistryService(chrome_schema, global_registry);
-  registries_[context] = registry;
-  return make_scoped_ptr(registry);
+  scoped_ptr<SchemaRegistry> registry(new SchemaRegistry);
+  SchemaRegistryService* service = new SchemaRegistryService(

[bartfab (slow), 2014/06/20 14:48:01]

Nit: Since you are updating this part of the code anyway: The
SchemaRegistryService should be captured in a scoped_ptr from the moment it is
created, not just at the end of the method.

[Joao da Silva, 2014/06/20 15:30:14]

Done.

+      registry.Pass(), chrome_schema, global_registry);
+  registries_[context] = service;
+  return make_scoped_ptr(service);
 }
 
 void SchemaRegistryServiceFactory::BrowserContextShutdown(