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

Side by Side Diff: chrome/browser/browsing_data/browsing_data_remover.cc

Issue 2240883002: Make ClearBrowsingDataHandler only observe its own removal task (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/browsing_data/browsing_data_remover.h" 5 #include "chrome/browser/browsing_data/browsing_data_remover.h"
6 6
7 #include <map> 7 #include <map>
8 #include <set> 8 #include <set>
9 #include <string> 9 #include <string>
10 10
(...skipping 306 matching lines...) Expand 10 before | Expand all | Expand 10 after
317 } 317 }
318 318
319 void BrowsingDataRemover::Shutdown() { 319 void BrowsingDataRemover::Shutdown() {
320 history_task_tracker_.TryCancelAll(); 320 history_task_tracker_.TryCancelAll();
321 template_url_sub_.reset(); 321 template_url_sub_.reset();
322 } 322 }
323 323
324 void BrowsingDataRemover::SetRemoving(bool is_removing) { 324 void BrowsingDataRemover::SetRemoving(bool is_removing) {
325 DCHECK_NE(is_removing_, is_removing); 325 DCHECK_NE(is_removing_, is_removing);
326 is_removing_ = is_removing; 326 is_removing_ = is_removing;
327 FOR_EACH_OBSERVER(Observer, observer_list_,
328 OnBrowsingDataRemoving(is_removing));
329 } 327 }
330 328
331 void BrowsingDataRemover::Remove(const TimeRange& time_range, 329 void BrowsingDataRemover::Remove(const TimeRange& time_range,
332 int remove_mask, 330 int remove_mask,
333 int origin_type_mask) { 331 int origin_type_mask) {
334 RemoveInternal(time_range, remove_mask, origin_type_mask, 332 RemoveInternal(time_range, remove_mask, origin_type_mask,
335 std::unique_ptr<RegistrableDomainFilterBuilder>(), nullptr); 333 std::unique_ptr<RegistrableDomainFilterBuilder>(), nullptr);
336 } 334 }
337 335
338 void BrowsingDataRemover::RemoveAndReply( 336 void BrowsingDataRemover::RemoveAndReply(
(...skipping 1072 matching lines...) Expand 10 before | Expand all | Expand 10 after
1411 waiting_for_clear_offline_page_data_ = false; 1409 waiting_for_clear_offline_page_data_ = false;
1412 NotifyIfDone(); 1410 NotifyIfDone();
1413 } 1411 }
1414 #endif 1412 #endif
1415 1413
1416 void BrowsingDataRemover::OnClearedDomainReliabilityMonitor() { 1414 void BrowsingDataRemover::OnClearedDomainReliabilityMonitor() {
1417 DCHECK_CURRENTLY_ON(BrowserThread::UI); 1415 DCHECK_CURRENTLY_ON(BrowserThread::UI);
1418 waiting_for_clear_domain_reliability_monitor_ = false; 1416 waiting_for_clear_domain_reliability_monitor_ = false;
1419 NotifyIfDone(); 1417 NotifyIfDone();
1420 } 1418 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698