file_browser_handler/ -> remove linked_ptr usage.

chrome/common/extensions/api/file_browser_handlers/file_browser_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/file_browser_handlers/file_browser_handler.h"
 
 #include <stddef.h>
 
 #include <memory>
 
@@ skipping 120 matching lines @@
 }
 
 FileBrowserHandlerParser::FileBrowserHandlerParser() {
 }
 
 FileBrowserHandlerParser::~FileBrowserHandlerParser() {
 }
 
 namespace {
 
-FileBrowserHandler* LoadFileBrowserHandler(
+std::unique_ptr<FileBrowserHandler> LoadFileBrowserHandler(
     const std::string& extension_id,
     const base::DictionaryValue* file_browser_handler,
     base::string16* error) {
   std::unique_ptr<FileBrowserHandler> result(new FileBrowserHandler());
   result->set_extension_id(extension_id);
 
   std::string handler_id;
   // Read the file action |id| (mandatory).
   if (!file_browser_handler->HasKey(keys::kPageActionId) ||
       !file_browser_handler->GetString(keys::kPageActionId, &handler_id)) {
     *error = base::ASCIIToUTF16(errors::kInvalidPageActionId);
-    return NULL;
+    return nullptr;
   }
   result->set_id(handler_id);
 
   // Read the page action title from |default_title| (mandatory).
   std::string title;
   if (!file_browser_handler->HasKey(keys::kPageActionDefaultTitle) ||
       !file_browser_handler->GetString(keys::kPageActionDefaultTitle, &title)) {
     *error = base::ASCIIToUTF16(errors::kInvalidPageActionDefaultTitle);
-    return NULL;
+    return nullptr;
   }
   result->set_title(title);
 
   // Initialize access permissions (optional).
-  const base::ListValue* access_list_value = NULL;
+  const base::ListValue* access_list_value = nullptr;
   if (file_browser_handler->HasKey(keys::kFileAccessList)) {
     if (!file_browser_handler->GetList(keys::kFileAccessList,
                                        &access_list_value) ||
         access_list_value->empty()) {
       *error = base::ASCIIToUTF16(errors::kInvalidFileAccessList);
-      return NULL;
+      return nullptr;
     }
     for (size_t i = 0; i < access_list_value->GetSize(); ++i) {
       std::string access;
       if (!access_list_value->GetString(i, &access) ||
           result->AddFileAccessPermission(access)) {
         *error = extensions::ErrorUtils::FormatErrorMessageUTF16(
             errors::kInvalidFileAccessValue, base::IntToString(i));
-        return NULL;
+        return nullptr;
       }
     }
   }
   if (!result->ValidateFileAccessPermissions()) {
     *error = base::ASCIIToUTF16(errors::kInvalidFileAccessList);
-    return NULL;
+    return nullptr;
   }
 
   // Initialize file filters (mandatory, unless "create" access is specified,
   // in which case is ignored). The list can be empty.
   if (!result->HasCreateAccessPermission()) {
-    const base::ListValue* file_filters = NULL;
+    const base::ListValue* file_filters = nullptr;
     if (!file_browser_handler->HasKey(keys::kFileFilters) ||
         !file_browser_handler->GetList(keys::kFileFilters, &file_filters)) {
       *error = base::ASCIIToUTF16(errors::kInvalidFileFiltersList);
-      return NULL;
+      return nullptr;
     }
     for (size_t i = 0; i < file_filters->GetSize(); ++i) {
       std::string filter;
       if (!file_filters->GetString(i, &filter)) {
         *error = extensions::ErrorUtils::FormatErrorMessageUTF16(
             errors::kInvalidFileFilterValue, base::IntToString(i));
-        return NULL;
+        return nullptr;
       }
       filter = base::ToLowerASCII(filter);
       if (!base::StartsWith(filter, std::string(url::kFileSystemScheme) + ':',
                             base::CompareCase::SENSITIVE)) {
         *error = extensions::ErrorUtils::FormatErrorMessageUTF16(
             errors::kInvalidURLPatternError, filter);
-        return NULL;
+        return nullptr;
       }
       // The user inputs filesystem:*; we don't actually implement scheme
       // wildcards in URLPattern, so transform to what will match correctly.
       filter.replace(0, 11, "chrome-extension://*/");
       URLPattern pattern(URLPattern::SCHEME_EXTENSION);
       if (pattern.Parse(filter) != URLPattern::PARSE_SUCCESS) {
         *error = extensions::ErrorUtils::FormatErrorMessageUTF16(
             errors::kInvalidURLPatternError, filter);
-        return NULL;
+        return nullptr;
       }
       std::string path = pattern.path();
       bool allowed = path == "/*" || path == "/*.*" ||
           (path.compare(0, 3, "/*.") == 0 &&
            path.find_first_of('*', 3) == std::string::npos);
       if (!allowed) {
         *error = extensions::ErrorUtils::FormatErrorMessageUTF16(
             errors::kInvalidURLPatternError, filter);
-        return NULL;
+        return nullptr;
       }
       result->AddPattern(pattern);
     }
   }
 
   std::string default_icon;
   // Read the file browser action |default_icon| (optional).
   if (file_browser_handler->HasKey(keys::kPageActionDefaultIcon)) {
     if (!file_browser_handler->GetString(
             keys::kPageActionDefaultIcon, &default_icon) ||
         default_icon.empty()) {
       *error = base::ASCIIToUTF16(errors::kInvalidPageActionIconPath);
-      return NULL;
+      return nullptr;
     }
     result->set_icon_path(default_icon);
   }
 
-  return result.release();
+  return result;
 }
 
 // Loads FileBrowserHandlers from |extension_actions| into a list in |result|.
 bool LoadFileBrowserHandlers(
     const std::string& extension_id,
     const base::ListValue* extension_actions,
     FileBrowserHandler::List* result,
     base::string16* error) {
   for (const auto& entry : *extension_actions) {
     base::DictionaryValue* dict;
     if (!entry->GetAsDictionary(&dict)) {
       *error = base::ASCIIToUTF16(errors::kInvalidFileBrowserHandler);
       return false;
     }
-    std::unique_ptr<FileBrowserHandler> action(
-        LoadFileBrowserHandler(extension_id, dict, error));
-    if (!action.get())
+    std::unique_ptr<FileBrowserHandler> action =
+        LoadFileBrowserHandler(extension_id, dict, error);
+    if (!action)
       return false;  // Failed to parse file browser action definition.
-    result->push_back(linked_ptr<FileBrowserHandler>(action.release()));
+    result->push_back(std::move(action));
   }
   return true;
 }
 
 }  // namespace
 
 bool FileBrowserHandlerParser::Parse(extensions::Extension* extension,
                                      base::string16* error) {
   const base::Value* file_browser_handlers_value = nullptr;
   if (!extension->manifest()->Get(keys::kFileBrowserHandlers,
@@ skipping 22 matching lines @@
     return false;  // Failed to parse file browser actions definition.
   }
 
   extension->SetManifestData(keys::kFileBrowserHandlers, std::move(info));
   return true;
 }
 
 const std::vector<std::string> FileBrowserHandlerParser::Keys() const {
   return SingleKey(keys::kFileBrowserHandlers);
 }