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

Unified Diff: third_party/WebKit/Source/platform/loader/fetch/Resource.cpp

Issue 2796163002: Migrate WTF::HashCountedSet::add() to ::insert() (Closed)
Patch Set: Created 3 years, 8 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/platform/loader/fetch/Resource.cpp
diff --git a/third_party/WebKit/Source/platform/loader/fetch/Resource.cpp b/third_party/WebKit/Source/platform/loader/fetch/Resource.cpp
index cb81f55b8fb38a5f340113ebc354fd4d4fb35453..1f11621b4146b6aba108405ad21c99439a4d4aee 100644
--- a/third_party/WebKit/Source/platform/loader/fetch/Resource.cpp
+++ b/third_party/WebKit/Source/platform/loader/fetch/Resource.cpp
@@ -370,7 +370,7 @@ void Resource::checkNotify() {
void Resource::markClientFinished(ResourceClient* client) {
if (m_clients.contains(client)) {
- m_finishedClients.add(client);
+ m_finishedClients.insert(client);
m_clients.remove(client);
}
}
@@ -659,7 +659,7 @@ void Resource::didAddClient(ResourceClient* c) {
if (isLoaded()) {
c->notifyFinished(this);
if (m_clients.contains(c)) {
- m_finishedClients.add(c);
+ m_finishedClients.insert(c);
m_clients.remove(c);
}
}
@@ -712,7 +712,7 @@ void Resource::addClient(ResourceClient* client,
willAddClientOrObserver(policy);
if (m_isRevalidating) {
- m_clients.add(client);
+ m_clients.insert(client);
return;
}
@@ -720,12 +720,12 @@ void Resource::addClient(ResourceClient* client,
// and the resource type supprts it, send it asynchronously.
if ((errorOccurred() || !response().isNull()) &&
!typeNeedsSynchronousCacheHit(getType()) && !m_needsSynchronousCacheHit) {
- m_clientsAwaitingCallback.add(client);
+ m_clientsAwaitingCallback.insert(client);
ResourceCallback::callbackHandler().schedule(this);
return;
}
- m_clients.add(client);
+ m_clients.insert(client);
didAddClient(client);
return;
}
@@ -814,7 +814,7 @@ void Resource::finishPendingClients() {
// Handle case (2) to skip removed clients.
if (!m_clientsAwaitingCallback.remove(client))
continue;
- m_clients.add(client);
+ m_clients.insert(client);
// When revalidation starts after waiting clients are scheduled and
// before they are added here. In such cases, we just add the clients
« no previous file with comments | « third_party/WebKit/Source/platform/heap/HeapTest.cpp ('k') | third_party/WebKit/Source/platform/wtf/HashCountedSet.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698