Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(358)

Side by Side Diff: chrome/browser/media/router/media_route_response.cc

Issue 1020743003: [Media Router] Design MediaRouter interface with stub implementation. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed Kevin's comments Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "chrome/browser/media/router/media_route_response.h"
6
7 namespace media_router {
8
9 // static
10 scoped_ptr<MediaRouteResponse> MediaRouteResponse::CreateFromSuccess(
11 const RouteRequestId& request_id, const MediaRoute& route) {
12 return make_scoped_ptr(new MediaRouteResponse(request_id, route));
13 }
14
15 // static
16 scoped_ptr<MediaRouteResponse> MediaRouteResponse::CreateFromError(
17 const RouteRequestId& request_id, const std::string& error) {
18 return make_scoped_ptr(new MediaRouteResponse(request_id, error));
19 }
20
21 MediaRouteResponse::~MediaRouteResponse() {
22 }
23
24 MediaRouteResponse::MediaRouteResponse(
25 const RouteRequestId& request_id, const MediaRoute& route)
xhwang 2015/04/02 17:13:02 What if |route| is Empty()? Is that response valid
imcheng 2015/04/02 23:05:24 Changed back to nullptr
26 : request_id_(request_id), route_(route) {
27 }
28
29 MediaRouteResponse::MediaRouteResponse(
30 const RouteRequestId& request_id, const std::string& error)
31 : request_id_(request_id), error_(error) {
32 }
33
34 } // namespace media_router
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698