WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
138153
Remove MediaSessionClient::hasMediaCharacteristics() and MediaSessionClient::mediaStateDidChange()
https://bugs.webkit.org/show_bug.cgi?id=138153
Summary
Remove MediaSessionClient::hasMediaCharacteristics() and MediaSessionClient::...
Ada Chan
Reported
2014-10-28 15:15:14 PDT
After
http://trac.webkit.org/changeset/175273
, Document no longer keeps a set of MediaSessions and it doesn't need to query each MediaSession for its characteristics and play state. MediaSessionClient::hasMediaCharacteristics() is no longer needed and MediaSessionClient::mediaStateDidChange() can be removed if HTMLMediaElement calls Document::updateIsPlayingAudio() after m_playing has changed.
Attachments
Patch
(10.25 KB, patch)
2014-10-28 16:34 PDT
,
Ada Chan
jer.noble
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Ada Chan
Comment 1
2014-10-28 16:34:10 PDT
Created
attachment 240576
[details]
Patch
Jer Noble
Comment 2
2014-10-29 10:17:33 PDT
Comment on
attachment 240576
[details]
Patch r=me, with one nit: The comment you made above is really good, and I'd like to see it in the ChangeLog too.
Ada Chan
Comment 3
2014-10-29 10:42:17 PDT
Thanks Jer. Committed:
http://trac.webkit.org/changeset/175330
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug