predictors: Improve comment clarity and remove unecessary namespace specifiers.

chrome/browser/predictors/loading_data_collector.h

Index: chrome/browser/predictors/loading_data_collector.h
diff --git a/chrome/browser/predictors/loading_data_collector.h b/chrome/browser/predictors/loading_data_collector.h
index 65274eef91284820edce6fbc3a8a1dbd26af836e..cb5c3217ae2c51a8e283571e36ed1713c5d4dd94 100644
--- a/chrome/browser/predictors/loading_data_collector.h
+++ b/chrome/browser/predictors/loading_data_collector.h
@@ -16,14 +16,13 @@ class URLRequest;
 
 namespace predictors {
 
-// Records navigation events as reported by various observers to the database
-// and stats collection classes. All the non-static methods of this class need
+// Records to the database and stats collection classes navigation events as
+// reported by various observers. All the non-static methods of this class need

[trevordixon, 2017/06/13 11:50:49]

In response to this comment from pasko on a previous CL
(https://codereview.chromium.org/2896713003/diff/20001/chrome/browser/predicto...):
"Does this say that the observers of the database record navigation events? I
thought it's the other way around - navigation observers are used for recording
events and updating the database."

It's ambiguous as written. Observers report events to this LoadingDataCollector,
then LoadingDataCollector records to the database and stats collection classes.
Hopefully this change makes it clear?

[pasko, 2017/06/13 12:53:28]

Yes, it makes it much clearer, thank you.

 // to be called on the UI thread.
 class LoadingDataCollector
     : public base::SupportsWeakPtr<LoadingDataCollector> {
  public:
-  explicit LoadingDataCollector(
-      predictors::ResourcePrefetchPredictor* predictor);
+  explicit LoadingDataCollector(ResourcePrefetchPredictor* predictor);

[trevordixon, 2017/06/13 11:50:49]

Got rid of unnecessary predictors:: namespace specifier.

(In response to a comment from alexilin on a previous CL.)

   ~LoadingDataCollector();
 
   // Thread safe.
@@ -37,11 +36,11 @@ class LoadingDataCollector
   // requests. Should only be called if the corresponding Should* functions
   // return true.
   void RecordURLRequest(
-      const predictors::ResourcePrefetchPredictor::URLRequestSummary& request);
+      const ResourcePrefetchPredictor::URLRequestSummary& request);
   void RecordURLResponse(
-      const predictors::ResourcePrefetchPredictor::URLRequestSummary& response);
+      const ResourcePrefetchPredictor::URLRequestSummary& response);
   void RecordURLRedirect(
-      const predictors::ResourcePrefetchPredictor::URLRequestSummary& response);
+      const ResourcePrefetchPredictor::URLRequestSummary& response);
 
   // Called when the main frame of a page completes loading.
   void RecordMainFrameLoadComplete(const NavigationID& navigation_id);
@@ -69,7 +68,7 @@ class LoadingDataCollector
 
   static void SetAllowPortInUrlsForTesting(bool state);
 
-  predictors::ResourcePrefetchPredictor* const predictor_;
+  ResourcePrefetchPredictor* const predictor_;
 };
 
 }  // namespace predictors