Media Engagement: Improve audio only support

Desktop / Chromium - Becca Hughes [chromium.org] - 10 October 2017 11:06 EDT

Disables the video frame size signal if the media source does not have a video track. Also adds unit and browser tests for testing audio only sources.

BUG=772831

Change-Id: Iee87ec76226ea9d3e2a9ee60b58ef2580dee637c Reviewed-on: https://chromium-review.googlesource.com/707059

327bb21 Media Engagement: Improve audio only support
.../browser/media/media_engagement_browsertest.cc | 66 ++++++++++++++-
.../media/media_engagement_contents_observer.cc | 11 ++-
.../media/media_engagement_contents_observer.h | 1 +
.../media_engagement_contents_observer_unittest.cc | 94 +++++++++++++--------
.../engagement_silent_audio_track_no_video.webm | Bin 0 -> 6435 bytes
.../data/media/engagement/engagement_test.html | 2 +-
.../test/data/media/engagement/engagement_test.js | 2 +-
.../media/engagement/engagement_test_audio.html | 5 ++
.../engagement/engagement_test_audio_muted.html | 5 ++
.../engagement_test_iframe_audio_child.html | 11 +++
.../media/engagement/engagement_test_muted.html | 2 +-
.../engagement/engagement_test_no_audio_track.html | 2 +-
.../engagement_test_silent_audio_track.html | 2 +-
.../engagement_test_silent_audio_track_audio.html | 5 ++
.../engagement_test_small_frame_size.html | 2 +-
.../media/engagement/engagement_valid_audio.webm | Bin 0 -> 27518 bytes
16 files changed, 167 insertions(+), 43 deletions(-)

Upstream: git.chromium.org


  • Share