Make FileReader return ownership of the string content

extensions/browser/file_reader.cc

Index: extensions/browser/file_reader.cc
diff --git a/extensions/browser/file_reader.cc b/extensions/browser/file_reader.cc
index 89c1c297e42e287939266c3f42648b243e56b144..a14d9492ec846f3d6559f4f891207e396b7a0666 100644
--- a/extensions/browser/file_reader.cc
+++ b/extensions/browser/file_reader.cc
@@ -26,8 +26,8 @@ void FileReader::Start() {
 FileReader::~FileReader() {}
 
 void FileReader::ReadFileOnBackgroundThread() {
-  std::string data;
-  bool success = base::ReadFileToString(resource_.GetFilePath(), &data);
-  origin_task_runner_->PostTask(FROM_HERE,
-                                base::Bind(callback_, success, data));
+  std::unique_ptr<std::string> data(new std::string);

[Devlin, 2016/08/12 00:35:11]

nitty nit: doesn't really matter here, but I slightly prefer explicit
construction (std::string())

[lazyboy, 2016/08/12 01:31:22]

Done.

+  bool success = base::ReadFileToString(resource_.GetFilePath(), data.get());
+  origin_task_runner_->PostTask(
+      FROM_HERE, base::Bind(callback_, success, base::Passed(std::move(data))));
 }