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

Unified Diff: third_party/WebKit/Source/core/fetch/Resource.cpp

Issue 1686483002: Oilpan: Remove most WillBe types from the code base (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 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: third_party/WebKit/Source/core/fetch/Resource.cpp
diff --git a/third_party/WebKit/Source/core/fetch/Resource.cpp b/third_party/WebKit/Source/core/fetch/Resource.cpp
index fb32fc9cb99bbfe20480f148aad71201944379dc..0e51c7755426cd8328d865674732a43e8498b938 100644
--- a/third_party/WebKit/Source/core/fetch/Resource.cpp
+++ b/third_party/WebKit/Source/core/fetch/Resource.cpp
@@ -93,9 +93,9 @@ static inline bool shouldUpdateHeaderAfterRevalidation(const AtomicString& heade
class Resource::CacheHandler : public CachedMetadataHandler {
public:
- static PassOwnPtrWillBeRawPtr<CacheHandler> create(Resource* resource)
+ static RawPtr<CacheHandler> create(Resource* resource)
{
- return adoptPtrWillBeNoop(new CacheHandler(resource));
+ return (new CacheHandler(resource));
}
~CacheHandler() override { }
DECLARE_VIRTUAL_TRACE();
@@ -106,7 +106,7 @@ public:
private:
explicit CacheHandler(Resource*);
- RawPtrWillBeMember<Resource> m_resource;
+ Member<Resource> m_resource;
};
Resource::CacheHandler::CacheHandler(Resource* resource)
@@ -507,7 +507,7 @@ void Resource::clearCachedMetadata(CachedMetadataHandler::CacheType cacheType)
Platform::current()->cacheMetadata(m_response.url(), m_response.responseTime(), 0, 0);
}
-WeakPtrWillBeRawPtr<Resource> Resource::asWeakPtr()
+RawPtr<Resource> Resource::asWeakPtr()
{
#if ENABLE(OILPAN)
return this;
@@ -642,7 +642,7 @@ void Resource::removeClient(ResourceClient* client)
ResourceCallback::callbackHandler()->cancel(this);
if (!hasClients()) {
- RefPtrWillBeRawPtr<Resource> protect(this);
+ RawPtr<Resource> protect(this);
memoryCache()->makeDead(this);
allClientsRemoved();
@@ -677,7 +677,7 @@ void Resource::cancelTimerFired(Timer<Resource>* timer)
ASSERT_UNUSED(timer, timer == &m_cancelTimer);
if (hasClients() || !m_loader)
return;
- RefPtrWillBeRawPtr<Resource> protect(this);
+ RawPtr<Resource> protect(this);
m_loader->cancelIfNotFinishing();
if (m_status != Cached)
memoryCache()->remove(this);
@@ -936,7 +936,7 @@ Resource::ResourceCallback* Resource::ResourceCallback::callbackHandler()
//
// Keep it out of LSan's reach instead.
LEAK_SANITIZER_DISABLED_SCOPE;
- DEFINE_STATIC_LOCAL(OwnPtrWillBePersistent<ResourceCallback>, callbackHandler, (adoptPtrWillBeNoop(new ResourceCallback)));
+ DEFINE_STATIC_LOCAL(Persistent<ResourceCallback>, callbackHandler, ((new ResourceCallback)));
return callbackHandler.get();
}
@@ -975,8 +975,8 @@ bool Resource::ResourceCallback::isScheduled(Resource* resource) const
void Resource::ResourceCallback::runTask()
{
- WillBeHeapVector<RefPtrWillBeMember<Resource>> resources;
- for (const RefPtrWillBeMember<Resource>& resource : m_resourcesWithPendingClients)
+ HeapVector<Member<Resource>> resources;
+ for (const Member<Resource>& resource : m_resourcesWithPendingClients)
resources.append(resource.get());
m_resourcesWithPendingClients.clear();

Powered by Google App Engine
This is Rietveld 408576698