Linux: Do not use a hard coded app name for PulseAudio output.

Linux: Do not use a hard coded app name for PulseAudio output.

BUG=526807
Committed: https://crrev.com/d2efdad3f588b423d284a141999aeac8d2e82dd1
Cr-Commit-Position: refs/heads/master@{#346965}

[Lei Zhang, 2015-09-02 00:40:43]

thestig@chromium.org changed reviewers:
+ dalecurtis@chromium.org

[DaleCurtis, 2015-09-02 02:38:21]

Thanks for fixing! I wonder if we can put this in audio_manager.h and
audio_manager.cc instead since it already has a lazy instance for global
settings.

https://codereview.chromium.org/1317033006/diff/40001/media/audio/audio_manag...
File media/audio/audio_manager_base.cc (right):

https://codereview.chromium.org/1317033006/diff/40001/media/audio/audio_manag...
media/audio/audio_manager_base.cc:30: base::LazyInstance<std::string>::Leaky
g_app_name;
Can this string just go in the existing audio_manager.cc lazy instance?

[Lei Zhang, 2015-09-02 06:16:39]

Done, see patch set 4.

[DaleCurtis, 2015-09-02 16:08:42]

lgtm, thanks!

[Lei Zhang, 2015-09-02 17:34:35]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2015-09-02 17:35:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1317033006/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1317033006/60001

[commit-bot: I haz the power, 2015-09-02 18:03:19]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-09-02 18:04:02]

Patchset 4 (id:??) landed as
https://crrev.com/d2efdad3f588b423d284a141999aeac8d2e82dd1
Cr-Commit-Position: refs/heads/master@{#346965}