URLFetcher won't call delegate until file is closed.

net/url_request/url_fetcher_response_writer.cc

Index: net/url_request/url_fetcher_response_writer.cc
diff --git a/net/url_request/url_fetcher_response_writer.cc b/net/url_request/url_fetcher_response_writer.cc
index cb30dad8089efd6acd2d699126782cce3d147c45..b3e4a12b92b528c7533a7e5e2e9ddc24f7e1057e 100644
--- a/net/url_request/url_fetcher_response_writer.cc
+++ b/net/url_request/url_fetcher_response_writer.cc
@@ -102,8 +102,19 @@ int URLFetcherFileWriter::Write(IOBuffer* buffer,
 }
 
 int URLFetcherFileWriter::Finish(const CompletionCallback& callback) {
+  int result = file_stream_->Close(base::Bind(
+      &URLFetcherFileWriter::CloseComplete, base::Unretained(this), callback));
+  if (result != ERR_IO_PENDING) {

[Sorin Jianu, 2013/09/05 19:59:12]

Remove the full bracing for the if statement, per Chrome style guide.

[waffles, 2013/09/05 20:31:52]

Done.

+    file_stream_.reset();
+  }
+  return result;
+}
+
+void URLFetcherFileWriter::CloseComplete(const CompletionCallback& callback,
+                                         int result) {
+  // Either we closed the file or we can't; either way destroy the file_stream.
   file_stream_.reset();
-  return OK;
+  callback.Run(result);
 }
 
 void URLFetcherFileWriter::DidWrite(const CompletionCallback& callback,