Enable fetching of admin policies for login screen apps

components/policy/core/common/schema_registry.cc

Index: components/policy/core/common/schema_registry.cc
diff --git a/components/policy/core/common/schema_registry.cc b/components/policy/core/common/schema_registry.cc
index a773634a7009926a80c10c84a5c4152bf6cc0fb0..517259d997e798647b8378d87c285085e36dc7cc 100644
--- a/components/policy/core/common/schema_registry.cc
+++ b/components/policy/core/common/schema_registry.cc
@@ -186,11 +186,13 @@ void CombinedSchemaRegistry::Combine(bool has_new_schemas) {
   Notify(has_new_schemas);
 }
 
-ForwardingSchemaRegistry::ForwardingSchemaRegistry(SchemaRegistry* wrapped)
-    : wrapped_(wrapped) {
+ForwardingSchemaRegistry::ForwardingSchemaRegistry(SchemaRegistry* wrapped,
+                                                   bool forward_readiness)
+    : wrapped_(wrapped), forward_readiness_(forward_readiness) {
   schema_map_ = wrapped_->schema_map();
   wrapped_->AddObserver(this);
   wrapped_->AddInternalObserver(this);
+  UpdateReadiness();
 }
 
 ForwardingSchemaRegistry::~ForwardingSchemaRegistry() {
@@ -222,6 +224,10 @@ void ForwardingSchemaRegistry::OnSchemaRegistryUpdated(bool has_new_schemas) {
   Notify(has_new_schemas);
 }
 
+void ForwardingSchemaRegistry::OnSchemaRegistryReady() {
+  UpdateReadiness();
+}
+
 void ForwardingSchemaRegistry::OnSchemaRegistryShuttingDown(
     SchemaRegistry* registry) {
   DCHECK_EQ(wrapped_, registry);
@@ -231,4 +237,11 @@ void ForwardingSchemaRegistry::OnSchemaRegistryShuttingDown(
   // Keep serving the same |schema_map_|.
 }
 
+void ForwardingSchemaRegistry::UpdateReadiness() {

[Andrew T Wilson (Slow), 2016/10/28 14:49:56]

How can UpdateReadiness() be called when wrapped_ is NULL? You remove the
observer before setting it to null.

Also, not totally clear when you *wouldn't* want forward_readiness_ = true. Can
you clarify this in the class documentation (i.e. exactly when it becomes ready
when forward_readiness_ = false)?

[emaxx, 2016/10/31 15:51:20]

You are right, check for NULL wrapped_ is unnecessary. I've remove it.
I introduced this flag because I was afraid to break the existing code by this
change.

The original behavior of this class was to have separate readiness state for the
wrapped registry and for the forwarding registry.
This may make sense in some scenarios: imagine a piece of code that accepts a
registry and is then controlling its readiness in one part and observing it in
another part - then it's really important to not have any unexpected switches of
the readiness.

So what I found in the existing code was the following structure:

1. The DeviceLocalAccountPolicyBroker instance owns a SchemaRegistry instance.
2. The DeviceLocalAccountExtensionTracker class contains a code that calls
SetReady method on this SchemaRegistry.
3. The ComponentCloudPolicyService instance observes the readiness state of that
same SchemaRegistry.
4. ProfileImpl constructor calls SchemaRegistryServiceFactory::CreateForContext,
which internally creates a ForwardingSchemaRegistry instance that wraps that
same SchemaRegistry.
5. Now all the profile-tied stuff that works with SchemaRegistry* is in case of
device-local account really working with a ForwardingSchemaRegistry instance.
For instance, a ManagedValueStoreCache::ExtensionTracker instance is constructed
that calls SetReady on the registry. And there is a number of observers of the
registry state.

So I really don't know how to make sure that none of the observers mentioned in
#5 could actually break if the forwarding schema registry may become ready
earlier...

+  if (forward_readiness_ && wrapped_ && wrapped_->IsReady()) {
+    for (int i = 0; i < POLICY_DOMAIN_SIZE; ++i)
+      SetReady(static_cast<PolicyDomain>(i));
+  }
+}
+
 }  // namespace policy