Closed Bug 483563 Opened 15 years ago Closed 15 years ago

MediaLoadListener shouldn't try to load canceled channels

Categories

(Core :: Audio/Video, defect)

Product:
Core
Component:
Audio/Video
Platform:
x86
All
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED DUPLICATE of bug 483324

People

(Reporter: cpearce, Unassigned)

Details

In nsHTMLMediaElement::MediaLoadListener::OnStartRequest(), we always try to initialized a channel. However when an access controls check has failed (and maybe at other times), nsCrossSiteListenerProxy (or something else) will Cancel() the channel, but we still load it. MediaLoadListener::OnStartRequest() should not try to load the channel if it's already been canceled. Eventually the call path in MediaLoadListener::OnStartRequest() calls down to nsChannelToPipeListener::OnStartRequest() which checks the request status and aborts the load. We could avoid a bit of work if we just aborted the load in MediaLoadListener::OnStartRequest().
Summary: MediaLoadListener shouldn't try to load cancelled channels → MediaLoadListener shouldn't try to load canceled channels
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.