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

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: Only forward-declare in the header file Created 4 years, 3 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 313 matching lines...) Expand 10 before | Expand all | Expand 10 after
324 } 324 }
325 325
326 void BrowsingDataRemover::Shutdown() { 326 void BrowsingDataRemover::Shutdown() {
327 history_task_tracker_.TryCancelAll(); 327 history_task_tracker_.TryCancelAll();
328 template_url_sub_.reset(); 328 template_url_sub_.reset();
329 } 329 }
330 330
331 void BrowsingDataRemover::SetRemoving(bool is_removing) { 331 void BrowsingDataRemover::SetRemoving(bool is_removing) {
332 DCHECK_NE(is_removing_, is_removing); 332 DCHECK_NE(is_removing_, is_removing);
333 is_removing_ = is_removing; 333 is_removing_ = is_removing;
334 FOR_EACH_OBSERVER(Observer, observer_list_,
335 OnBrowsingDataRemoving(is_removing));
336 } 334 }
337 335
338 void BrowsingDataRemover::Remove(const TimeRange& time_range, 336 void BrowsingDataRemover::Remove(const TimeRange& time_range,
339 int remove_mask, 337 int remove_mask,
340 int origin_type_mask) { 338 int origin_type_mask) {
341 RemoveInternal(time_range, remove_mask, origin_type_mask, 339 RemoveInternal(time_range, remove_mask, origin_type_mask,
342 std::unique_ptr<RegistrableDomainFilterBuilder>(), nullptr); 340 std::unique_ptr<RegistrableDomainFilterBuilder>(), nullptr);
343 } 341 }
344 342
345 void BrowsingDataRemover::RemoveAndReply( 343 void BrowsingDataRemover::RemoveAndReply(
(...skipping 1135 matching lines...) Expand 10 before | Expand all | Expand 10 after
1481 waiting_for_clear_offline_page_data_ = false; 1479 waiting_for_clear_offline_page_data_ = false;
1482 NotifyIfDone(); 1480 NotifyIfDone();
1483 } 1481 }
1484 #endif 1482 #endif
1485 1483
1486 void BrowsingDataRemover::OnClearedDomainReliabilityMonitor() { 1484 void BrowsingDataRemover::OnClearedDomainReliabilityMonitor() {
1487 DCHECK_CURRENTLY_ON(BrowserThread::UI); 1485 DCHECK_CURRENTLY_ON(BrowserThread::UI);
1488 waiting_for_clear_domain_reliability_monitor_ = false; 1486 waiting_for_clear_domain_reliability_monitor_ = false;
1489 NotifyIfDone(); 1487 NotifyIfDone();
1490 } 1488 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698