Bug 1602333 - InternalRequest cleanup: default member initialize mSynchronous r=dom-workers-and-storage-reviewers,ytausky

Fetch spec says "A request has an associated synchronous flag. Unless stated otherwise it is unset.""

Differential Revision: https://phabricator.services.mozilla.com/D60072

--HG--
extra : moz-landing-system : lando

dom/fetch/InternalRequest.cpp

@@ -102,7 +102,6 @@ InternalRequest::InternalRequest(const nsACString& aURL,
       ,
       mSameOriginDataURL(true),
       mSkipServiceWorker(false),
-      mSynchronous(false),
       mUnsafeRequest(false) {
   MOZ_ASSERT(!aURL.IsEmpty());
   AddURL(aURL, aFragment);
@@ -132,7 +131,6 @@ InternalRequest::InternalRequest(
       ,
       mSameOriginDataURL(true),
       mSkipServiceWorker(false),
-      mSynchronous(false),
       mUnsafeRequest(false) {
   MOZ_ASSERT(!aURL.IsEmpty());
   AddURL(aURL, aFragment);

dom/fetch/InternalRequest.h

@@ -404,7 +404,7 @@ class InternalRequest final {
   MOZ_INIT_OUTSIDE_CTOR bool mPreserveContentCodings;
   MOZ_INIT_OUTSIDE_CTOR bool mSameOriginDataURL;
   MOZ_INIT_OUTSIDE_CTOR bool mSkipServiceWorker;
-  MOZ_INIT_OUTSIDE_CTOR bool mSynchronous;
+  bool mSynchronous = false;
   MOZ_INIT_OUTSIDE_CTOR bool mUnsafeRequest;
   bool mUseURLCredentials = false;
   // This is only set when Request.overrideContentPolicyType() has been set.