[Media Router] Support dual discovery

chrome/browser/media/router/discovery/mdns/cast_media_sink_service.cc

 // Copyright 2017 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/discovery/mdns/cast_media_sink_service.h"
 
 #include "base/memory/ptr_util.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/common/media_router/discovery/media_sink_internal.h"
 #include "components/cast_channel/cast_socket_service.h"
@@ skipping 10 matching lines @@
 enum ErrorType {
   NONE,
   NOT_CAST_DEVICE,
   MISSING_ID,
   MISSING_FRIENDLY_NAME,
   MISSING_OR_INVALID_IP_ADDRESS,
   MISSING_OR_INVALID_PORT,
 };
 
 ErrorType CreateCastMediaSink(const media_router::DnsSdService& service,
-                              int channel_id,
-                              bool audio_only,
                               media_router::MediaSinkInternal* cast_sink) {
   DCHECK(cast_sink);
   if (service.service_name.find(
           media_router::CastMediaSinkService::kCastServiceType) ==
       std::string::npos)
     return ErrorType::NOT_CAST_DEVICE;
 
   net::IPAddress ip_address;
   if (!ip_address.AssignFromIPLiteral(service.ip_address))
     return ErrorType::MISSING_OR_INVALID_IP_ADDRESS;
 
+  net::HostPortPair host_port_pair =
+      net::HostPortPair::FromString(service.service_host_port);
+  int port = host_port_pair.port();
+
   std::map<std::string, std::string> service_data;
   for (const auto& item : service.service_data) {
     // |item| format should be "id=xxxxxx", etc.
     size_t split_idx = item.find('=');
     if (split_idx == std::string::npos)
       continue;
 
     std::string key = item.substr(0, split_idx);
     std::string val =
         split_idx < item.length() ? item.substr(split_idx + 1) : "";
     service_data[key] = val;
   }
 
   // When use this "sink" within browser, please note it will have a different
   // ID when it is sent to the extension, because it derives a different sink ID
   // using the given sink ID.
   std::string unique_id = service_data["id"];
   if (unique_id.empty())
     return ErrorType::MISSING_ID;
   std::string friendly_name = service_data["fn"];
   if (friendly_name.empty())
     return ErrorType::MISSING_FRIENDLY_NAME;
   media_router::MediaSink sink(unique_id, friendly_name,
                                media_router::MediaSink::IconType::CAST);
 
   media_router::CastSinkExtraData extra_data;
   extra_data.ip_address = ip_address;
   extra_data.model_name = service_data["md"];
-  extra_data.capabilities = cast_channel::CastDeviceCapability::AUDIO_OUT;
-  if (!audio_only)
-    extra_data.capabilities |= cast_channel::CastDeviceCapability::VIDEO_OUT;
-  extra_data.cast_channel_id = channel_id;
+  extra_data.port = port;
+  extra_data.discovered_by_dial = false;
 
   cast_sink->set_sink(sink);
   cast_sink->set_cast_data(extra_data);
 
   return ErrorType::NONE;
 }
 
+static media_router::MediaSinkInternal CreateCastSinkFromDialSink(
+    const media_router::MediaSinkInternal& dial_sink) {
+  std::string unique_id = dial_sink.sink().id();
+  std::string friendly_name = dial_sink.sink().name();
+  media_router::MediaSink sink(unique_id, friendly_name,
+                               media_router::MediaSink::IconType::CAST);
+
+  media_router::CastSinkExtraData extra_data;
+  extra_data.ip_address = dial_sink.dial_data().ip_address;
+  extra_data.port = media_router::CastMediaSinkService::kCastControlPort;
+  extra_data.model_name = dial_sink.dial_data().model_name;
+  extra_data.discovered_by_dial = true;
+
+  return media_router::MediaSinkInternal(sink, extra_data);
+}
+
 }  // namespace
 
 namespace media_router {
 
 // static
 const char CastMediaSinkService::kCastServiceType[] = "_googlecast._tcp.local";
 
+// static
+int CastMediaSinkService::kCastControlPort = 8009;
+
 CastMediaSinkService::CastMediaSinkService(
     const OnSinksDiscoveredCallback& callback,
     content::BrowserContext* browser_context)
     : MediaSinkServiceBase(callback) {
   DCHECK_CALLED_ON_VALID_THREAD(thread_checker_);
   cast_socket_service_ = cast_channel::CastSocketServiceFactory::GetInstance()
                              ->GetForBrowserContext(browser_context);
   DCHECK(cast_socket_service_);
 }
 
@@ skipping 31 matching lines @@
 }
 
 void CastMediaSinkService::SetDnsSdRegistryForTest(DnsSdRegistry* registry) {
   DCHECK(!dns_sd_registry_);
   dns_sd_registry_ = registry;
   dns_sd_registry_->AddObserver(this);
   dns_sd_registry_->RegisterDnsSdListener(kCastServiceType);
   MediaSinkServiceBase::StartTimer();
 }
 
