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

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

Issue 949293004: Add media router common classes and unittests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Resync with latest changes Created 5 years, 9 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_sink.h"
6
7 #include "base/logging.h"
8
9 namespace media_router {
10
11 MediaSink::MediaSink(const MediaSinkId& sink_id,
12 const std::string& name,
13 Status status)
14 : sink_id_(sink_id), name_(name), status_(status) {
15 }
16
17 MediaSink::~MediaSink() {
18 }
19
20 std::string MediaSink::StatusAsString() const {
21 switch (status_) {
22 case MediaSink::IDLE:
23 return "IDLE";
24 case MediaSink::ACTIVE:
25 return "ACTIVE";
26 case MediaSink::REQUEST_PENDING:
27 return "REQUEST_PENDING";
28 default:
29 NOTREACHED();
30 return "";
31 }
32 }
33
34 bool MediaSink::operator==(const MediaSink& other) const {
35 return sink_id_ == other.sink_id_ &&
mark a. foltz 2015/03/02 21:40:09 Is comparison by sink_id sufficient? Since this h
Kevin M 2015/03/03 21:53:19 Done.
36 name_ == other.name_ &&
37 status_ == other.status_;
38 }
39
40 bool MediaSink::operator!=(const MediaSink& other) const {
41 return !(*this == other);
mark a. foltz 2015/03/02 21:40:09 Or more simply sink_id_ != other.sink_id_
Kevin M 2015/03/03 21:53:19 Done.
42 }
43
44 } // namespace media_router
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698