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

chrome/browser/media/router/mojo/media_router_struct_traits.cc

Index: chrome/browser/media/router/mojo/media_router_struct_traits.cc
diff --git a/chrome/browser/media/router/mojo/media_router_struct_traits.cc b/chrome/browser/media/router/mojo/media_router_struct_traits.cc
index 47e9283b822136dc7566def4dd916f951d64cba7..d09ac50a7faf51330efeb0e2b2aae76049f1ebab 100644
--- a/chrome/browser/media/router/mojo/media_router_struct_traits.cc
+++ b/chrome/browser/media/router/mojo/media_router_struct_traits.cc
@@ -127,4 +127,32 @@ bool StructTraits<media_router::mojom::MediaRouteDataView,
   return true;
 }
 
+// static
+bool StructTraits<media_router::mojom::MediaStatusDataView,
+                  media_router::MediaStatus>::
+    Read(media_router::mojom::MediaStatusDataView data,
+         media_router::MediaStatus* out) {
+  std::string title;
+  if (!data.ReadTitle(&title))

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

Check that title and status are valid UTF-8.

[dcheng, 2017/03/08 01:08:20]

Just read it directly into out->title, rather than constructing a temporary)

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

Done.

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

Done.

+    return false;
+  out->title = title;
+
+  std::string status;
+  if (!data.ReadStatus(&status))
+    return false;
+  out->status = status;
+
+  out->can_play_pause = data.can_play_pause();
+  out->can_mute = data.can_mute();
+  out->can_set_volume = data.can_set_volume();
+  out->can_seek = data.can_seek();
+  out->is_paused = data.is_paused();
+  out->is_muted = data.is_muted();
+  out->volume = data.volume();
+  out->duration = data.duration();
+  out->current_time = data.current_time();
+
+  return true;
+}
+
 }  // namespace mojo