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

Unified Diff: components/domain_reliability/monitor.cc

Issue 238863005: Domain Reliability: Remove browsing data when requested. (Closed) Base URL: http://git.chromium.org/chromium/src.git@domrel_bakedin
Patch Set: Add unittest Created 6 years, 7 months 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
Index: components/domain_reliability/monitor.cc
diff --git a/components/domain_reliability/monitor.cc b/components/domain_reliability/monitor.cc
index 06d5f55f11a31eb252b831bab8696e1346f6c460..52881d95c385502e9dd7156507b9f520ba7c86aa 100644
--- a/components/domain_reliability/monitor.cc
+++ b/components/domain_reliability/monitor.cc
@@ -65,7 +65,9 @@ DomainReliabilityMonitor::DomainReliabilityMonitor(
DomainReliabilityScheduler::Params::GetFromFieldTrialsOrDefaults()),
dispatcher_(time_.get()),
uploader_(
- DomainReliabilityUploader::Create(url_request_context_getter_)) {
+ DomainReliabilityUploader::Create(url_request_context_getter_)),
+ was_cleared_(false),
+ cleared_mode_(MAX_CLEAR_MODE) {
DCHECK(OnIOThread());
}
@@ -82,7 +84,9 @@ DomainReliabilityMonitor::DomainReliabilityMonitor(
DomainReliabilityScheduler::Params::GetFromFieldTrialsOrDefaults()),
dispatcher_(time_.get()),
uploader_(
- DomainReliabilityUploader::Create(url_request_context_getter_)) {
+ DomainReliabilityUploader::Create(url_request_context_getter_)),
+ was_cleared_(false),
+ cleared_mode_(MAX_CLEAR_MODE) {
DCHECK(OnIOThread());
}
@@ -128,6 +132,29 @@ void DomainReliabilityMonitor::OnCompleted(net::URLRequest* request,
}
}
+void DomainReliabilityMonitor::ClearBrowsingData(
+ DomainReliabilityClearMode mode) {
+ DCHECK(OnIOThread());
+
+ was_cleared_ = true;
+ cleared_mode_ = mode;
+
+ switch (mode) {
+ case CLEAR_BEACONS: {
+ ContextMap::const_iterator it;
+ for (it = contexts_.begin(); it != contexts_.end(); ++it)
+ it->second->ClearBeacons();
+ break;
+ };
+ case CLEAR_CONTEXTS:
+ STLDeleteContainerPairSecondPointers(
+ contexts_.begin(), contexts_.end());
+ break;
+ default:
Mike West 2014/05/08 11:00:40 Nit: If you replace this with 'case MAX_CLEAR_MODE
Deprecated (see juliatuttle) 2014/05/08 15:47:39 Done.
+ NOTREACHED();
+ }
+}
+
DomainReliabilityContext* DomainReliabilityMonitor::AddContextForTesting(
scoped_ptr<const DomainReliabilityConfig> config) {
return AddContext(config.Pass());

Powered by Google App Engine
This is Rietveld 408576698