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

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: Another rebase 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
« no previous file with comments | « Source/modules/serviceworkers/Response.h ('k') | Source/modules/serviceworkers/Response.idl » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/modules/serviceworkers/Response.cpp
diff --git a/Source/modules/serviceworkers/Response.cpp b/Source/modules/serviceworkers/Response.cpp
index ad17d28fcfb789e05fcff2b060e664e2e3e8ef1e..224c949ba2bd71b1ebc39e3eed38b9a2e7f5898c 100644
--- a/Source/modules/serviceworkers/Response.cpp
+++ b/Source/modules/serviceworkers/Response.cpp
@@ -11,12 +11,14 @@
namespace WebCore {
-PassRefPtr<Response> Response::create(Blob* body, const Dictionary& responseInit, ExceptionState& exceptionState)
+DEFINE_EMPTY_DESTRUCTOR_WILL_BE_REMOVED(Response);
+
+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 +32,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 +85,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 +133,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 +152,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 +160,10 @@ Response::Response(PassRefPtr<FetchResponseData> response)
ScriptWrappable::init(this);
}
+void Response::trace(Visitor* visitor)
+{
+ visitor->trace(m_response);
+ visitor->trace(m_headers);
+}
+
} // namespace WebCore
« no previous file with comments | « Source/modules/serviceworkers/Response.h ('k') | Source/modules/serviceworkers/Response.idl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698