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

Unified Diff: base/power_monitor/power_observer.h

Issue 544913003: Revert of Check to ensure PowerObservers are added and removed on the same thread (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « base/power_monitor/power_monitor.cc ('k') | base/power_monitor/power_observer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/power_monitor/power_observer.h
diff --git a/base/power_monitor/power_observer.h b/base/power_monitor/power_observer.h
index e7d40a3a6d5a3eb07e1bb45632d236eea3858d21..6be70bba9d5368dfa0cf7a4f145247a1334430ec 100644
--- a/base/power_monitor/power_observer.h
+++ b/base/power_monitor/power_observer.h
@@ -7,20 +7,14 @@
#include "base/base_export.h"
#include "base/compiler_specific.h"
-#include "base/memory/scoped_ptr.h"
-#include "base/threading/thread_checker.h"
namespace base {
class BASE_EXPORT PowerObserver {
- friend class PowerMonitor;
-
public:
- PowerObserver();
-
// Notification of a change in power status of the computer, such
// as from switching between battery and A/C power.
- virtual void OnPowerStateChange(bool on_battery_power) {}
+ virtual void OnPowerStateChange(bool on_battery_power) {};
// Notification that the system is suspending.
virtual void OnSuspend() {}
@@ -29,12 +23,7 @@
virtual void OnResume() {}
protected:
- virtual ~PowerObserver();
-
- private:
- // Allows PowerMonitor to ensure this observer is added and removed
- // from the same thread, which is required by the ObserverListThreadSafe.
- scoped_ptr<base::ThreadChecker> power_monitor_thread_checker_;
+ virtual ~PowerObserver() {}
};
} // namespace base
« no previous file with comments | « base/power_monitor/power_monitor.cc ('k') | base/power_monitor/power_observer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698