[Webkit-unassigned] [Bug 22105] New: Removing a media element from the document may trigger last second load()

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=22105

           Summary: Removing a media element from the document may trigger
                    last second load()
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: P3
         Component: WebKit Misc.
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: tavestbo at trolltech.com


Removing a media element from the document may trigger last-second load()

The current spec says:

"When a media element is removed from a Document, if the media element's
networkState attribute has a value other than NETWORK_EMPTY then the user agent
must act as if the pause() method had been invoked."

And for pause():

"If the media element's networkState attribute has the value NETWORK_EMPTY,
then the user agent must invoke the load()  method and wait for it to return."

So in effect, if we add a guard for NETWORK_EMPTY in
HTMLMediaElement::removedFromDocument() we can get rid of one of the conditions
where we load() during removedFromDocument().  

The other condition is when there's no m_player. What I'm wondering is if we
really need to load() if m_player is nil in pause() (and play for that matter)
and/or if the networkState() == EMPTY check enough? Ie., will there be cases
where networkState() > EMPTY but m_player is 0?


-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.