media-internals: Fix audio update logic.
commit002c154fd6b0b5a8d6a1b636444907e502cb3a94
authorxhwang <xhwang@chromium.org>
Tue, 16 Jun 2015 02:55:54 +0000 (15 19:55 -0700)
committerCommit bot <commit-bot@chromium.org>
Tue, 16 Jun 2015 02:56:28 +0000 (16 02:56 +0000)
tree6f25019f0f16af50f37639116b727b1a1ee2bf27
parent5e38e9d34bae15c93d64c275d314e9c28c30b668
media-internals: Fix audio update logic.

This fixes a regression introduced in r333285. Even if CanUpdate() is false, we
should still go through the full logic to update |audio_streams_cached_data_| so
that future UpdateAudioLog() calls will include the previous data.

Also rename SendAudioLogUpdate() to UpdateAudioLog() to better reflect it's
intent (which confused me when I worked on r333285).

BUG=260005
TEST=Manually tested as follows:
1. Start a youtube video in one tab.
2. Start a chrome://media-internals page in another tab.
3. Navigate to "Audio" tab in the chrome://media-internals page.
4. You should be able to see an audio controler and an audio stream.

Review URL: https://codereview.chromium.org/1184093006

Cr-Commit-Position: refs/heads/master@{#334541}
content/browser/media/media_internals.cc
content/browser/media/media_internals.h