Create mojom and StructTraits for ui/display/types/display_mode.cc

ui/display/mojo/display_mode_mojo.h

Index: ui/display/mojo/display_mode_mojo.h
diff --git a/ui/display/mojo/display_mode_mojo.h b/ui/display/mojo/display_mode_mojo.h
new file mode 100644
index 0000000000000000000000000000000000000000..42036ee5700cba991ac95cbb3e10265397cd5400
--- /dev/null
+++ b/ui/display/mojo/display_mode_mojo.h
@@ -0,0 +1,24 @@
+// 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_TYPES_MOJO_DISPLAY_MODE_MOJO_H_

[kylechar, 2017/01/24 21:15:07]

UI_DISPLAY_DISPLAY_MODE_MOJO_H_

[thanhph1, 2017/01/25 16:26:17]

Done.

+#define UI_DISPLAY_TYPES_MOJO_DISPLAY_MODE_MOJO_H_
+
+#include "ui/display/display_export.h"
+#include "ui/display/types/display_mode.h"
+
+namespace display {
+
+//Move-only class.

[kylechar, 2017/01/24 21:15:07]

This comment isn't helpful. It would be better to explain why this
implementation of DisplayMode exists.

[thanhph1, 2017/01/25 16:26:17]

Done.

+class DISPLAY_EXPORT DisplayModeMojo: public DisplayMode {
+  public:

[kylechar, 2017/01/24 21:15:07]

You will also need a constructor that lets you set size, is_interlaced and
refresh_rate.

[thanhph1, 2017/01/25 16:26:17]

Done.

+    DisplayModeMojo();
+    ~DisplayModeMojo() override;
+    DisplayModeMojo(DisplayModeMojo &&) = default;

[kylechar, 2017/01/24 21:15:07]

Move to cc file.

[thanhph1, 2017/01/25 16:26:17]

Done.

+    DisplayModeMojo& operator=(DisplayModeMojo &&) = default;

[kylechar, 2017/01/24 21:15:07]

Move to cc file.

+};

[kylechar, 2017/01/24 21:15:07]

Missing DISALLOW_COPY_AND_ASSIGN.

+
+} // namespace
+
+#endif // UI_DISPLAY_TYPES_MOJO_DISPLAY_MODE_MOJO_H_