Refactoring display configuration state to allow generic state objects

ui/display/chromeos/display_mode.h

Index: ui/display/chromeos/display_mode.h
diff --git a/ui/display/chromeos/display_mode.h b/ui/display/chromeos/display_mode.h
new file mode 100644
index 0000000000000000000000000000000000000000..57a259ac0e519e92f9fdecc82b8c87e2e9db33e7
--- /dev/null
+++ b/ui/display/chromeos/display_mode.h
@@ -0,0 +1,33 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef UI_DISPLAY_CHROMEOS_DISPLAY_MODE_H_
+#define UI_DISPLAY_CHROMEOS_DISPLAY_MODE_H_
+
+#include "base/macros.h"
+#include "ui/display/display_export.h"
+#include "ui/gfx/geometry/size.h"
+
+namespace ui {
+
+class DISPLAY_EXPORT DisplayMode {

[Daniel Erat, 2014/03/05 01:56:12]

nit: add a short comment describing what this corresponds to (and maybe
mentioning that there are implementation-specific derived classes)

[dnicoara, 2014/03/05 17:29:25]

Done.

+ public:
+  DisplayMode(const gfx::Size& size, bool interlaced, float refresh_rate);
+  virtual ~DisplayMode();
+
+  const gfx::Size& size() const { return size_; }
+  bool is_interlaced() const { return is_interlaced_; }
+  float refresh_rate() const { return refresh_rate_; }
+
+ private:
+  gfx::Size size_;
+  bool is_interlaced_;
+  float refresh_rate_;
+
+  DISALLOW_COPY_AND_ASSIGN(DisplayMode);
+};
+
+}  // namespace ui
+
+#endif  // UI_DISPLAY_CHROMEOS_DISPLAY_MODE_H_