predictors: Move more methods from ResourcePrefetchPredictor into LoadingDataCollector.

chrome/browser/net/loading_predictor_observer.cc

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/net/loading_predictor_observer.h"
 
 #include <memory>
 #include <string>
 #include <utility>
 
 #include "base/memory/ptr_util.h"
 #include "base/metrics/histogram_macros.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/resource_request_info.h"
 #include "net/url_request/url_request.h"
 #include "url/gurl.h"
 
 namespace content {
 class WebContents;
 }
 
 using content::BrowserThread;
 using predictors::LoadingPredictor;
 using predictors::LoadingDataCollector;
-using URLRequestSummary =
-    predictors::ResourcePrefetchPredictor::URLRequestSummary;
+using URLRequestSummary = predictors::URLRequestSummary;

[alexilin, 2017/06/16 09:31:57]

nit:
Why not `using predictors::URLRequestSummary;`?

[trevordixon, 2017/06/16 10:53:19]

Done.

 
 namespace {
 
 // Enum for measuring statistics pertaining to observed request, responses and
 // redirects.
 enum RequestStats {
   REQUEST_STATS_TOTAL_RESPONSES = 0,
   REQUEST_STATS_TOTAL_PROCESSED_RESPONSES = 1,
   REQUEST_STATS_NO_RESOURCE_REQUEST_INFO = 2,  // Not recorded (never was).
   REQUEST_STATS_NO_RENDER_FRAME_ID_FROM_REQUEST_INFO = 3,  // Not recorded.
@@ skipping 190 matching lines @@
   if (!TryToFillNavigationID(&summary->navigation_id, web_contents_getter,
                              main_frame_url, creation_time)) {
     return;
   }
   if (summary->resource_type == content::RESOURCE_TYPE_MAIN_FRAME)
     predictor_->OnMainFrameResponse(*summary);
   predictor_->loading_data_collector()->RecordURLResponse(*summary);
 }
 
 }  // namespace chrome_browser_net