Bug 1287684 - Report HTMLMediaElement telemetry for non-MSE - r?rillian draft
authorGerald Squelart <gsquelart@mozilla.com>
Tue, 19 Jul 2016 14:33:55 +1000
changeset 389369 67594a9300dc8450487a2fb1d87186c65175bbf6
parent 389281 feaaf1af1065257b9178faca8b67eed9657b4a17
child 525734 6d3bd8800f62693ca04432fd69b482e9af23390b
push id23384
push usergsquelart@mozilla.com
push dateTue, 19 Jul 2016 06:21:36 +0000
reviewersrillian
bugs1287684
milestone50.0a1
Bug 1287684 - Report HTMLMediaElement telemetry for non-MSE - r?rillian Removed test blocking execution of ReportTelemetry in non-MSE case. EME currently depends on MSE, so ReportEMETelemetry won't do anything in the non-MSE case anyway. But I think it's good to remove the MSE test as well, in case we change that limitation in the future. MozReview-Commit-ID: 8sTg5F13KMs
dom/html/HTMLMediaElement.cpp
--- a/dom/html/HTMLMediaElement.cpp
+++ b/dom/html/HTMLMediaElement.cpp
@@ -4709,22 +4709,20 @@ void HTMLMediaElement::SuspendOrResumeEl
   LOG(LogLevel::Debug, ("%p SuspendOrResumeElement(pause=%d, suspendEvents=%d) hidden=%d",
       this, aPauseElement, aSuspendEvents, OwnerDoc()->Hidden()));
 
   if (aPauseElement != mPausedForInactiveDocumentOrChannel) {
     mPausedForInactiveDocumentOrChannel = aPauseElement;
     UpdateSrcMediaStreamPlaying();
     UpdateAudioChannelPlayingState();
     if (aPauseElement) {
-      if (mMediaSource) {
-        ReportTelemetry();
+      ReportTelemetry();
 #ifdef MOZ_EME
-        ReportEMETelemetry();
+      ReportEMETelemetry();
 #endif
-      }
 
 #ifdef MOZ_EME
       // For EME content, force destruction of the CDM client (and CDM
       // instance if this is the last client for that CDM instance) and
       // the CDM's decoder. This ensures the CDM gets reliable and prompt
       // shutdown notifications, as it may have book-keeping it needs
       // to do on shutdown.
       if (mMediaKeys) {