Implement an audio focus manager on non-Mobile Chrome |
||||
Issue descriptionThe media session delegate for desktop is very basic and doesn't really handle audio focus. Ideally, it should talk to an audio focus manager as the Android media session delegate does except that Chrome would implement it instead of the system. Requirements for the audio focus manager would be: - lazy initialised singleton (one per browser process); - it handles a stack of audio focus requests so it can give back audio focus to WebContents that lost it; - it keep tracks of WebContents' death; - it handles at least two types of audio focus gain (content and transient).
,
Jun 10 2016
,
Jul 7 2016
,
Jul 21 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ce42802dbc1e900b9d0689508b2f6284ee617a55 commit ce42802dbc1e900b9d0689508b2f6284ee617a55 Author: mlamouri <mlamouri@chromium.org> Date: Thu Jul 21 14:21:55 2016 Implement Audio Focus Manager for desktop. This CL implements a real Audio Focus Manager internal to Chrome. The intent is to use it on Chrome desktop, when there is no system audio focus manager. The Audio Focus Manager handles ducking and audio focus gain. At the moment, the rules are that only one entry can have audio focus and any transient will duck the audio focus entry. BUG= 609503 Review-Url: https://codereview.chromium.org/1971443002 Cr-Commit-Position: refs/heads/master@{#406843} [add] https://crrev.com/ce42802dbc1e900b9d0689508b2f6284ee617a55/content/browser/media/session/audio_focus_manager.cc [add] https://crrev.com/ce42802dbc1e900b9d0689508b2f6284ee617a55/content/browser/media/session/audio_focus_manager.h [add] https://crrev.com/ce42802dbc1e900b9d0689508b2f6284ee617a55/content/browser/media/session/audio_focus_manager_unittest.cc [modify] https://crrev.com/ce42802dbc1e900b9d0689508b2f6284ee617a55/content/browser/media/session/media_session.h [modify] https://crrev.com/ce42802dbc1e900b9d0689508b2f6284ee617a55/content/browser/media/session/media_session_delegate_default.cc [modify] https://crrev.com/ce42802dbc1e900b9d0689508b2f6284ee617a55/content/content_browser.gypi [modify] https://crrev.com/ce42802dbc1e900b9d0689508b2f6284ee617a55/content/content_tests.gypi
,
Jul 21 2016
|
||||
►
Sign in to add a comment |
||||
Comment 1 by mlamouri@chromium.org
, May 23 2016