AudioSessionManager tracks Audio, Video, and WebAudio instances so it should be a MediaSessionManager instead. Some platforms may need to manage other aspects of media playback, eg. limiting the number instances that can play simultaneously, when a media is allowed to play, etc, it should be refactored so it is possible for the session manager to inform clients of state changes.
Created attachment 219825 [details] Proposed patch
Attachment 219825 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCore/WebCore.xcodeproj/project.pbxproj', u'Source/WebCore/html/HTMLMediaElement.cpp', u'Source/WebCore/html/HTMLMediaElement.h', u'Source/WebCore/platform/audio/AudioSessionListener.h', u'Source/WebCore/platform/audio/AudioSessionManager.cpp', u'Source/WebCore/platform/audio/AudioSessionManager.h', u'Source/WebCore/platform/audio/MediaSessionManager.cpp', u'Source/WebCore/platform/audio/MediaSessionManager.h', u'Source/WebCore/platform/audio/mac/AudioDestinationMac.cpp', u'Source/WebCore/platform/audio/mac/AudioDestinationMac.h', u'Source/WebCore/platform/audio/mac/AudioSessionManagerMac.cpp', u'Source/WebCore/platform/audio/mac/MediaSessionManagerMac.cpp', '--commit-queue']" exit_code: 1 ERROR: Source/WebCore/platform/audio/mac/AudioDestinationMac.cpp:37: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/platform/audio/MediaSessionManager.cpp:64: This { should be at the end of the previous line [whitespace/braces] [4] ERROR: Source/WebCore/platform/audio/MediaSessionManager.cpp:78: This { should be at the end of the previous line [whitespace/braces] [4] Total errors found: 3 in 10 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 219827 [details] Updated patch
Comment on attachment 219827 [details] Updated patch r=me
Comment on attachment 219827 [details] Updated patch Clearing flags on attachment: 219827 Committed r161000: <http://trac.webkit.org/changeset/161000>
All reviewed patches have been landed. Closing bug.