[Media Router] Custom Controls 1 - Add MediaStatus, MediaRouteController, and mojo interfaces

chrome/browser/media/router/media_status.cc

Index: chrome/browser/media/router/media_status.cc
diff --git a/chrome/browser/media/router/media_status.cc b/chrome/browser/media/router/media_status.cc
new file mode 100644
index 0000000000000000000000000000000000000000..32eb416e7f198cf995def72a17da60c171d85e8a
--- /dev/null
+++ b/chrome/browser/media/router/media_status.cc
@@ -0,0 +1,26 @@
+// 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.
+
+#include "chrome/browser/media/router/media_status.h"
+
+namespace media_router {
+
+MediaStatus::MediaStatus() {}

[mark a. foltz, 2017/03/06 20:06:19]

Style nit: using = default allows the compiler to generate a default move
constructor even if a custom constructor is defined later.

http://en.cppreference.com/w/cpp/language/default_constructor
http://en.cppreference.com/w/cpp/language/move_constructor

[imcheng, 2017/03/06 23:11:41]

Please also make sure member variables with primitive types are initialized.

[takumif, 2017/03/08 04:24:02]

Done.

[takumif, 2017/03/08 04:24:02]

Done.

+
+MediaStatus::MediaStatus(const MediaStatus& other) = default;
+
+MediaStatus::~MediaStatus() = default;
+
+MediaStatus& MediaStatus::operator=(const MediaStatus& other) = default;
+
+bool MediaStatus::operator==(const MediaStatus& other) const {
+  return title == other.title && status == other.status &&
+         can_play_pause == other.can_play_pause && can_mute == other.can_mute &&
+         can_set_volume == other.can_set_volume && can_seek == other.can_seek &&
+         is_paused == other.is_paused && is_muted == other.is_muted &&
+         volume == other.volume && duration == other.duration &&
+         current_time == other.current_time;
+}
+
+}  // namespace media_router