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

Unified Diff: chromeos/dbus/dbus_thread_manager.cc

Issue 457563002: Remove unused DBusThreadManagerObserver (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chromeos/dbus/dbus_thread_manager.h ('k') | chromeos/dbus/dbus_thread_manager_observer.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chromeos/dbus/dbus_thread_manager.cc
diff --git a/chromeos/dbus/dbus_thread_manager.cc b/chromeos/dbus/dbus_thread_manager.cc
index e372005dc52663ba1a7723190b1fe63da211a494..ecd10182d32ae259b6bd3064a2c8aa714b25eff2 100644
--- a/chromeos/dbus/dbus_thread_manager.cc
+++ b/chromeos/dbus/dbus_thread_manager.cc
@@ -5,7 +5,6 @@
#include "chromeos/dbus/dbus_thread_manager.h"
#include "base/command_line.h"
-#include "base/observer_list.h"
#include "base/sys_info.h"
#include "base/threading/thread.h"
#include "chromeos/chromeos_switches.h"
@@ -22,7 +21,6 @@
#include "chromeos/dbus/cros_disks_client.h"
#include "chromeos/dbus/cryptohome_client.h"
#include "chromeos/dbus/dbus_client.h"
-#include "chromeos/dbus/dbus_thread_manager_observer.h"
#include "chromeos/dbus/debug_daemon_client.h"
#include "chromeos/dbus/easy_unlock_client.h"
#include "chromeos/dbus/fake_dbus_thread_manager.h"
@@ -279,9 +277,6 @@ class DBusThreadManagerImpl : public DBusThreadManager {
}
virtual ~DBusThreadManagerImpl() {
- FOR_EACH_OBSERVER(DBusThreadManagerObserver, observers_,
- OnDBusThreadManagerDestroying(this));
-
// PowerPolicyController's destructor depends on PowerManagerClient.
power_policy_controller_.reset();
@@ -297,16 +292,6 @@ class DBusThreadManagerImpl : public DBusThreadManager {
}
// DBusThreadManager overrides:
- virtual void AddObserver(DBusThreadManagerObserver* observer) OVERRIDE {
- DCHECK(observer);
- observers_.AddObserver(observer);
- }
-
- virtual void RemoveObserver(DBusThreadManagerObserver* observer) OVERRIDE {
- DCHECK(observer);
- observers_.RemoveObserver(observer);
- }
-
virtual dbus::Bus* GetSystemBus() OVERRIDE {
return system_bus_.get();
}
@@ -466,10 +451,6 @@ class DBusThreadManagerImpl : public DBusThreadManager {
power_policy_controller_.reset(new PowerPolicyController);
}
- // Note: Keep this before other members so they can call AddObserver() in
- // their c'tors.
- ObserverList<DBusThreadManagerObserver> observers_;
-
scoped_ptr<base::Thread> dbus_thread_;
scoped_refptr<dbus::Bus> system_bus_;
scoped_ptr<DBusClientBundle> client_bundle_;
« no previous file with comments | « chromeos/dbus/dbus_thread_manager.h ('k') | chromeos/dbus/dbus_thread_manager_observer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698