+void CastMediaSinkService::OnFetchCompleted() {
+  DCHECK_CALLED_ON_VALID_THREAD(thread_checker_);
+
+  MediaSinkServiceBase::RemoveSinks();
+
+  // Copy cast sink from mDNS service to |current_sinks_|.
+  for (const auto& sink_it : current_sinks_by_mdns_map_)
+    MediaSinkServiceBase::AddSink(sink_it.second);
+
+  // Copy cast sink from DIAL discovery to |current_sinks_|.
+  for (const auto& sink_it : current_sinks_by_dial_map_) {
+    if (!base::ContainsKey(current_sinks_by_mdns_map_, sink_it.first))
+      MediaSinkServiceBase::AddSink(sink_it.second);
+  }
+  MediaSinkServiceBase::OnFetchCompleted();
+}
+
 void CastMediaSinkService::OnDnsSdEvent(
     const std::string& service_type,
     const DnsSdRegistry::DnsSdServiceList& services) {
   DCHECK_CALLED_ON_VALID_THREAD(thread_checker_);
   DVLOG(2) << "CastMediaSinkService::OnDnsSdEvent found " << services.size()
            << " services";
 
-  current_sinks_.clear();
-  current_services_ = services;
+  current_sinks_by_mdns_map_.clear();
+  current_service_ip_endpoints_.clear();
 
   for (const auto& service : services) {
-    net::IPAddress ip_address;
-    if (!ip_address.AssignFromIPLiteral(service.ip_address)) {
-      DVLOG(2) << "Invalid ip_addresss: " << service.ip_address;
+    // Create Cast sink from mDNS service description.
+    MediaSinkInternal cast_sink;
+    ErrorType error = CreateCastMediaSink(service, &cast_sink);
+    if (error != ErrorType::NONE) {
+      DVLOG(2) << "Fail to create Cast device [error]: " << error;
       continue;
     }
-    net::HostPortPair host_port_pair =
-        net::HostPortPair::FromString(service.service_host_port);
 
+    net::IPEndPoint ip_endpoint(cast_sink.cast_data().ip_address,
+                                cast_sink.cast_data().port);
+    current_service_ip_endpoints_.insert(ip_endpoint);
+
+    // Open Cast socket on the IO thread.
     content::BrowserThread::PostTask(
         content::BrowserThread::IO, FROM_HERE,
-        base::Bind(&CastMediaSinkService::OpenChannelOnIOThread, this, service,
-                   net::IPEndPoint(ip_address, host_port_pair.port())));
+        base::Bind(&CastMediaSinkService::OpenChannelOnIOThread, this,
+                   ip_endpoint,
+                   base::Bind(&CastMediaSinkService::OnChannelOpenedOnIOThread,
+                              this, cast_sink)));
   }
 
   MediaSinkServiceBase::RestartTimer();
 }
 
