Oilpan: add transition types to remaining Fetch and ServiceWorker objects

Source/modules/serviceworkers/Request.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 "Request.h"
 
 #include "bindings/core/v8/Dictionary.h"
 #include "core/dom/ExecutionContext.h"
 #include "core/fetch/CrossOriginAccessControl.h"
 #include "core/fetch/ResourceLoaderOptions.h"
 #include "core/loader/ThreadableLoader.h"
 #include "core/xml/XMLHttpRequest.h"
 #include "modules/serviceworkers/FetchManager.h"
 #include "modules/serviceworkers/RequestInit.h"
 #include "platform/NotImplemented.h"
 #include "platform/network/HTTPParsers.h"
 #include "platform/network/ResourceRequest.h"
 #include "platform/weborigin/Referrer.h"
 #include "public/platform/WebServiceWorkerRequest.h"
 #include "public/platform/WebURLRequest.h"
 
 namespace WebCore {
 
 namespace {
 
-PassRefPtr<Request> createRequestWithRequestData(PassRefPtr<FetchRequestData> request, const RequestInit& init, FetchRequestData::Mode mode, FetchRequestData::Credentials credentials, ExceptionState& exceptionState)
+PassRefPtrWillBeRawPtr<Request> createRequestWithRequestData(PassRefPtrWillBeRawPtr<FetchRequestData> request, const RequestInit& init, FetchRequestData::Mode mode, FetchRequestData::Credentials credentials, ExceptionState& exceptionState)
 {
     // "6. Let |mode| be |init|'s mode member if it is present, and
     // |fallbackMode| otherwise."
     // "7. If |mode| is non-null, set |request|'s mode to |mode|."
     if (init.mode == "same-origin") {
         request->setMode(FetchRequestData::SameOriginMode);
     } else if (init.mode == "no-cors") {
         request->setMode(mode = FetchRequestData::NoCORSMode);
     } else if (init.mode == "cors") {
         request->setMode(FetchRequestData::CORSMode);
@@ skipping 30 matching lines @@
         if (!isValidHTTPToken(init.method)) {
             exceptionState.throwTypeError("'" + init.method + "' is not a valid HTTP method.");
             return nullptr;
         }
         // FIXME: "2. Add case correction as in XMLHttpRequest?"
         // "3. Set |request|'s method to |method|."
         request->setMethod(XMLHttpRequest::uppercaseKnownHTTPMethod(AtomicString(init.method)));
     }
     // "11. Let |r| be a new Request object associated with |request|, Headers
     // object, and FetchBodyStream object."
-    RefPtr<Request> r = Request::create(request);
+    RefPtrWillBeRawPtr<Request> r = Request::create(request);
 
     // "12. Let |headers| be a copy of |r|'s Headers object."
     // "13. If |init|'s headers member is present, set |headers| to |init|'s
     // headers member."
     // We don't create a copy of r's Headers object when init's headers member
     // is present.
-    RefPtr<Headers> headers;
+    RefPtrWillBeRawPtr<Headers> headers = nullptr;
     if (!init.headers && init.headersDictionary.isUndefinedOrNull()) {
         headers = r->headers()->createCopy();
     }
     // "14. Empty |r|'s request's header list."
     r->request()->headerList()->clearList();
 
     // "15. If |r|'s request's mode is no CORS, run these substeps:
     if (r->request()->mode() == FetchRequestData::NoCORSMode) {
         // "1. If |r|'s request's method is not a simple method, throw a
         // TypeError."
@@ skipping 18 matching lines @@
     if (exceptionState.hadException())
         return nullptr;
     // FIXME: Support body.
     // "20. Return |r|."
     return r.release();
 }
 
 
 } // namespace
 
-PassRefPtr<Request> Request::create(ExecutionContext* context, const String& input, ExceptionState& exceptionState)
+DEFINE_EMPTY_DESTRUCTOR_WILL_BE_REMOVED(Request);
+
+PassRefPtrWillBeRawPtr<Request> Request::create(ExecutionContext* context, const String& input, ExceptionState& exceptionState)
 {
     return create(context, input, Dictionary(), exceptionState);
 }
 
-PassRefPtr<Request> Request::create(ExecutionContext* context, const String& input, const Dictionary& init, ExceptionState& exceptionState)
+PassRefPtrWillBeRawPtr<Request> Request::create(ExecutionContext* context, const String& input, const Dictionary& init, ExceptionState& exceptionState)
 {
     // "1. Let |request| be |input|'s associated request, if |input| is a
     // Request object, and a new request otherwise."
-    RefPtr<FetchRequestData> request(FetchRequestData::create(context));
+    RefPtrWillBeRawPtr<FetchRequestData> request(FetchRequestData::create(context));
     // "2. Set |request| to a restricted copy of itself."
     request = request->createRestrictedCopy(context, SecurityOrigin::create(context->url()));
     // "5. If |input| is a string, run these substeps:"
     // "1. Let |parsedURL| be the result of parsing |input| with entry settings
     // object's API base URL."
     KURL parsedURL = context->completeURL(input);
     // "2. If |parsedURL| is failure, throw a TypeError."
     if (!parsedURL.isValid()) {
         exceptionState.throwTypeError("Invalid URL");
         return nullptr;
     }
     // "3. Set |request|'s url to |parsedURL|."
     request->setURL(parsedURL);
     // "4. Set |fallbackMode| to CORS."
     // "5. Set |fallbackCredentials| to omit."
     return createRequestWithRequestData(request.release(), RequestInit(init), FetchRequestData::CORSMode, FetchRequestData::OmitCredentials, exceptionState);
 }
 
 
-PassRefPtr<Request> Request::create(ExecutionContext* context, Request* input, ExceptionState& exceptionState)
+PassRefPtrWillBeRawPtr<Request> Request::create(ExecutionContext* context, Request* input, ExceptionState& exceptionState)
 {
     return create(context, input, Dictionary(), exceptionState);
 }
 
-PassRefPtr<Request> Request::create(ExecutionContext* context, Request* input, const Dictionary& init, ExceptionState& exceptionState)
+PassRefPtrWillBeRawPtr<Request> Request::create(ExecutionContext* context, Request* input, const Dictionary& init, ExceptionState& exceptionState)
 {
     // "1. Let |request| be |input|'s associated request, if |input| is a
     // Request object, and a new request otherwise."
     // "2. Set |request| to a restricted copy of itself."
-    RefPtr<FetchRequestData> request(input->request()->createRestrictedCopy(context, SecurityOrigin::create(context->url())));
+    RefPtrWillBeRawPtr<FetchRequestData> request(input->request()->createRestrictedCopy(context, SecurityOrigin::create(context->url())));
     // "3. Let |fallbackMode| be null."
     // "4. Let |fallbackCredentials| be null."
     // Instead of using null as a special fallback value, just pass the current
     // mode and credentials; it has the same effect.
     const FetchRequestData::Mode currentMode = request->mode();
     const FetchRequestData::Credentials currentCredentials = request->credentials();
     return createRequestWithRequestData(request.release(), RequestInit(init), currentMode, currentCredentials, exceptionState);
 }
 
-PassRefPtr<Request> Request::create(PassRefPtr<FetchRequestData> request)
+PassRefPtrWillBeRawPtr<Request> Request::create(PassRefPtrWillBeRawPtr<FetchRequestData> request)
 {
-    return adoptRef(new Request(request));
+    return adoptRefWillBeNoop(new Request(request));
 }
 
-Request::Request(PassRefPtr<FetchRequestData> request)
+Request::Request(PassRefPtrWillBeRawPtr<FetchRequestData> request)
     : m_request(request)
     , m_headers(Headers::create(m_request->headerList()))
 {
     m_headers->setGuard(Headers::RequestGuard);
     ScriptWrappable::init(this);
 }
 
-PassRefPtr<Request> Request::create(const blink::WebServiceWorkerRequest& webRequest)
+PassRefPtrWillBeRawPtr<Request> Request::create(const blink::WebServiceWorkerRequest& webRequest)
 {
-    return adoptRef(new Request(webRequest));
+    return adoptRefWillBeNoop(new Request(webRequest));
 }
 
 Request::Request(const blink::WebServiceWorkerRequest& webRequest)
     : m_request(FetchRequestData::create(webRequest))
     , m_headers(Headers::create(m_request->headerList()))
 {
     m_headers->setGuard(Headers::RequestGuard);
     ScriptWrappable::init(this);
 }
 
@@ skipping 57 matching lines @@
 
 PassOwnPtr<ResourceRequest> Request::createResourceRequest() const
 {
     OwnPtr<ResourceRequest> request = adoptPtr(new ResourceRequest(url()));
     request->setRequestContext(blink::WebURLRequest::RequestContextFetch);
     request->setHTTPMethod("GET");
     // FIXME: Fill more info.
     return request.release();
 }
 
+void Request::trace(Visitor* visitor)
+{
+    visitor->trace(m_request);
+    visitor->trace(m_headers);
+}
+
 } // namespace WebCore