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

Unified Diff: chrome/browser/media/router/discovery/dial/dial_media_sink_service_impl.h

Issue 2916163002: [MediaRouter] Record DIAL device counts in DialMediaSinkServiceImpl (Closed)
Patch Set: resolve code review comments from isherman 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
« no previous file with comments | « no previous file | chrome/browser/media/router/discovery/dial/dial_media_sink_service_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/media/router/discovery/dial/dial_media_sink_service_impl.h
diff --git a/chrome/browser/media/router/discovery/dial/dial_media_sink_service_impl.h b/chrome/browser/media/router/discovery/dial/dial_media_sink_service_impl.h
index d84a64c9805dc8d509d1b8a62a89247da945736a..e238bc4ed7de4679807f3c1ba538b5642fea9592 100644
--- a/chrome/browser/media/router/discovery/dial/dial_media_sink_service_impl.h
+++ b/chrome/browser/media/router/discovery/dial/dial_media_sink_service_impl.h
@@ -11,6 +11,7 @@
#include "chrome/browser/media/router/discovery/dial/device_description_service.h"
#include "chrome/browser/media/router/discovery/dial/dial_registry.h"
#include "chrome/browser/media/router/discovery/media_sink_service_base.h"
+#include "chrome/browser/media/router/media_router_metrics.h"
namespace media_router {
@@ -63,6 +64,9 @@ class DialMediaSinkServiceImpl : public MediaSinkServiceBase,
void OnDeviceDescriptionError(const DialDeviceData& device,
const std::string& error_message);
+ // MediaSinkServiceBase implementation.
+ void RecordDeviceCounts() override;
+
std::unique_ptr<DeviceDescriptionService> description_service_;
// Raw pointer to DialRegistry singleton.
@@ -75,6 +79,8 @@ class DialMediaSinkServiceImpl : public MediaSinkServiceBase,
DialRegistry::DeviceList current_devices_;
scoped_refptr<net::URLRequestContextGetter> request_context_;
+
+ MediaRouterMetrics metrics_;
};
} // namespace media_router
« no previous file with comments | « no previous file | chrome/browser/media/router/discovery/dial/dial_media_sink_service_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698