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

Unified Diff: Source/modules/serviceworkers/Response.cpp

Issue 384633002: Oilpan: add transition types to remaining Fetch and ServiceWorker objects (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Rebased Created 6 years, 5 months 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: Source/modules/serviceworkers/Response.cpp
diff --git a/Source/modules/serviceworkers/Response.cpp b/Source/modules/serviceworkers/Response.cpp
index ad17d28fcfb789e05fcff2b060e664e2e3e8ef1e..fdd65239cbbdff3fe81fb4c1d2f4f9bce4061947 100644
--- a/Source/modules/serviceworkers/Response.cpp
+++ b/Source/modules/serviceworkers/Response.cpp
@@ -11,12 +11,12 @@
namespace WebCore {
-PassRefPtr<Response> Response::create(Blob* body, const Dictionary& responseInit, ExceptionState& exceptionState)
+PassRefPtrWillBeRawPtr<Response> Response::create(Blob* body, const Dictionary& responseInit, ExceptionState& exceptionState)
{
return create(body, ResponseInit(responseInit), exceptionState);
}
-PassRefPtr<Response> Response::create(Blob* body, const ResponseInit& responseInit, ExceptionState& exceptionState)
+PassRefPtrWillBeRawPtr<Response> Response::create(Blob* body, const ResponseInit& responseInit, ExceptionState& exceptionState)
{
// "1. If |init|'s status member is not in the range 200 to 599, throw a
// RangeError."
@@ -30,7 +30,7 @@ PassRefPtr<Response> Response::create(Blob* body, const ResponseInit& responseIn
// "3. Let |r| be a new Response object, associated with a new response,
// Headers object, and FetchBodyStream object."
- RefPtr<Response> r = adoptRef(new Response());
+ RefPtrWillBeRawPtr<Response> r = adoptRefWillBeNoop(new Response());
// "4. Set |r|'s response's status to |init|'s status member."
r->m_response->setStatus(responseInit.status);
@@ -83,9 +83,9 @@ PassRefPtr<Response> Response::create(Blob* body, const ResponseInit& responseIn
return r.release();
}
-PassRefPtr<Response> Response::create(PassRefPtr<FetchResponseData> response)
+PassRefPtrWillBeRawPtr<Response> Response::create(PassRefPtrWillBeRawPtr<FetchResponseData> response)
{
- return adoptRef(new Response(response));
+ return adoptRefWillBeNoop(new Response(response));
}
String Response::type() const
@@ -131,7 +131,7 @@ String Response::statusText() const
return m_response->statusMessage();
}
-PassRefPtr<Headers> Response::headers() const
+PassRefPtrWillBeRawPtr<Headers> Response::headers() const
{
// "The headers attribute's getter must return the associated Headers object."
return m_headers;
@@ -150,7 +150,7 @@ Response::Response()
ScriptWrappable::init(this);
}
-Response::Response(PassRefPtr<FetchResponseData> response)
+Response::Response(PassRefPtrWillBeRawPtr<FetchResponseData> response)
: m_response(response)
, m_headers(Headers::create(m_response->headerList()))
{
@@ -158,4 +158,10 @@ Response::Response(PassRefPtr<FetchResponseData> response)
ScriptWrappable::init(this);
}
+void Response::trace(Visitor* visitor)
+{
+ visitor->trace(m_response);
+ visitor->trace(m_headers);
+}
+
} // namespace WebCore

Powered by Google App Engine
This is Rietveld 408576698