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

Unified Diff: chrome/browser/browsing_data/browsing_data_remover.h

Issue 12094106: Refactor: Simplify WaitableEventWatcher. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 7 years, 11 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: chrome/browser/browsing_data/browsing_data_remover.h
diff --git a/chrome/browser/browsing_data/browsing_data_remover.h b/chrome/browser/browsing_data/browsing_data_remover.h
index 7222237bee2f931b97357304d24963f6b995611f..9a5f8a7eb7ef68c80a2a25b5400050a2497e297b 100644
--- a/chrome/browser/browsing_data/browsing_data_remover.h
+++ b/chrome/browser/browsing_data/browsing_data_remover.h
@@ -50,11 +50,11 @@ struct SessionStorageUsageInfo;
// BrowsingDataRemover is responsible for removing data related to browsing:
// visits in url database, downloads, cookies ...
-class BrowsingDataRemover : public content::NotificationObserver,
+class BrowsingDataRemover : public content::NotificationObserver
#if defined(ENABLE_PLUGINS)
- public PepperFlashSettingsManager::Client,
+ , public PepperFlashSettingsManager::Client
#endif
- public base::WaitableEventWatcher::Delegate {
+ {
public:
// Time period ranges available when doing browsing data removals.
enum TimePeriod {
@@ -210,10 +210,8 @@ class BrowsingDataRemover : public content::NotificationObserver,
const content::NotificationSource& source,
const content::NotificationDetails& details) OVERRIDE;
- // WaitableEventWatcher implementation.
// Called when plug-in data has been cleared. Invokes NotifyAndDeleteIfDone.
- virtual void OnWaitableEventSignaled(
- base::WaitableEvent* waitable_event) OVERRIDE;
+ void OnWaitableEventSignaled(base::WaitableEvent* waitable_event);
#if defined(ENABLE_PLUGINS)
// PepperFlashSettingsManager::Client implementation.

Powered by Google App Engine
This is Rietveld 408576698