Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1238)

Unified Diff: content/browser/fileapi/fileapi_message_filter.cc

Issue 2368913002: Fix stream errors that occur after a stream starts not propagating. (Closed)
Patch Set: fix tests Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/fileapi/fileapi_message_filter.cc
diff --git a/content/browser/fileapi/fileapi_message_filter.cc b/content/browser/fileapi/fileapi_message_filter.cc
index 2ce097865fea74bf674b9fbc39c2e0d35af74cd8..e67d1c591166332d52ef5b0a1312aa7d1012388d 100644
--- a/content/browser/fileapi/fileapi_message_filter.cc
+++ b/content/browser/fileapi/fileapi_message_filter.cc
@@ -565,7 +565,7 @@ void FileAPIMessageFilter::OnFinishBuildingStream(const GURL& url) {
DCHECK_CURRENTLY_ON(BrowserThread::IO);
scoped_refptr<Stream> stream(GetStreamForURL(url));
if (stream.get())
- stream->Finalize();
+ stream->Finalize(0);
tyoshino (SeeGerritForStatus) 2016/09/27 10:24:56 net::OK
jam 2016/09/27 15:41:07 Done.
}
void FileAPIMessageFilter::OnAbortBuildingStream(const GURL& url) {
« no previous file with comments | « no previous file | content/browser/loader/navigation_resource_handler.cc » ('j') | content/browser/streams/stream_unittest.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698