[Webkit-unassigned] [Bug 66230] Also release media resources when media is completely loaded.
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Aug 17 07:05:14 PDT 2011
https://bugs.webkit.org/show_bug.cgi?id=66230
Eric Carlson <eric.carlson at apple.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #104161|review?, commit-queue? |review-, commit-queue-
Flag| |
--- Comment #10 from Eric Carlson <eric.carlson at apple.com> 2011-08-17 07:05:14 PST ---
(From update of attachment 104161)
View in context: https://bugs.webkit.org/attachment.cgi?id=104161&action=review
> Source/WebCore/html/HTMLMediaElement.cpp:2654
> +void HTMLMediaElement::documentWillBecomeInactive()
> +{
> +#if !ENABLE(PLUGIN_PROXY_FOR_VIDEO)
> + LOG(Media, "HTMLMediaElement::documentWillBecomeInactive");
> + if (m_player)
> + m_player->willBecomeInactive();
> +#endif
> +}
> +
> +void HTMLMediaElement::documentDidBecomeActive()
> +{
> +#if !ENABLE(PLUGIN_PROXY_FOR_VIDEO)
> + LOG(Media, "HTMLMediaElement::documentDidBecomeActive");
> + if (m_player)
> + m_player->didBecomeActive();
> +#endif
> +}
> +
These new methods are unnecessary. documentWillBecomeInactive is called by Document::detach, *after* it calls stopActiveDOMObjects(). HTMLMediaElement is an ActiveDOMObject.
> Source/WebKit/chromium/src/WebMediaPlayerClientImpl.cpp:243
> +void WebMediaPlayerClientImpl::willBecomeInactive()
> +{
> + if (m_webMediaPlayer.get())
> + m_webMediaPlayer->willBecomeInactive();
> +}
> +
> +void WebMediaPlayerClientImpl::didBecomeActive()
> +{
> + if (m_webMediaPlayer.get())
> + m_webMediaPlayer->didBecomeActive();
> +}
> +
I assume that the Chromium media player sets network and ready states correctly when it unloads cached media data? Where are the tests to show that the behavior is correct.?
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list