Write mojom and StructTraits for DisplaySnapshot.

ui/display/types/display_mode.h

Index: ui/display/types/display_mode.h
diff --git a/ui/display/types/display_mode.h b/ui/display/types/display_mode.h
index 0b972d5f497a8288007c0d154c74ca982964f31f..305dbd49e85b8f3f803e20094cb879ec271311b8 100644
--- a/ui/display/types/display_mode.h
+++ b/ui/display/types/display_mode.h
@@ -26,7 +26,7 @@ class DISPLAY_TYPES_EXPORT DisplayMode {
   const gfx::Size& size() const { return size_; }
   bool is_interlaced() const { return is_interlaced_; }
   float refresh_rate() const { return refresh_rate_; }
-
+  bool operator==(const DisplayMode& display_mode) const;

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

The parameter is typically named |other| here, in both the header and cc.

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

You should always define operator=! if you define operator==. It's easy
thankfully,

bool DisplayMode::operator!=(const DisplayMode& other) const {
  return !operator==(other);
}

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

Nice, thanks!

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

Done.

   virtual std::string ToString() const;
 
  private: