Index: components/data_use_measurement/core/data_use_measurement.cc |
diff --git a/components/data_use_measurement/content/data_use_measurement.cc b/components/data_use_measurement/core/data_use_measurement.cc |
similarity index 86% |
rename from components/data_use_measurement/content/data_use_measurement.cc |
rename to components/data_use_measurement/core/data_use_measurement.cc |
index 18f910482f7c1465f0bb270e3ea0747b97dde75c..87a0810f35f9da50559eb6e2e917b0555bf927b0 100644 |
--- a/components/data_use_measurement/content/data_use_measurement.cc |
+++ b/components/data_use_measurement/core/data_use_measurement.cc |
@@ -2,12 +2,10 @@ |
// Use of this source code is governed by a BSD-style license that can be |
// found in the LICENSE file. |
-#include "components/data_use_measurement/content/data_use_measurement.h" |
- |
+#include <components/data_use_measurement/core/data_use_measurement.h> |
#include "base/metrics/histogram.h" |
#include "base/metrics/sparse_histogram.h" |
#include "base/strings/stringprintf.h" |
-#include "content/public/browser/resource_request_info.h" |
#include "net/base/network_change_notifier.h" |
#include "net/base/upload_data_stream.h" |
#include "net/http/http_response_headers.h" |
@@ -43,28 +41,24 @@ void IncreaseSparseHistogramByValue(const std::string& name, |
} // namespace |
-DataUseMeasurement::DataUseMeasurement() |
+DataUseMeasurement::DataUseMeasurement( |
+ DataUseMeasurementDelegate* data_use_measurement_delegate) |
+ : |
#if defined(OS_ANDROID) |
- : app_state_(base::android::APPLICATION_STATE_HAS_RUNNING_ACTIVITIES), |
+ app_state_(base::android::APPLICATION_STATE_HAS_RUNNING_ACTIVITIES), |
app_listener_(new base::android::ApplicationStatusListener( |
base::Bind(&DataUseMeasurement::OnApplicationStateChange, |
- base::Unretained(this)))) |
+ base::Unretained(this)))), |
#endif |
-{ |
+ data_use_measurement_delegate_(data_use_measurement_delegate) { |
} |
DataUseMeasurement::~DataUseMeasurement(){}; |
void DataUseMeasurement::ReportDataUseUMA( |
const net::URLRequest* request) const { |
- const content::ResourceRequestInfo* info = |
- content::ResourceRequestInfo::ForRequest(request); |
- // Having |info| is the sign of a request for a web content from user. For now |
- // we could add a condition to check ProcessType in info is |
- // content::PROCESS_TYPE_RENDERER, but it won't be compatible with upcoming |
- // PlzNavigate architecture. So just existence of |info| is verified, and the |
- // current check should be compatible with upcoming changes in PlzNavigate. |
- bool is_user_traffic = info != nullptr; |
+ bool is_user_traffic = |
+ data_use_measurement_delegate_->RequestIsFromUser(request); |
// Counts rely on URLRequest::GetTotalReceivedBytes() and |
// URLRequest::GetTotalSentBytes(), which does not include the send path, |