[Fetch API] Reflect spec changes of bodyUsed property

third_party/WebKit/Source/modules/fetch/Body.h

Index: third_party/WebKit/Source/modules/fetch/Body.h
diff --git a/third_party/WebKit/Source/modules/fetch/Body.h b/third_party/WebKit/Source/modules/fetch/Body.h
index 1e9ee5374eb14ee5d5a8cdba5b51b8c763dc7da7..16972505f5ad2e935a97f730ff9f83d6e5413b07 100644
--- a/third_party/WebKit/Source/modules/fetch/Body.h
+++ b/third_party/WebKit/Source/modules/fetch/Body.h
@@ -35,12 +35,12 @@ public:
     ScriptPromise formData(ScriptState*);
     ScriptPromise json(ScriptState*);
     ScriptPromise text(ScriptState*);
-    ReadableByteStream* body();
+    ReadableByteStream* bodyWithUseCounter();
     virtual BodyStreamBuffer* bodyBuffer() = 0;
     virtual const BodyStreamBuffer* bodyBuffer() const = 0;
 
-    bool bodyUsed();
-    void setBodyPassed() { m_bodyPassed = true; }
+    virtual bool bodyUsed();
+    bool isBodyLocked();

[hiroshige, 2015/11/19 12:50:36]

In the Fetch spec, |locked| is only defined for Response (not for Request) and
used in Response's clone() and consume body.
However, in this CL, isBodyLocked() is also used for Request: Request's consume
body, Request::clone().
Is this OK?
(I assumes this is because we cannot observe intermediate state of Request body
while reading and thus isBodyLocked() always returns false where it is used for
Request, and perhaps there's a future plan to introduce |locked| to Request. If
so, leaving a brief comment might be helpful.)

[yhirano, 2015/11/19 13:19:03]

It's OK, I'm introducing it right now[1] and it doesn't change the behavior.

1: https://github.com/yutakahirano/fetch-with-streams/pull/62

 
     // ActiveDOMObject override.
     bool hasPendingActivity() const override;
@@ -51,9 +51,10 @@ public:
     }
 
 private:
+    ReadableByteStream* body();
     virtual String mimeType() const = 0;
 
-    bool m_bodyPassed;
+    bool m_opaque;

[hiroshige, 2015/11/19 12:50:36]

Seems not used and thus we can delete m_opaque.

[yhirano, 2015/11/19 13:19:03]

Sorry, it's a rebase error. Fixed.

 };
 
 } // namespace blink