Introduce "navigate" mode in Requests

third_party/WebKit/Source/modules/fetch/RequestInit.cpp

 // 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.
 
 #include "config.h"
 #include "modules/fetch/RequestInit.h"
 
 #include "bindings/core/v8/Dictionary.h"
 #include "bindings/core/v8/V8ArrayBuffer.h"
 #include "bindings/core/v8/V8ArrayBufferView.h"
 #include "bindings/core/v8/V8Binding.h"
 #include "bindings/core/v8/V8Blob.h"
 #include "bindings/core/v8/V8FormData.h"
 #include "core/fileapi/Blob.h"
 #include "core/html/FormData.h"
 #include "modules/fetch/FetchBlobDataConsumerHandle.h"
 #include "modules/fetch/FetchFormDataConsumerHandle.h"
 #include "modules/fetch/Headers.h"
 #include "platform/blob/BlobData.h"
 #include "platform/network/EncodedFormData.h"
 #include "platform/weborigin/ReferrerPolicy.h"
 
 namespace blink {
 
 RequestInit::RequestInit(ExecutionContext* context, const Dictionary& options, ExceptionState& exceptionState)
-    : opaque(false), isReferrerSet(false)
+    : opaque(false), isAnyMembersSet(false)
 {
     bool areAnyMembersSet = false;

[yhirano, 2015/10/16 18:15:53]

You can use the member variable directly instead of having a local variable.

[shiva.jm, 2015/10/19 09:03:49]

Done.

 
     areAnyMembersSet = DictionaryHelper::get(options, "method", method) || areAnyMembersSet;
     areAnyMembersSet = DictionaryHelper::get(options, "headers", headers) || areAnyMembersSet;
     if (!headers) {
         Vector<Vector<String>> headersVector;
         if (DictionaryHelper::get(options, "headers", headersVector, exceptionState)) {
             headers = Headers::create(headersVector, exceptionState);
             areAnyMembersSet = true;
         } else {
             areAnyMembersSet = DictionaryHelper::get(options, "headers", headersDictionary) || areAnyMembersSet;
@@ skipping 15 matching lines @@
         // If any of init's members are present, unset request's
         // omit-Origin-header flag, set request's referrer to "client",
         // and request's referrer policy to the empty string.
         //
         // We need to use "about:client" instead of |clientReferrerString|,
         // because "about:client" => |clientReferrerString| conversion is done
         // in Request::createRequestWithRequestOrString.
         referrer = Referrer("about:client", ReferrerPolicyDefault);
         if (isReferrerStringSet)
             referrer.referrer = referrerString;
-        isReferrerSet = true;
+        isAnyMembersSet = true;
     }
 
     if (!isBodySet || v8Body->IsUndefined() || v8Body->IsNull())
         return;
     v8::Isolate* isolate = toIsolate(context);
     if (v8Body->IsArrayBuffer()) {
         body = FetchFormDataConsumerHandle::create(V8ArrayBuffer::toImpl(v8::Local<v8::Object>::Cast(v8Body)));
     } else if (v8Body->IsArrayBufferView()) {
         body = FetchFormDataConsumerHandle::create(V8ArrayBufferView::toImpl(v8::Local<v8::Object>::Cast(v8Body)));
     } else if (V8Blob::hasInstance(v8Body, isolate)) {
         RefPtr<BlobDataHandle> blobDataHandle = V8Blob::toImpl(v8::Local<v8::Object>::Cast(v8Body))->blobDataHandle();
         contentType = blobDataHandle->type();
         body = FetchBlobDataConsumerHandle::create(context, blobDataHandle.release());
     } else if (V8FormData::hasInstance(v8Body, isolate)) {
         FormData* domFormData = V8FormData::toImpl(v8::Local<v8::Object>::Cast(v8Body));
         opaque = domFormData->opaque();
 
         RefPtr<EncodedFormData> formData = domFormData->encodeMultiPartFormData();
         // Here we handle formData->boundary() as a C-style string. See
         // FormDataEncoder::generateUniqueBoundaryString.
         contentType = AtomicString("multipart/form-data; boundary=", AtomicString::ConstructFromLiteral) + formData->boundary().data();
         body = FetchFormDataConsumerHandle::create(context, formData.release());
     } else if (v8Body->IsString()) {
         contentType = "text/plain;charset=UTF-8";
         body = FetchFormDataConsumerHandle::create(toUSVString(isolate, v8Body, exceptionState));
     }
 }
 
 }