Make the internal storage of policy::Schemas ref counted.

chrome/common/extensions/api/storage/storage_schema_manifest_handler.cc

 // Copyright 2013 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/common/extensions/api/storage/storage_schema_manifest_handler.h"
 
 #include <string>
 #include <vector>
 
 #include "base/file_util.h"
@@ skipping 15 matching lines @@
 using extensions::manifest_keys::kStorageManagedSchema;
 
 namespace extensions {
 
 StorageSchemaManifestHandler::StorageSchemaManifestHandler() {}
 
 StorageSchemaManifestHandler::~StorageSchemaManifestHandler() {}
 
 #if defined(ENABLE_CONFIGURATION_POLICY)
 // static
-scoped_ptr<policy::SchemaOwner> StorageSchemaManifestHandler::GetSchema(
+policy::Schema StorageSchemaManifestHandler::GetSchema(
     const Extension* extension,
     std::string* error) {
   if (!extension->HasAPIPermission(APIPermission::kStorage)) {
     *error = base::StringPrintf("The storage permission is required to use %s",
                                 kStorageManagedSchema);
-    return scoped_ptr<policy::SchemaOwner>();
+    return policy::Schema();
   }
   std::string path;
   extension->manifest()->GetString(kStorageManagedSchema, &path);
   base::FilePath file = base::FilePath::FromUTF8Unsafe(path);
   if (file.IsAbsolute() || file.ReferencesParent()) {
     *error = base::StringPrintf("%s must be a relative path without ..",
                                 kStorageManagedSchema);
-    return scoped_ptr<policy::SchemaOwner>();
+    return policy::Schema();
   }
   file = extension->path().AppendASCII(path);
   if (!base::PathExists(file)) {
     *error =
         base::StringPrintf("File does not exist: %s", file.value().c_str());
-    return scoped_ptr<policy::SchemaOwner>();
+    return policy::Schema();
   }
   std::string content;
   if (!base::ReadFileToString(file, &content)) {
     *error = base::StringPrintf("Can't read %s", file.value().c_str());
-    return scoped_ptr<policy::SchemaOwner>();
+    return policy::Schema();
   }
-  return policy::SchemaOwner::Parse(content, error);
+  return policy::Schema::Parse(content, error);
 }
 #endif
 
 bool StorageSchemaManifestHandler::Parse(Extension* extension,
                                          string16* error) {
   std::string path;
   if (!extension->manifest()->GetString(kStorageManagedSchema, &path)) {
     *error = ASCIIToUTF16(
         base::StringPrintf("%s must be a string", kStorageManagedSchema));
     return false;
   }
   return true;
 }
 
 bool StorageSchemaManifestHandler::Validate(
     const Extension* extension,
     std::string* error,
     std::vector<InstallWarning>* warnings) const {
 #if defined(ENABLE_CONFIGURATION_POLICY)
-  return !!GetSchema(extension, error);
+  return GetSchema(extension, error).valid();
 #else
   return true;
 #endif
 }
 
 const std::vector<std::string> StorageSchemaManifestHandler::Keys() const {
   return SingleKey(kStorageManagedSchema);
 }
 
 }  // namespace extensions