As discussed with Eric Carlson and Jer Noble, we should rename HTMLMediaSession to HTMLMediaElementSession to reserve "media session" for the Media Session API (bug 145411).
<rdar://problem/21142951>
Created attachment 253854 [details] Patch
Attachment 253854 [details] did not pass style-queue: ERROR: Source/WebCore/html/HTMLMediaElementSession.cpp:30: You should not add a blank line before implementation file's own header. [build/include_order] [4] ERROR: Source/WebCore/html/HTMLMediaElementSession.cpp:58: Multi line control clauses should use braces. [whitespace/braces] [4] Total errors found: 2 in 10 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 253854 [details] Patch Generally we only want to use the HTML prefix for classes actually exposed to the DOM. So I think MediaElementSession would be better.
I'll send out a new patch. I already ran this change by Eric as well.
Created attachment 253866 [details] Patch
Attachment 253866 [details] did not pass style-queue: ERROR: Source/WebCore/html/MediaElementSession.cpp:30: You should not add a blank line before implementation file's own header. [build/include_order] [4] ERROR: Source/WebCore/html/MediaElementSession.cpp:58: Multi line control clauses should use braces. [whitespace/braces] [4] Total errors found: 2 in 10 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 253866 [details] Patch Clearing flags on attachment: 253866 Committed r184966: <http://trac.webkit.org/changeset/184966>
All reviewed patches have been landed. Closing bug.