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

Unified Diff: ui/display/chromeos/display_configurator.h

Issue 972153004: Make ui::DisplayConfigurator a singleton (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Make a Singleton Created 5 years, 9 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: ui/display/chromeos/display_configurator.h
diff --git a/ui/display/chromeos/display_configurator.h b/ui/display/chromeos/display_configurator.h
index b61d3653a490975bdf4bbe7774e4678ae6cdd73a..8d5cba79e4f6e17977762a0e364302035e827fd4 100644
--- a/ui/display/chromeos/display_configurator.h
+++ b/ui/display/chromeos/display_configurator.h
@@ -13,6 +13,7 @@
#include "base/event_types.h"
#include "base/memory/scoped_ptr.h"
+#include "base/memory/singleton.h"
#include "base/memory/weak_ptr.h"
#include "base/observer_list.h"
#include "base/timer/timer.h"
@@ -34,6 +35,10 @@ class DisplaySnapshot;
class NativeDisplayDelegate;
class UpdateDisplayConfigurationTask;
+namespace test {
+class DisplayConfiguratorTest;
+} // namespace test
+
// This class interacts directly with the system display configurator.
class DISPLAY_EXPORT DisplayConfigurator : public NativeDisplayObserver {
public:
@@ -175,9 +180,6 @@ class DISPLAY_EXPORT DisplayConfigurator : public NativeDisplayObserver {
const DisplaySnapshot& display,
const gfx::Size& size);
- DisplayConfigurator();
- ~DisplayConfigurator() override;
-
MultipleDisplayState display_state() const { return current_display_state_; }
chromeos::DisplayPowerState requested_power_state() const {
return requested_power_state_;
@@ -287,9 +289,17 @@ class DISPLAY_EXPORT DisplayConfigurator : public NativeDisplayObserver {
bool SetColorCalibrationProfile(int64_t display_id,
ui::ColorCalibrationProfile new_profile);
+ // static
+ static DisplayConfigurator* GetInstance();
+
private:
+ friend struct DefaultSingletonTraits<DisplayConfigurator>;
+ friend class ui::test::DisplayConfiguratorTest;
class DisplayLayoutManagerImpl;
+ DisplayConfigurator();
+ ~DisplayConfigurator() override;
+
// Mapping a display_id to a protection request bitmask.
typedef std::map<int64_t, uint32_t> ContentProtections;
// Mapping a client to its protection request.
« no previous file with comments | « extensions/shell/browser/shell_desktop_controller_aura.cc ('k') | ui/display/chromeos/display_configurator.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698