Bug 135108

Summary: [iOS] a Paused media session is not active
Product: WebKit Reporter: Eric Carlson <eric.carlson>
Component: MediaAssignee: Eric Carlson <eric.carlson>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, darin, glenn, jeremyj-wk, jer.noble, philipj, sergio
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Proposed patch darin: review+

Description Eric Carlson 2014-07-20 19:43:14 PDT
MediaSessionManager::hasActive should not return true for a session in the Paused state, we should only activate the AudioSession for Playing sessions.
Comment 1 Eric Carlson 2014-07-20 19:43:48 PDT
<rdar://problem/17188368>
Comment 2 Eric Carlson 2014-07-20 19:57:56 PDT
Created attachment 235200 [details]
Proposed patch
Comment 3 Darin Adler 2014-07-20 20:40:07 PDT
With no regression test, we’re a little vulnerable here. Might be nice to have a brief comment explaining why this function does exactly what it does.
Comment 4 Eric Carlson 2014-07-21 08:05:29 PDT
(In reply to comment #3)
> With no regression test, we’re a little vulnerable here. Might be nice to have a brief comment explaining why this function does exactly what it does.

Good point. I renamed MediaSessionManager::hasActive to "activeAudioSessionRequired" to make obvious what the method does.
Comment 5 Eric Carlson 2014-07-21 08:06:19 PDT
Committed r171299 - https://trac.webkit.org/r171299