Make FileReader return ownership of the string content

extensions/browser/guest_view/web_view/web_ui/web_ui_url_fetcher.cc

Index: extensions/browser/guest_view/web_view/web_ui/web_ui_url_fetcher.cc
diff --git a/extensions/browser/guest_view/web_view/web_ui/web_ui_url_fetcher.cc b/extensions/browser/guest_view/web_view/web_ui/web_ui_url_fetcher.cc
index 6293c21188d9d3d58ad6d4be30f2670ac6b086e8..5c000518e6eeb6259ad8bdd209531fce599a28d8 100644
--- a/extensions/browser/guest_view/web_view/web_ui/web_ui_url_fetcher.cc
+++ b/extensions/browser/guest_view/web_view/web_ui/web_ui_url_fetcher.cc
@@ -39,10 +39,10 @@ void WebUIURLFetcher::Start() {
 void WebUIURLFetcher::OnURLFetchComplete(const net::URLFetcher* source) {
   CHECK_EQ(fetcher_.get(), source);
 
-  std::string data;
+  std::unique_ptr<std::string> data(new std::string);

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

ditto re explicit ction

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

Done.

   bool result = false;
   if (fetcher_->GetStatus().status() == net::URLRequestStatus::SUCCESS) {
-    result = fetcher_->GetResponseAsString(&data);
+    result = fetcher_->GetResponseAsString(data.get());
     DCHECK(result);
   }
   fetcher_.reset();
@@ -50,5 +50,5 @@ void WebUIURLFetcher::OnURLFetchComplete(const net::URLFetcher* source) {
   // are destroyed at the end of the method.
   auto callback_cache = callback_;
   callback_.Reset();
-  callback_cache.Run(result, data);
+  callback_cache.Run(result, std::move(data));
 }