Hide media sinks that don't have a name in the cast system tray.

chrome/browser/ui/ash/cast_config_delegate_media_router.cc

 // 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/ui/ash/cast_config_delegate_media_router.h"
 
 #include <string>
 #include <vector>
 
 #include "base/macros.h"
@@ skipping 77 matching lines @@
                          GURL(chrome::kChromeUIMediaRouterURL)),
       cast_config_delegate_(cast_config_delegate) {}
 
 CastDeviceCache::~CastDeviceCache() {}
 
 void CastDeviceCache::Init() {
   CHECK(MediaSinksObserver::Init());
 }
 
 void CastDeviceCache::OnSinksReceived(const MediaSinks& sinks) {
-  sinks_ = sinks;
+  // The media router adds a MediaSink instance that doesn't have a name. Make
+  // sure to filter that sink out from the UI so it is not rendered, as it will
+  // be a line that only has a icon with no apparent meaning.
+  sinks_.clear();
+  for (const media_router::MediaSink& sink : sinks) {
+    if (sink.name().empty())

[amp, 2016/06/02 16:49:24]

Until we can duplicate the logic in
https://cs.chromium.org/chromium/src/chrome/browser/ui/webui/media_router/med...
we should also filter out all sinks with a domain.

if (!sink.domain().empty())
  continue;

This should also cover the case where sinks are coming back with no name, as
those are specifically tied to having a domain.

[jdufault, 2016/06/02 17:35:14]

Done.

+      continue;
+
+    sinks_.push_back(sink);
+  }
+
   cast_config_delegate_->RequestDeviceRefresh();
 }
 
 void CastDeviceCache::OnRoutesUpdated(
     const MediaRoutes& routes,
     const MediaRouteIds& unused_joinable_route_ids) {
   routes_ = routes;
   cast_config_delegate_->RequestDeviceRefresh();
 }
 
@@ skipping 108 matching lines @@
 
 void CastConfigDelegateMediaRouter::AddObserver(
     ash::CastConfigDelegate::Observer* observer) {
   observer_list_.AddObserver(observer);
 }
 
 void CastConfigDelegateMediaRouter::RemoveObserver(
     ash::CastConfigDelegate::Observer* observer) {
   observer_list_.RemoveObserver(observer);
 }