Reland: Stop passing raw pointers to base::Value API in c/b/chromeos and c/b/extensions

chrome/browser/chromeos/file_system_provider/provided_file_system_unittest.cc

 // Copyright 2014 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/chromeos/file_system_provider/provided_file_system.h"
 
 #include <memory>
 #include <string>
 #include <utility>
 #include <vector>
@@ skipping 65 matching lines @@
                                          OnAddWatcherRequested::kEventName ||
                 event->event_name == extensions::api::file_system_provider::
                                          OnRemoveWatcherRequested::kEventName ||
                 event->event_name == extensions::api::file_system_provider::
                                          OnOpenFileRequested::kEventName ||
                 event->event_name == extensions::api::file_system_provider::
                                          OnCloseFileRequested::kEventName);
 
     if (reply_result_ == base::File::FILE_OK) {
       base::ListValue value_as_list;
-      value_as_list.Set(0, new base::Value(kFileSystemId));
-      value_as_list.Set(1, new base::Value(request_id));
-      value_as_list.Set(2, new base::Value(0) /* execution_time */);
+      value_as_list.Set(0, base::MakeUnique<base::Value>(kFileSystemId));
+      value_as_list.Set(1, base::MakeUnique<base::Value>(request_id));
+      value_as_list.Set(2,
+                        base::MakeUnique<base::Value>(0) /* execution_time */);
 
       using extensions::api::file_system_provider_internal::
           OperationRequestedSuccess::Params;
       std::unique_ptr<Params> params(Params::Create(value_as_list));
       ASSERT_TRUE(params.get());
       file_system_->GetRequestManager()->FulfillRequest(
           request_id,
           RequestValue::CreateForOperationSuccess(std::move(params)),
           false /* has_more */);
     } else {
@@ skipping 817 matching lines @@
 
   ASSERT_EQ(1u, close_log.size());
   EXPECT_EQ(base::File::FILE_ERROR_NOT_FOUND, close_log[0]);
   EXPECT_EQ(0u, opened_files.size());
 
   provided_file_system_->RemoveObserver(&observer);
 }
 
 }  // namespace file_system_provider
 }  // namespace chromeos