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

ui/display/BUILD.gn

 # 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.
 
 import("//build/config/ui.gni")
 import("//testing/test.gni")
 
 component("display") {
   sources = [
     "display.cc",
@@ skipping 12 matching lines @@
     "display_observer.cc",
     "display_observer.h",
     "display_switches.cc",
     "display_switches.h",
     "fake_display_delegate.cc",
     "fake_display_delegate.h",
     "fake_display_snapshot.cc",
     "fake_display_snapshot.h",
     "ios/screen_ios.mm",
     "mac/screen_mac.mm",
+    "mojo/display_mode_mojo.cc",

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

Move the files to ui/display/ instead.

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

Done, thanks!

+    "mojo/display_mode_mojo.h",
     "screen.cc",
     "screen.h",
     "screen_aura.cc",
     "screen_base.cc",
     "screen_base.h",
     "win/display_info.cc",
     "win/display_info.h",
     "win/dpi.cc",
     "win/dpi.h",
     "win/scaling_util.cc",
@@ skipping 97 matching lines @@
     "manager/chromeos/apply_content_protection_task_unittest.cc",
     "manager/chromeos/configure_displays_task_unittest.cc",
     "manager/chromeos/display_change_observer_unittest.cc",
     "manager/chromeos/display_configurator_unittest.cc",
     "manager/chromeos/query_content_protection_task_unittest.cc",
     "manager/chromeos/touch_transform_controller_unittest.cc",
     "manager/chromeos/touchscreen_util_unittest.cc",
     "manager/chromeos/update_display_configuration_task_unittest.cc",
     "manager/display_manager_utilities_unittest.cc",
     "manager/managed_display_info_unittest.cc",
+    "mojo/display_struct_traits_unittest.cc",
     "screen_unittest.cc",
     "util/display_util_unittest.cc",
     "util/edid_parser_unittest.cc",
     "win/scaling_util_unittest.cc",
     "win/screen_win_unittest.cc",
   ]
 
   deps = [
     ":display",
     ":display_manager_test_api",
     ":test_support",
     "//base",
     "//base/test:run_all_unittests",

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

Remove this since you added another target with a main().

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

Done, thanks!

     "//base/test:test_support",
+    "//mojo/edk/test:run_all_unittests",
     "//testing/gmock",
     "//testing/gtest",
     "//ui/display/manager",
+    "//ui/display/mojo:interfaces",

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

Remove this, it's a public dep of test_interfaces.

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

Done.

+    "//ui/display/mojo:test_interfaces",
     "//ui/display/types",
     "//ui/display/util",
     "//ui/events/devices",
     "//ui/gfx:test_support",
     "//ui/gfx/geometry",
   ]
 
   if (is_android) {
     # Do not run display_change_notifier_unittest.cc on Android because it
     # does not compile display_observer.cc
     sources -= [ "display_change_notifier_unittest.cc" ]
   }
 
   if (is_chromeos && use_x11) {
     sources += [
       "manager/chromeos/x11/display_util_x11_unittest.cc",
       "manager/chromeos/x11/native_display_event_dispatcher_x11_unittest.cc",
     ]
   }
 }