[1/2][Ozone-Drm] Refactor GPU display management

ui/ozone/platform/drm/common/drm_util.h

 // 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_OZONE_PLATFORM_DRM_COMMON_DRM_UTIL_H_
 #define UI_OZONE_PLATFORM_DRM_COMMON_DRM_UTIL_H_
 
 #include "base/files/file_path.h"
 #include "base/macros.h"
 #include "base/memory/scoped_vector.h"
+#include "ui/ozone/common/gpu/ozone_gpu_message_params.h"
 #include "ui/ozone/platform/drm/common/scoped_drm_types.h"
 
 typedef struct _drmModeModeInfo drmModeModeInfo;
 
 namespace ui {
 
 // Representation of the information required to initialize and configure a
 // native display.
 class HardwareDisplayControllerInfo {
  public:
@@ skipping 11 matching lines @@
   DISALLOW_COPY_AND_ASSIGN(HardwareDisplayControllerInfo);
 };
 
 // Looks-up and parses the native display configurations returning all available
 // displays.
 ScopedVector<HardwareDisplayControllerInfo> GetAvailableDisplayControllerInfos(
     int fd);
 
 bool SameMode(const drmModeModeInfo& lhs, const drmModeModeInfo& rhs);
 
+DisplayMode_Params ParseDrmMode(const drmModeModeInfo& mode);

[spang, 2015/05/11 22:21:32]

I don't think "Parse" is the right verb here. The input is not a string. Maybe
"Convert".

[dnicoara, 2015/05/12 15:17:10]

Actually, I don't like the "DrmMode" part too. Renamed it.

+
+DisplaySnapshot_Params ParseDrmDisplay(HardwareDisplayControllerInfo* info,
+                                       int fd,
+                                       size_t index);

[spang, 2015/05/11 22:21:32]

What is 'index'? Needs more docs. 

[dnicoara, 2015/05/12 15:17:10]

Done.

+
 }  // namespace ui
 
 #endif  // UI_OZONE_PLATFORM_DRM_COMMON_DRM_UTIL_H_