Remove ResourceFetcher::DeadResourceStatsRecorder

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 604 matching lines @@
       if (resource->IsLinkPreload() && !params.IsLinkPreload())
         resource->SetLinkPreload(false);
       break;
   }
   if (!resource)
     return nullptr;
 
   // TODO(yoav): turn to a DCHECK. See https://crbug.com/690632
   CHECK_EQ(resource->GetType(), factory.GetType());
 
-  if (!resource->IsAlive())
-    dead_stats_recorder_.Update(policy);
-
   if (policy != kUse)
     resource->SetIdentifier(identifier);
 
   // TODO(yoav): It is not clear why preloads are exempt from this check. Can we
   // remove the exemption?
   if (!params.IsSpeculativePreload() || policy != kUse) {
     // When issuing another request for a resource that is already in-flight
     // make sure to not demote the priority of the in-flight request. If the new
     // request isn't at the same priority as the in-flight request, only allow
     // promotions. This can happen when a visible image's priority is increased
@@ skipping 891 matching lines @@
   ResourceRequest resource_request(url);
   resource_request.SetRequestContext(request_context);
   FetchParameters params(resource_request, initiator_name, resource->Options());
   Context().CanRequest(resource->GetType(), resource->LastResourceRequest(),
                        resource->LastResourceRequest().Url(), params.Options(),
                        SecurityViolationReportingPolicy::kReport,
                        params.GetOriginRestriction());
   RequestLoadStarted(resource->Identifier(), resource, params, kUse);
 }
 
-ResourceFetcher::DeadResourceStatsRecorder::DeadResourceStatsRecorder()
-    : use_count_(0), revalidate_count_(0), load_count_(0) {}
-
-ResourceFetcher::DeadResourceStatsRecorder::~DeadResourceStatsRecorder() {
-  DEFINE_THREAD_SAFE_STATIC_LOCAL(
-      CustomCountHistogram, hit_count_histogram,
-      new CustomCountHistogram("WebCore.ResourceFetcher.HitCount", 0, 1000,
-                               50));
-  hit_count_histogram.Count(use_count_);
-  DEFINE_THREAD_SAFE_STATIC_LOCAL(
-      CustomCountHistogram, revalidate_count_histogram,
-      new CustomCountHistogram("WebCore.ResourceFetcher.RevalidateCount", 0,
-                               1000, 50));
-  revalidate_count_histogram.Count(revalidate_count_);
-  DEFINE_THREAD_SAFE_STATIC_LOCAL(
-      CustomCountHistogram, load_count_histogram,
-      new CustomCountHistogram("WebCore.ResourceFetcher.LoadCount", 0, 1000,
-                               50));
-  load_count_histogram.Count(load_count_);
-}
-
-void ResourceFetcher::DeadResourceStatsRecorder::Update(
-    RevalidationPolicy policy) {
-  switch (policy) {
-    case kReload:
-    case kLoad:
-      ++load_count_;
-      return;
-    case kRevalidate:
-      ++revalidate_count_;
-      return;
-    case kUse:
-      ++use_count_;
-      return;
-  }
-}
-
 DEFINE_TRACE(ResourceFetcher) {
   visitor->Trace(context_);
   visitor->Trace(archive_);
   visitor->Trace(loaders_);
   visitor->Trace(non_blocking_loaders_);
   visitor->Trace(document_resources_);
   visitor->Trace(preloads_);
   visitor->Trace(resource_timing_info_map_);
 }
 
 }  // namespace blink