| OLD | NEW |
| (Empty) | |
| 1 // Copyright 2016 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. |
| 4 |
| 5 #include "components/data_use_measurement/content/content_url_request_classifier
.h" |
| 6 |
| 7 #include "content/public/browser/resource_request_info.h" |
| 8 #include "net/url_request/url_request.h" |
| 9 |
| 10 namespace data_use_measurement { |
| 11 |
| 12 bool IsUserRequest(const net::URLRequest& request) { |
| 13 // The presence of ResourecRequestInfo in |request| implies that this request |
| 14 // was created for a content::WebContents. For now we could add a condition to |
| 15 // check ProcessType in info is content::PROCESS_TYPE_RENDERER, but it won't |
| 16 // be compatible with upcoming PlzNavigate architecture. So just existence of |
| 17 // ResourceRequestInfo is verified, and the current check should be compatible |
| 18 // with upcoming changes in PlzNavigate. |
| 19 // TODO(rajendrant): Verify this condition for different use cases. See |
| 20 // crbug.com/626063. |
| 21 return content::ResourceRequestInfo::ForRequest(&request) != nullptr; |
| 22 } |
| 23 |
| 24 bool ContentURLRequestClassifier::IsUserRequest( |
| 25 const net::URLRequest& request) const { |
| 26 return data_use_measurement::IsUserRequest(request); |
| 27 } |
| 28 |
| 29 } // namespace data_use_measurement |
| OLD | NEW |