Migrate WTF::HashSet::add() to ::insert() [part 1 of N]

third_party/WebKit/Source/platform/loader/fetch/ResourceFetcher.cpp

 /*
     Copyright (C) 1998 Lars Knoll (knoll@mpi-hd.mpg.de)
     Copyright (C) 2001 Dirk Mueller (mueller@kde.org)
     Copyright (C) 2002 Waldo Bastian (bastian@kde.org)
     Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All
     rights reserved.
     Copyright (C) 2009 Torch Mobile Inc. http://www.torchmobile.com/
 
     This library is free software; you can redistribute it and/or
     modify it under the terms of the GNU Library General Public
@@ skipping 302 matching lines @@
     info->clearLoadTimings();
     info->setLoadFinishTime(info->initialTime());
     m_scheduledResourceTimingReports.push_back(std::move(info));
     if (!m_resourceTimingReportTimer.isActive())
       m_resourceTimingReportTimer.startOneShot(0, BLINK_FROM_HERE);
   }
 
   if (m_validatedURLs.size() >= kMaxValidatedURLsSize) {
     m_validatedURLs.clear();
   }
-  m_validatedURLs.add(request.resourceRequest().url());
+  m_validatedURLs.insert(request.resourceRequest().url());
 }
 
 static std::unique_ptr<TracedValue> urlForTraceEvent(const KURL& url) {
   std::unique_ptr<TracedValue> value = TracedValue::create();
   value->setString("url", url.getString());
   return value;
 }
 
 Resource* ResourceFetcher::resourceForStaticData(
     const FetchRequest& request,
@@ skipping 82 matching lines @@
 void ResourceFetcher::moveCachedNonBlockingResourceToBlocking(
     Resource* resource,
     const FetchRequest& request) {
   // TODO(yoav): Test that non-blocking resources (video/audio/track) continue
   // to not-block even after being preloaded and discovered.
   if (resource && resource->loader() &&
       resource->isLoadEventBlockingResourceType() &&
       m_nonBlockingLoaders.contains(resource->loader()) &&
       resource->isLinkPreload() && !request.forPreload()) {
     m_nonBlockingLoaders.remove(resource->loader());
-    m_loaders.add(resource->loader());
+    m_loaders.insert(resource->loader());
   }
 }
 
 void ResourceFetcher::updateMemoryCacheStats(Resource* resource,
                                              RevalidationPolicy policy,
                                              const FetchRequest& request,
                                              const ResourceFactory& factory,
                                              bool isStaticData) const {
   if (isStaticData)
     return;
@@ skipping 587 matching lines @@
 void ResourceFetcher::preloadStarted(Resource* resource) {
   if (m_preloads && m_preloads->contains(resource))
     return;
   resource->increasePreloadCount();
 
   if (!m_preloads)
     m_preloads = new HeapListHashSet<Member<Resource>>;
   m_preloads->add(resource);
 
   if (m_preloadedURLsForTest)
-    m_preloadedURLsForTest->add(resource->url().getString());
+    m_preloadedURLsForTest->insert(resource->url().getString());
 }
 
 void ResourceFetcher::enableIsPreloadedForTest() {
   if (m_preloadedURLsForTest)
     return;
   m_preloadedURLsForTest = WTF::wrapUnique(new HashSet<String>);
 
   if (m_preloads) {
     for (const auto& resource : *m_preloads)
-      m_preloadedURLsForTest->add(resource->url().getString());
+      m_preloadedURLsForTest->insert(resource->url().getString());
   }
 }
 
 bool ResourceFetcher::isPreloadedForTest(const KURL& url) const {
   DCHECK(m_preloadedURLsForTest);
   return m_preloadedURLsForTest->contains(url.getString());
 }
 
 void ResourceFetcher::clearPreloads(ClearPreloadsPolicy policy) {
   if (!m_preloads)
@@ skipping 123 matching lines @@
 
   resource->error(error);
 
   handleLoadCompletion(resource);
 }
 
 void ResourceFetcher::moveResourceLoaderToNonBlocking(ResourceLoader* loader) {
   DCHECK(loader);
   // TODO(yoav): Convert CHECK to DCHECK if no crash reports come in.
   CHECK(m_loaders.contains(loader));
-  m_nonBlockingLoaders.add(loader);
+  m_nonBlockingLoaders.insert(loader);
   m_loaders.remove(loader);
 }
 
 bool ResourceFetcher::startLoad(Resource* resource) {
   DCHECK(resource);
   DCHECK(resource->stillNeedsLoad());
   if (!context().shouldLoadNewResource(resource->getType())) {
     memoryCache()->remove(resource);
     return false;
   }
@@ skipping 11 matching lines @@
   // Resource requests from suborigins should not be intercepted by the service
   // worker of the physical origin. This has the effect that, for now,
   // suborigins do not work with service workers. See
   // https://w3c.github.io/webappsec-suborigins/.
   SecurityOrigin* sourceOrigin = context().getSecurityOrigin();
   if (sourceOrigin && sourceOrigin->hasSuborigin())
     request.setSkipServiceWorker(WebURLRequest::SkipServiceWorker::All);
 
   ResourceLoader* loader = ResourceLoader::create(this, resource);
   if (resource->shouldBlockLoadEvent())
-    m_loaders.add(loader);
+    m_loaders.insert(loader);
   else
-    m_nonBlockingLoaders.add(loader);
+    m_nonBlockingLoaders.insert(loader);
 
   storePerformanceTimingInitiatorInformation(resource);
   resource->setFetcherSecurityOrigin(sourceOrigin);
 
   loader->activateCacheAwareLoadingIfNeeded(request);
   loader->start(request);
   return true;
 }
 
 void ResourceFetcher::removeResourceLoader(ResourceLoader* loader) {
@@ skipping 271 matching lines @@
   visitor->trace(m_context);
   visitor->trace(m_archive);
   visitor->trace(m_loaders);
   visitor->trace(m_nonBlockingLoaders);
   visitor->trace(m_documentResources);
   visitor->trace(m_preloads);
   visitor->trace(m_resourceTimingInfoMap);
 }
 
 }  // namespace blink