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

Unified Diff: third_party/WebKit/Source/modules/fetch/BytesConsumer.h

Issue 2229313002: Make FetchDataLoader accept BytesConsumer (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@remove-c++-rs
Patch Set: fix Created 4 years, 4 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: third_party/WebKit/Source/modules/fetch/BytesConsumer.h
diff --git a/third_party/WebKit/Source/modules/fetch/BytesConsumer.h b/third_party/WebKit/Source/modules/fetch/BytesConsumer.h
index d822e752df476febb9e598f6952d587187f2a366..83288c716115866b3a4e7f522fab1fc9abb9a68c 100644
--- a/third_party/WebKit/Source/modules/fetch/BytesConsumer.h
+++ b/third_party/WebKit/Source/modules/fetch/BytesConsumer.h
@@ -58,7 +58,9 @@ public:
// This function is called when the state changes. This function can
// be called more than needed, i.e., it can be called even when the
- // state is not actually changed.
+ // state is not actually changed, but it is guaranteed that this
+ // function cannot be called when the state is closed or errored before
hiroshige 2016/08/15 07:09:57 nit: how about "... but it is guaranteed that this
yhirano 2016/08/16 09:02:21 Done.
+ // this function is called.
//
// This function is not called when the state change is trigerred by
// public methods called by a user. For example, when a user reads

Powered by Google App Engine
This is Rietveld 408576698