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

Unified Diff: chrome/common/media_router/discovery/media_sink_internal.cc

Issue 2965843002: [Media Router] Support dual discovery (Closed)
Patch Set: resolve code review comments from Derek Created 3 years, 5 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 side-by-side diff with in-line comments
Download patch
Index: chrome/common/media_router/discovery/media_sink_internal.cc
diff --git a/chrome/common/media_router/discovery/media_sink_internal.cc b/chrome/common/media_router/discovery/media_sink_internal.cc
index 0cab52e6c67ec6b50f39863e5d4e9981b8d69510..66071ddf284f6ed56b7e2f209be7e982c5f44904 100644
--- a/chrome/common/media_router/discovery/media_sink_internal.cc
+++ b/chrome/common/media_router/discovery/media_sink_internal.cc
@@ -175,9 +175,10 @@ CastSinkExtraData::CastSinkExtraData(const CastSinkExtraData& other) = default;
CastSinkExtraData::~CastSinkExtraData() = default;
bool CastSinkExtraData::operator==(const CastSinkExtraData& other) const {
- return ip_address == other.ip_address && model_name == other.model_name &&
- capabilities == other.capabilities &&
- cast_channel_id == other.cast_channel_id;
+ return ip_address == other.ip_address && port == other.port &&
+ model_name == other.model_name && capabilities == other.capabilities &&
+ cast_channel_id == other.cast_channel_id &&
+ discovered_by_dial == other.discovered_by_dial;
}
} // namespace media_router

Powered by Google App Engine
This is Rietveld 408576698