Accept invalid Schemas for the managed storage API.

chrome/browser/extensions/api/storage/managed_value_store_cache.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 "chrome/browser/extensions/api/storage/managed_value_store_cache.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/file_util.h"
@@ skipping 188 matching lines @@
             manifest_keys::kStorageManagedSchema, &schema_file)) {
       // TODO(joaodasilva): Remove this. http://crbug.com/325349
       (*components)[(*it)->id()] = policy::Schema();
       continue;
     }
     // The extension should have been validated, so assume the schema exists
     // and is valid.
     std::string error;
     policy::Schema schema =
         StorageSchemaManifestHandler::GetSchema(it->get(), &error);
-    CHECK(schema.valid()) << error;
+    // If the schema is invalid then proceed with an empty schema. The extension
+    // will be listed in chrome://policy but won't be able to load any policies.

[not at google - send to devlin, 2014/08/13 17:27:31]

It'd be nice to try to recover the policy in this case, e.g. by re-installing
the extension. I've seen that requested before for enterprise extensions. I
can't find the bug but maybe asargent/erikkay can remember. Anyhow, not
something to try in this CL... but it'd be nice to have a bug to reference.

[Joao da Silva, 2014/08/14 11:56:57]

sgtm. Shouldn't we try to do the same recovery for non-enterprise extensions
too?

+    if (!schema.valid())
+      schema = policy::Schema();
     (*components)[(*it)->id()] = schema;
   }
 
   BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
                           base::Bind(&ExtensionTracker::Register, self,
                                      base::Owned(components.release())));
 }
 
 void ManagedValueStoreCache::ExtensionTracker::Register(
     const policy::ComponentMap* components) {
@@ skipping 145 matching lines @@
 
   return store;
 }
 
 bool ManagedValueStoreCache::HasStore(const std::string& extension_id) const {
   // TODO(joaodasilva): move this check to a ValueStore method.
   return base::DirectoryExists(base_path_.AppendASCII(extension_id));
 }
 
 }  // namespace extensions