Record data use of user traffic by different core page transition types

components/data_use_measurement/content/content_url_request_classifier.h

 // Copyright 2016 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.
 
 #ifndef COMPONENTS_DATA_USE_MEASUREMENT_CONTENT_CONTENT_URL_REQUEST_CLASSIFIER_H_
 #define COMPONENTS_DATA_USE_MEASUREMENT_CONTENT_CONTENT_URL_REQUEST_CLASSIFIER_H_
 
 #include "components/data_use_measurement/core/url_request_classifier.h"
 
 namespace net {
 class URLRequest;
 }
 
 namespace data_use_measurement {
 
 // Returns true if the URLRequest |request| is initiated by user traffic.
 bool IsUserRequest(const net::URLRequest& request);
 
 class ContentURLRequestClassifier : public URLRequestClassifier {
  public:
   bool IsUserRequest(const net::URLRequest& request) const override;

[RyanSturm, 2017/01/10 18:32:23]

Two things: Can you add "// UrlRequestClassifier:" comment and can you make
these methods private so they are only accessible through the base class

[Raj, 2017/01/10 22:21:57]

Done.

+
+  void RecordPageTransitionUMA(int32_t page_transition,
+                               int64_t received_bytes) const override;
 };
 
 }  // namespace data_use_measurement
 
 #endif  // COMPONENTS_DATA_USE_MEASUREMENT_CONTENT_CONTENT_URL_REQUEST_CLASSIFIER_H_