Write mojom and StructTraits for DisplaySnapshot.

ui/display/types/display_mode.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_DISPLAY_TYPES_DISPLAY_MODE_H_
 #define UI_DISPLAY_TYPES_DISPLAY_MODE_H_
 
 #include <memory>
 #include <ostream>
 #include <string>
 
 #include "base/macros.h"
 #include "ui/display/types/display_types_export.h"
 #include "ui/gfx/geometry/size.h"
 
 namespace display {
 
 // This class represents the basic information for a native mode. Platforms will
 // extend this class to add platform specific information about the mode.
 class DISPLAY_TYPES_EXPORT DisplayMode {
  public:
   DisplayMode(const gfx::Size& size, bool interlaced, float refresh_rate);
   virtual ~DisplayMode();
   virtual std::unique_ptr<DisplayMode> Clone() const;
 
   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:
   gfx::Size size_;
   bool is_interlaced_;
   float refresh_rate_;
 
   DISALLOW_COPY_AND_ASSIGN(DisplayMode);
 };
 
 // Used to by gtest to print readable errors.
 DISPLAY_TYPES_EXPORT void PrintTo(const DisplayMode& mode, std::ostream* os);
 
 }  // namespace display
 
 #endif  // UI_DISPLAY_TYPES_DISPLAY_MODE_H_