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

Unified Diff: chrome/browser/media/router/discovery/mdns/cast_media_sink_service.h

Issue 2927833002: [Media Router] Add CastMediaSinkService (Closed)
Patch Set: add unit tests Created 3 years, 6 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/browser/media/router/discovery/mdns/cast_media_sink_service.h
diff --git a/chrome/browser/media/router/discovery/mdns/cast_media_sink_service.h b/chrome/browser/media/router/discovery/mdns/cast_media_sink_service.h
new file mode 100644
index 0000000000000000000000000000000000000000..2e6747b58c0ff3546e834946a981bfe7cbc67bbc
--- /dev/null
+++ b/chrome/browser/media/router/discovery/mdns/cast_media_sink_service.h
@@ -0,0 +1,113 @@
+// 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.
+
+#ifndef CHROME_BROWSER_MEDIA_ROUTER_DISCOVERY_MDNS_CAST_MEDIA_SINK_SERVICE_H_
+#define CHROME_BROWSER_MEDIA_ROUTER_DISCOVERY_MDNS_CAST_MEDIA_SINK_SERVICE_H_
+
+#include <memory>
+#include <set>
+
+#include "base/gtest_prod_util.h"
+#include "base/memory/ref_counted.h"
+#include "base/threading/thread_checker.h"
+#include "chrome/browser/media/router/discovery/mdns/dns_sd_delegate.h"
+#include "chrome/browser/media/router/discovery/mdns/dns_sd_registry.h"
+#include "chrome/browser/media/router/discovery/media_sink_service_base.h"
+#include "components/cast_channel/cast_channel_enum.h"
+#include "net/base/ip_endpoint.h"
+
+namespace cast_channel {
+class CastSocketService;
+} // namespace cast_channel
+
+namespace content {
+class BrowserContext;
+} // namespace content
+
+namespace media_router {
+
+// A service which can be used to start background discovery and resolution of
+// Cast devices.
+// Public APIs should be invoked on the UI thread.
+class CastMediaSinkService
+ : public MediaSinkServiceBase,
+ public DnsSdRegistry::DnsSdObserver,
+ public base::RefCountedThreadSafe<CastMediaSinkService> {
+ public:
+ CastMediaSinkService(const OnSinksDiscoveredCallback& callback,
+ content::BrowserContext* browser_context);
+
+ // MediaSinkService implementation
+ void Start() override;
+ void Stop() override;
+
+ protected:
+ // Used to mock out the DnsSdRegistry for testing.
+ void SetDnsSdRegistryForTesting(DnsSdRegistry* registry);
imcheng 2017/06/15 00:51:00 super nit: s/Testing/Test, here and below.
zhaobin 2017/06/20 02:30:28 Done.
+
+ // Used to mock out the CastSocketService for testing.
+ void SetCastSocketServiceForTesting(
+ cast_channel::CastSocketService* cast_socket_service);
+
+ ~CastMediaSinkService() override;
+
+ private:
+ friend class base::RefCountedThreadSafe<CastMediaSinkService>;
+ friend class CastMediaSinkServiceTest;
+
+ FRIEND_TEST_ALL_PREFIXES(CastMediaSinkServiceTest, TestReStartAfterStop);
+ FRIEND_TEST_ALL_PREFIXES(CastMediaSinkServiceTest,
+ TestOnChannelOpenedOnIOThread);
+ FRIEND_TEST_ALL_PREFIXES(CastMediaSinkServiceTest,
+ TestMultipleOnChannelOpenedOnIOThread);
+ FRIEND_TEST_ALL_PREFIXES(CastMediaSinkServiceTest, TestOnDnsSdEvent);
+ FRIEND_TEST_ALL_PREFIXES(CastMediaSinkServiceTest, TestMultipleOnDnsSdEvent);
+ FRIEND_TEST_ALL_PREFIXES(CastMediaSinkServiceTest, TestTimer);
+
+ // DnsSdRegistry::DnsSdObserver implementation
+ void OnDnsSdEvent(const std::string& service_type,
+ const DnsSdRegistry::DnsSdServiceList& services) override;
+
+ // Open cast channel on IO thread.
+ // |service|: mDNS service description.
+ // |ip_endpoint|: cast channel's target IP endpoint.
+ void OpenChannelOnIOThread(const DnsSdService& service,
+ const net::IPEndPoint& ip_endpoint);
+
+ // Invoked when opening cast channel on IO thread completes.
+ // |service|: mDNS service description.
+ // |channel_id|: channel id of newly created cast channel.
+ // |channel_error|: error encounted when opending cast channel.
+ void OnChannelOpenedOnIOThread(const DnsSdService& service,
+ int channel_id,
+ cast_channel::ChannelError channel_error);
+
+ // Invoked by |OnChannelOpenedOnIOThread| to post task on UI thread.
+ // |service|: mDNS service description.
+ // |channel_id|: channel id of newly created cast channel.
+ // |audio_only|: if cast channel is audio only or not.
+ void OnChannelOpenedOnUIThread(const DnsSdService& service,
+ int channel_id,
+ bool audio_only);
+
+ // Raw pointer to DnsSdRegistry singleton.
+ DnsSdRegistry* dns_sd_registry_ = nullptr;
+
+ // DnsSdRegistry for test.
+ DnsSdRegistry* test_dns_sd_registry_ = nullptr;
+
+ // Service list from current round of discovery.
+ DnsSdRegistry::DnsSdServiceList current_services_;
+
+ // Service managing creating and removing cast channels.
+ scoped_refptr<cast_channel::CastSocketService> cast_socket_service_;
+
+ THREAD_CHECKER(thread_checker_);
+
+ DISALLOW_COPY_AND_ASSIGN(CastMediaSinkService);
+};
+
+} // namespace media_router
+
+#endif // CHROME_BROWSER_MEDIA_ROUTER_DISCOVERY_MDNS_CAST_MEDIA_SINK_SERVICE_H_

Powered by Google App Engine
This is Rietveld 408576698