Summary: | MediaStream API: Stop means stop | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Eric Carlson <eric.carlson> | ||||
Component: | WebCore Misc. | Assignee: | Eric Carlson <eric.carlson> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | commit-queue, eric.carlson, glenn, hta, jer.noble, tommyw | ||||
Priority: | P2 | Keywords: | BlinkMergeCandidate | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 121101 | ||||||
Attachments: |
|
Description
Eric Carlson
2013-09-06 11:48:25 PDT
Created attachment 211044 [details]
Proposed patch
Comment on attachment 211044 [details]
Proposed patch
This seems weird. MediaStreamTrack is an ActiveDOMObject, but doesn't override any of ActiveDOMObject's methods besides ::stop(), and doesn't call setPendingActivity(). Why isn't MediaStream an ActiveDOMObject? After all, it has a m_stopped ivar too. If anything, MediaStreamSource is the active object, but MediaStreamTrack::stop() doesn't try to stop it's associated source.
This seems highly weird, and I'm not entirely comfortable with it. But my problems with this method don't have much to do with this change specifically, so LGTM with reservations.
Comment on attachment 211044 [details] Proposed patch Clearing flags on attachment: 211044 Committed r155364: <http://trac.webkit.org/changeset/155364> All reviewed patches have been landed. Closing bug. |