Record data use of favicon

components/data_use_measurement/content/content_url_request_classifier.h

 // Copyright 2016 The Chromium Authors. All rights reserved.

[RyanSturm, 2017/03/22 22:00:26]

You have a typo in the CL description s/Thsi CL records/This CL records/

[Raj, 2017/03/23 17:06:46]

I did not find that. May be you edited and fixed it?

 // 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/03/22 22:00:26]

nit: Can you add a comment to the top of this group of overrides like "//
URLRequestClassifier:" and remove the vertical spaces between the methods.

[Raj, 2017/03/23 17:06:46]

Done.

 
   DataUseUserData::DataUseContentType GetContentType(
       const net::URLRequest& request,
       const net::HttpResponseHeaders& response_headers) const override;
+
+  bool IsFavIconRequest(const net::URLRequest& request) const override;
 };
 
 }  // namespace data_use_measurement
 
 #endif  // COMPONENTS_DATA_USE_MEASUREMENT_CONTENT_CONTENT_URL_REQUEST_CLASSIFIER_H_