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

Unified Diff: chromeos/dbus/fake_power_manager_client.h

Issue 1206733002: ChromeOs Power Emulation Impl (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Updated everying!!! Created 5 years, 5 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: chromeos/dbus/fake_power_manager_client.h
diff --git a/chromeos/dbus/fake_power_manager_client.h b/chromeos/dbus/fake_power_manager_client.h
index d6cd0429f975b899ec2a64fc8f8158dce391a83d..356035ca217e08fa907937b9090b03b220b4935a 100644
--- a/chromeos/dbus/fake_power_manager_client.h
+++ b/chromeos/dbus/fake_power_manager_client.h
@@ -9,8 +9,11 @@
#include "base/basictypes.h"
#include "base/macros.h"
+#include "base/memory/scoped_ptr.h"
#include "base/observer_list.h"
+#include "chromeos/chromeos_export.h"
#include "chromeos/dbus/power_manager/policy.pb.h"
+#include "chromeos/dbus/power_manager/power_supply_properties.pb.h"
#include "chromeos/dbus/power_manager/suspend.pb.h"
#include "chromeos/dbus/power_manager_client.h"
@@ -19,12 +22,13 @@ namespace chromeos {
// A fake implementation of PowerManagerClient. This remembers the policy passed
// to SetPolicy() and the user of this class can inspect the last set policy by
// get_policy().
-class FakePowerManagerClient : public PowerManagerClient {
+class CHROMEOS_EXPORT FakePowerManagerClient : public PowerManagerClient {
public:
FakePowerManagerClient();
~FakePowerManagerClient() override;
power_manager::PowerManagementPolicy& policy() { return policy_; }
+ const power_manager::PowerSupplyProperties& props() const { return props_; }
int num_request_restart_calls() const { return num_request_restart_calls_; }
int num_request_shutdown_calls() const { return num_request_shutdown_calls_; }
int num_set_policy_calls() const { return num_set_policy_calls_; }
@@ -67,16 +71,26 @@ class FakePowerManagerClient : public PowerManagerClient {
// Notifies observers that the power button has been pressed or released.
void SendPowerButtonEvent(bool down, const base::TimeTicks& timestamp);
+ // Updates |props_| and notifies observers of its changes.
+ void UpdatePowerProperties(
+ const power_manager::PowerSupplyProperties& power_props);
+
private:
// Callback that will be run by asynchronous suspend delays to report
// readiness.
void HandleSuspendReadiness();
+ // Notifies |observers_| that |props_| has been updated.
+ void NotifyObservers();
+
base::ObserverList<Observer> observers_;
// Last policy passed to SetPolicy().
power_manager::PowerManagementPolicy policy_;
+ // Power status received from the power manager.
+ power_manager::PowerSupplyProperties props_;
+
// Number of times that various methods have been called.
int num_request_restart_calls_;
int num_request_shutdown_calls_;
@@ -92,6 +106,10 @@ class FakePowerManagerClient : public PowerManagerClient {
// Delegate for managing power consumption of Chrome's renderer processes.
base::WeakPtr<RenderProcessManagerDelegate> render_process_manager_delegate_;
+ // Note: This should remain the last member so it'll be destroyed and
+ // invalidate its weak pointers before any other members are destroyed.
+ base::WeakPtrFactory<FakePowerManagerClient> weak_ptr_factory_;
+
DISALLOW_COPY_AND_ASSIGN(FakePowerManagerClient);
};

Powered by Google App Engine
This is Rietveld 408576698