Use base::RefCountedBytes in user_manager::UserImage

chrome/browser/ui/webui/chromeos/image_source.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/ui/webui/chromeos/image_source.h"
 
 #include <stddef.h>
 
 #include <vector>
 
@@ skipping 20 matching lines @@
 const char* kWhitelistedDirectories[] = {
   "regulatory_labels"
 };
 
 // Callback for user_manager::UserImageLoader.
 void ImageLoaded(
     const content::URLDataSource::GotDataCallback& got_data_callback,
     std::unique_ptr<user_manager::UserImage> user_image) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 
-  // TODO(crbug.com/593251): Remove the data copy.
   if (user_image->has_image_bytes())
-    got_data_callback.Run(new base::RefCountedBytes(user_image->image_bytes()));
+    got_data_callback.Run(user_image->image_bytes());
   else
-    got_data_callback.Run(NULL);
+    got_data_callback.Run(nullptr);
 }
 
 }  // namespace
 
 ImageSource::ImageSource() : weak_factory_(this) {
   base::SequencedWorkerPool* blocking_pool =
       BrowserThread::GetBlockingPool();
   task_runner_ = blocking_pool->GetSequencedTaskRunnerWithShutdownBehavior(
       blocking_pool->GetSequenceToken(),
       base::SequencedWorkerPool::CONTINUE_ON_SHUTDOWN);
 }
 
 ImageSource::~ImageSource() {
 }
 
 std::string ImageSource::GetSource() const {
   return chrome::kChromeOSAssetHost;
 }
 
 void ImageSource::StartDataRequest(
     const std::string& path,
     const content::ResourceRequestInfo::WebContentsGetter& wc_getter,
     const content::URLDataSource::GotDataCallback& got_data_callback) {
   if (!IsWhitelisted(path)) {
-    got_data_callback.Run(NULL);
+    got_data_callback.Run(nullptr);
     return;
   }
 
   const base::FilePath asset_dir(FILE_PATH_LITERAL(chrome::kChromeOSAssetPath));
   const base::FilePath image_path = asset_dir.AppendASCII(path);
   base::PostTaskAndReplyWithResult(
       content::BrowserThread::GetBlockingPool(),
       FROM_HERE,
       base::Bind(&base::PathExists, image_path),
       base::Bind(&ImageSource::StartDataRequestAfterPathExists,
@@ skipping 36 matching lines @@
     return false;
 
   for (size_t i = 0; i < arraysize(kWhitelistedDirectories); i++) {
     if (components[0] == kWhitelistedDirectories[i])
       return true;
   }
   return false;
 }
 
 }  // namespace chromeos