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);

[achuithb, 2016/11/22 21:16:54]

nullptr?

[satorux1, 2016/11/24 02:13:40]

Fixed here and elsewhere in the file

 }
 
 }  // 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);
@@ skipping 61 matching lines @@
     return false;
 
   for (size_t i = 0; i < arraysize(kWhitelistedDirectories); i++) {
     if (components[0] == kWhitelistedDirectories[i])
       return true;
   }
   return false;
 }
 
 }  // namespace chromeos