+void CastMediaSinkService::OnMediaSinkAdded(const MediaSinkInternal& sink) {
+  auto ip_address = sink.dial_data().ip_address;
+  net::IPEndPoint ip_endpoint(ip_address, kCastControlPort);
+
+  if (base::ContainsKey(current_service_ip_endpoints_, ip_endpoint)) {
+    DVLOG(2) << "Sink discovered by mDNS, skip adding [name]: "
+             << sink.sink().name();
+    return;
+  }
+
+  auto cast_sink = CreateCastSinkFromDialSink(sink);
+  OpenChannelOnIOThread(
+      ip_endpoint, base::Bind(&CastMediaSinkService::OnChannelOpenedOnIOThread,
+                              this, cast_sink));

[imcheng, 2017/07/06 22:53:30]

We can enable moves for MediaSinkInternal to make passing it to tasks a bit more
efficient. WDYT?

[zhaobin, 2017/07/10 20:21:15]

Done.

+}
+
+void CastMediaSinkService::OnMediaSinksRemoved() {
+  current_sinks_by_dial_map_.clear();
+}
+
 void CastMediaSinkService::OpenChannelOnIOThread(
-    const DnsSdService& service,
-    const net::IPEndPoint& ip_endpoint) {
+    const net::IPEndPoint& ip_endpoint,
+    const cast_channel::CastSocket::OnOpenCallback& on_open_callback) {

[imcheng, 2017/07/06 22:53:30]

Does this need to take a callback as input, or can it take a cast_sink and we
create the callback here?

[zhaobin, 2017/07/10 20:21:15]

Done.

   auto* observer = cast_socket_service_->GetObserver(kObserverId);
   if (!observer) {
     observer = new CastSocketObserver();
     cast_socket_service_->AddObserver(kObserverId, base::WrapUnique(observer));
   }
 
-  cast_socket_service_->OpenSocket(
-      ip_endpoint, g_browser_process->net_log(),
-      base::Bind(&CastMediaSinkService::OnChannelOpenedOnIOThread, this,
-                 service),
-      observer);
+  cast_socket_service_->OpenSocket(ip_endpoint, g_browser_process->net_log(),
+                                   on_open_callback, observer);
 }
 
 void CastMediaSinkService::OnChannelOpenedOnIOThread(
-    const DnsSdService& service,
+    const MediaSinkInternal& cast_sink,
     int channel_id,
     cast_channel::ChannelError channel_error) {
   if (channel_error != cast_channel::ChannelError::NONE) {
-    DVLOG(2) << "Fail to open channel " << service.ip_address << ": "
-             << service.service_host_port
+    DVLOG(2) << "Fail to open channel "
+             << cast_sink.cast_data().ip_address.ToString()
              << " [ChannelError]: " << (int)channel_error;
     return;
   }
 
   auto* socket = cast_socket_service_->GetSocket(channel_id);
   if (!socket) {
     DVLOG(2) << "Fail to find socket with [channel_id]: " << channel_id;
     return;
   }
 
+  media_router::CastSinkExtraData extra_data = cast_sink.cast_data();
+  extra_data.capabilities = cast_channel::CastDeviceCapability::AUDIO_OUT;
+  if (!socket->audio_only())
+    extra_data.capabilities |= cast_channel::CastDeviceCapability::VIDEO_OUT;
+  extra_data.cast_channel_id = channel_id;
+
   content::BrowserThread::PostTask(
       content::BrowserThread::UI, FROM_HERE,
       base::Bind(&CastMediaSinkService::OnChannelOpenedOnUIThread, this,
-                 service, channel_id, socket->audio_only()));
+                 MediaSinkInternal(cast_sink.sink(), extra_data)));
 }
 
 void CastMediaSinkService::OnChannelOpenedOnUIThread(
-    const DnsSdService& service,
-    int channel_id,
-    bool audio_only) {
+    const MediaSinkInternal& cast_sink) {
   DCHECK_CALLED_ON_VALID_THREAD(thread_checker_);
-  MediaSinkInternal sink;
-  ErrorType error = CreateCastMediaSink(service, channel_id, audio_only, &sink);
-  if (error != ErrorType::NONE) {
-    DVLOG(2) << "Fail to create Cast device [error]: " << error;
-    return;
+  auto ip_endpoint = net::IPEndPoint(cast_sink.cast_data().ip_address,
+                                     cast_sink.cast_data().port);
+
+  // Skip adding if IP address not found in current round of mDNS discovery.
+  if (!cast_sink.cast_data().discovered_by_dial) {
+    if (!base::ContainsValue(current_service_ip_endpoints_, ip_endpoint)) {
+      DVLOG(2) << "Service data not found in current service data list...";
+      return;
+    }
   }
 
-  if (!base::ContainsValue(current_services_, service)) {
-    DVLOG(2) << "Service data not found in current service data list...";
-    return;
+  DVLOG(2) << "Ading sink to current_sinks_ [name]: "
+           << cast_sink.sink().name();
+  // Add or update existing cast sink.
+  if (cast_sink.cast_data().discovered_by_dial) {
+    current_sinks_by_dial_map_[ip_endpoint] = cast_sink;
+  } else {
+    current_sinks_by_mdns_map_[ip_endpoint] = cast_sink;
   }
-
-  DVLOG(2) << "Ading sink to current_sinks_ [id]: " << sink.sink().id();
-  current_sinks_.insert(sink);
   MediaSinkServiceBase::RestartTimer();
 }
 
 CastMediaSinkService::CastSocketObserver::CastSocketObserver() {}
 CastMediaSinkService::CastSocketObserver::~CastSocketObserver() {}
 
 void CastMediaSinkService::CastSocketObserver::OnError(
     const cast_channel::CastSocket& socket,
     cast_channel::ChannelError error_state) {
   DVLOG(1) << "OnError [ip_endpoint]: " << socket.ip_endpoint().ToString()
            << " [error_state]: "
            << cast_channel::ChannelErrorToString(error_state);
 }
 
 void CastMediaSinkService::CastSocketObserver::OnMessage(
     const cast_channel::CastSocket& socket,
     const cast_channel::CastMessage& message) {}
 
 }  // namespace media_router