widevine-partner/android
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Disable widevine metrics"

Browse Source
This commit is contained in:
Jeff Tinker
2017-02-08 02:00:25 +00:00
committed by Android (Google) Code Review
parent 1be8354553 3c22ce40cd
commit c934152265
2 changed files with 11 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
libwvdrmengine/cdm/metrics/include/metrics_front_end.h
View File

@@ -40,7 +40,14 @@ class MetricsFrontEnd {
#define NO_TIME 0 #define NO_TIME 0
#define M_RECORD(GROUP, METRIC, TIME, ...) \ #if 0 // Disable metrics for now to work around P0:b/35093325
#define M_RECORD(GROUP, METRIC, TIME, ...)
#define M_TIME(CALL, GROUP, METRIC, ...)
#else // Re-enable when we have time to debug
#define M_RECORD(GROUP, METRIC, TIME, ...) \
if ( GROUP ) { \ if ( GROUP ) { \
( GROUP ) -> METRIC . Record( TIME, ##__VA_ARGS__ ); \ ( GROUP ) -> METRIC . Record( TIME, ##__VA_ARGS__ ); \
} }
@@ -54,5 +61,6 @@ class MetricsFrontEnd {
} else { \ } else { \
CALL; \ CALL; \
} }
#endif // Disable
#endif #endif

2
libwvdrmengine/cdm/metrics/src/metrics_group.cpp
View File

@@ -393,11 +393,13 @@ MetricsGroup::MetricsGroup() :
"min_version") { } "min_version") { }
MetricsGroup::~MetricsGroup() { MetricsGroup::~MetricsGroup() {
#if 0 // Disable metrics for now to work around P0:b/35093325
MetricNotification* subscriber = MFE.CreateSubscriber(); MetricNotification* subscriber = MFE.CreateSubscriber();
if (subscriber) { if (subscriber) {
Publish(subscriber); Publish(subscriber);
} }
delete subscriber; delete subscriber;
#endif
} }
void MetricsGroup::Publish(MetricNotification* subscriber) { void MetricsGroup::Publish(MetricNotification* subscriber) {