Atomic: Don’t keep track of Cursor planes

ui/ozone/platform/drm/gpu/hardware_display_plane.h

Index: ui/ozone/platform/drm/gpu/hardware_display_plane.h
diff --git a/ui/ozone/platform/drm/gpu/hardware_display_plane.h b/ui/ozone/platform/drm/gpu/hardware_display_plane.h
index 778f344813a512762ab6ef1041174dbb878a2f00..0277a05e5bfb5b552d98a03c8cbf610482c41e94 100644
--- a/ui/ozone/platform/drm/gpu/hardware_display_plane.h
+++ b/ui/ozone/platform/drm/gpu/hardware_display_plane.h
@@ -22,11 +22,15 @@ class DrmDevice;
 
 class OZONE_EXPORT HardwareDisplayPlane {
  public:
+  enum Type { Primary, Overlay, Cursor, Dummy };

[dnicoara, 2015/08/20 14:22:45]

As per the coding guide, please use MACRO_STYLE for enum values.

[kalyank, 2015/08/20 16:36:18]

Done.

+
   HardwareDisplayPlane(uint32_t plane_id, uint32_t possible_crtcs);
 
   virtual ~HardwareDisplayPlane();
 
-  virtual bool Initialize(DrmDevice* drm, const std::vector<uint32_t>& formats);
+  virtual bool Initialize(DrmDevice* drm,
+                          const std::vector<uint32_t>& formats,
+                          bool is_dummy);
 
   virtual bool IsSupportedFormat(uint32_t format) const;
 
@@ -35,11 +39,10 @@ class OZONE_EXPORT HardwareDisplayPlane {
   bool in_use() const { return in_use_; }
   void set_in_use(bool in_use) { in_use_ = in_use; }
 
-  bool is_dummy() const { return is_dummy_; }
-  void set_is_dummy(bool is_dummy) { is_dummy_ = is_dummy; }
-
   uint32_t plane_id() const { return plane_id_; }
 
+  Type type() const { return type_; }
+
   void set_owning_crtc(uint32_t crtc) { owning_crtc_ = crtc; }
   uint32_t owning_crtc() const { return owning_crtc_; }
 
@@ -48,7 +51,7 @@ class OZONE_EXPORT HardwareDisplayPlane {
   uint32_t possible_crtcs_ = 0;
   uint32_t owning_crtc_ = 0;
   bool in_use_ = false;
-  bool is_dummy_ = false;
+  Type type_ = Primary;
 
   DISALLOW_COPY_AND_ASSIGN(HardwareDisplayPlane);
 };