Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(21)

Unified Diff: components/domain_reliability/uploader.cc

Issue 2538313002: Fix data usage tracking for domain reliability service (Closed)
Patch Set: Rebased Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/domain_reliability/uploader.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/domain_reliability/uploader.cc
diff --git a/components/domain_reliability/uploader.cc b/components/domain_reliability/uploader.cc
index 3ded0849d3392ceeaf20d35d7e4125da3f1ca4ef..f7d4cd9a44c5534b76c2e22c571ef0c825f7c272 100644
--- a/components/domain_reliability/uploader.cc
+++ b/components/domain_reliability/uploader.cc
@@ -10,7 +10,6 @@
#include "base/callback.h"
#include "base/metrics/histogram_macros.h"
#include "base/supports_user_data.h"
-#include "components/data_use_measurement/core/data_use_user_data.h"
#include "components/domain_reliability/util.h"
#include "net/base/load_flags.h"
#include "net/base/net_errors.h"
@@ -83,8 +82,6 @@ class DomainReliabilityUploaderImpl
std::unique_ptr<net::URLFetcher> owned_fetcher =
net::URLFetcher::Create(0, upload_url, net::URLFetcher::POST, this);
net::URLFetcher* fetcher = owned_fetcher.get();
- data_use_measurement::DataUseUserData::AttachToFetcher(
- fetcher, data_use_measurement::DataUseUserData::DOMAIN_RELIABILITY);
fetcher->SetRequestContext(url_request_context_getter_.get());
fetcher->SetLoadFlags(net::LOAD_DO_NOT_SEND_COOKIES |
net::LOAD_DO_NOT_SAVE_COOKIES);
@@ -178,6 +175,12 @@ std::unique_ptr<DomainReliabilityUploader> DomainReliabilityUploader::Create(
}
// static
+bool DomainReliabilityUploader::OriginatedFromDomainReliability(
+ const net::URLRequest& request) {
+ return request.GetUserData(UploadUserData::kUserDataKey) != nullptr;
+}
+
+// static
int DomainReliabilityUploader::GetURLRequestUploadDepth(
const net::URLRequest& request) {
UploadUserData* data = static_cast<UploadUserData*>(
« no previous file with comments | « components/domain_reliability/uploader.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698