[Media Router] Design MediaRouter interface with stub implementation.

chrome/browser/media/router/media_route_response.cc

Index: chrome/browser/media/router/media_route_response.cc
diff --git a/chrome/browser/media/router/media_route_response.cc b/chrome/browser/media/router/media_route_response.cc
new file mode 100644
index 0000000000000000000000000000000000000000..3cac968f03f9a68f9a0de7baf85b9e42c9106a38
--- /dev/null
+++ b/chrome/browser/media/router/media_route_response.cc
@@ -0,0 +1,34 @@
+// Copyright 2015 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_route_response.h"
+
+namespace media_router {
+
+// static
+scoped_ptr<MediaRouteResponse> MediaRouteResponse::CreateFromSuccess(
+    const RouteRequestId& request_id, const MediaRoute& route) {
+  return make_scoped_ptr(new MediaRouteResponse(request_id, route));
+}
+
+// static
+scoped_ptr<MediaRouteResponse> MediaRouteResponse::CreateFromError(
+    const RouteRequestId& request_id, const std::string& error) {
+  return make_scoped_ptr(new MediaRouteResponse(request_id, error));
+}
+
+MediaRouteResponse::~MediaRouteResponse() {
+}
+
+MediaRouteResponse::MediaRouteResponse(
+    const RouteRequestId& request_id, const MediaRoute& route)

[xhwang, 2015/04/02 17:13:02]

What if |route| is Empty()? Is that response valid or not? IMHO having a null
route is more clear (see comment above)...

[imcheng, 2015/04/02 23:05:24]

Changed back to nullptr

+    : request_id_(request_id), route_(route) {
+}
+
+MediaRouteResponse::MediaRouteResponse(
+    const RouteRequestId& request_id, const std::string& error)
+    : request_id_(request_id), error_(error) {
+}
+
+}  // namespace media_router