Write mojom and StructTraits for DisplaySnapshot.

ui/display/display_snapshot_mojo.h

+// Copyright 2017 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_MOJO_DISPLAY_SNAPSHOT_MOJO_H_
+#define UI_DISPLAY_MOJO_DISPLAY_SNAPSHOT_MOJO_H_
+
+#include "base/macros.h"
+#include "ui/display/display_export.h"
+#include "ui/display/types/display_snapshot.h"
+
+namespace display {
+
+// This class is used to create DisplaySnapshot IPC struct traits.
+class DISPLAY_EXPORT DisplaySnapshotMojo : public DisplaySnapshot {
+ public:
+  DisplaySnapshotMojo(int64_t display_id,
+                      const gfx::Point& origin,
+                      const gfx::Size& physical_size,
+                      DisplayConnectionType type,
+                      bool is_aspect_preserving_scaling,
+                      bool has_overscan,
+                      bool has_color_correction_matrix,
+                      std::string display_name,
+                      const base::FilePath& sys_path,
+                      int64_t product_id,
+                      DisplayModeList modes,
+                      const std::vector<uint8_t>& edid,
+                      const DisplayMode* current_mode,
+                      bool current_mode_exist,
+                      const DisplayMode* native_mode,
+                      bool native_mode_exist,
+                      const gfx::Size& maximum_cursor_size);
+  ~DisplaySnapshotMojo() override;
+  std::string ToString() const override;
+  const bool& current_mode_exist() const { return current_mode_exist_; }

[kylechar, 2017/02/13 22:01:23]

These two methods shouldn't be necessary, see comment below.

[thanhph1, 2017/02/14 20:20:52]

Removed!

+  const bool& native_mode_exist() const { return native_mode_exist_; }
+
+ private:
+  bool current_mode_exist_;

[kylechar, 2017/02/13 22:01:23]

Ditto for these members, this is just current_mode_ == nullptr, except you only
set it in the constructor. When you change the current mode this will not be the
right value.

[thanhph1, 2017/02/14 20:20:52]

Removed!

+  bool native_mode_exist_;
+
+  DISALLOW_COPY_AND_ASSIGN(DisplaySnapshotMojo);
+};
+
+}  // namespace display
+
+#endif  // UI_DISPLAY_MOJO_DISPLAY_SNAPSHOT_MOJO_H_