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

Unified Diff: third_party/WebKit/Source/bindings/core/v8/ReadableStreamOperationsTest.cpp

Issue 1506023003: Response construction with a ReadableStream (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years 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/bindings/core/v8/ReadableStreamOperationsTest.cpp
diff --git a/third_party/WebKit/Source/bindings/core/v8/ReadableStreamOperationsTest.cpp b/third_party/WebKit/Source/bindings/core/v8/ReadableStreamOperationsTest.cpp
index 6420acd0bc9b6e5c6d8c376259dede29824fdc0f..4b25d986ce83af6a6d6117eebf3088938e20b86c 100644
--- a/third_party/WebKit/Source/bindings/core/v8/ReadableStreamOperationsTest.cpp
+++ b/third_party/WebKit/Source/bindings/core/v8/ReadableStreamOperationsTest.cpp
@@ -21,24 +21,6 @@ namespace blink {
namespace {
-// Unpacks |item|, stores the value of "done" member to |done| and returns
-// the value of "value" member. Returns an empty handle when errored.
-v8::MaybeLocal<v8::Value> unpack(ScriptState* scriptState, v8::Local<v8::Value> item, bool* done)
-{
- if (!item->IsObject()) {
- V8ThrowException::throwTypeError(scriptState->isolate(), "The iteration item is not an object.");
- return v8::MaybeLocal<v8::Value>();
- }
- v8::Local<v8::Object> object = item.As<v8::Object>();
- v8::Local<v8::Value> doneValue;
- if (!v8Call(object->Get(scriptState->context(), v8String(scriptState->isolate(), "done")), doneValue))
- return v8::MaybeLocal<v8::Value>();
- v8::MaybeLocal<v8::Value> r = object->Get(scriptState->context(), v8String(scriptState->isolate(), "value"));
- if (!r.IsEmpty())
- *done = doneValue->ToBoolean()->Value();
- return r;
-}
-
class NotReached : public ScriptFunction {
public:
static v8::Local<v8::Function> createFunction(ScriptState* scriptState)
@@ -75,7 +57,8 @@ public:
m_isSet = true;
v8::TryCatch block(v.scriptState()->isolate());
v8::Local<v8::Value> value;
- if (!v8Call(unpack(v.scriptState(), v.v8Value(), &m_isDone), value)) {
+ v8::Local<v8::Value> item = v.v8Value();
+ if (!item->IsObject() || !v8Call(v8UnpackIteratorResult(v.scriptState(), item.As<v8::Object>(), &m_isDone), value)) {
m_isValid = false;
return;
}

Powered by Google App Engine
This is Rietveld 408576698