[Fetch API] Reflect spec changes of bodyUsed property

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

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef Body_h
 #define Body_h
 
 #include "bindings/core/v8/ScriptPromise.h"
 #include "bindings/core/v8/ScriptWrappable.h"
 #include "core/dom/ActiveDOMObject.h"
@@ skipping 17 matching lines @@
     WILL_BE_USING_GARBAGE_COLLECTED_MIXIN(Body);
 public:
     explicit Body(ExecutionContext*);
     ~Body() override { }
 
     ScriptPromise arrayBuffer(ScriptState*);
     ScriptPromise blob(ScriptState*);
     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;
 
     DEFINE_INLINE_VIRTUAL_TRACE()
     {
         ActiveDOMObject::trace(visitor);
     }
 
 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
 
 #endif // Body_h