Bug 123948 - Block load event dispatching on old document when an HTMLMediaElement is moved between documents
Summary: Block load event dispatching on old document when an HTMLMediaElement is move...
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Eric Carlson
URL:
Keywords: BlinkMergeCandidate
Depends on:
Blocks:
 
Reported: 2013-11-06 20:25 PST by Ryosuke Niwa
Modified: 2013-11-07 09:44 PST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ryosuke Niwa 2013-11-06 20:25:03 PST
Consider merging https://chromium.googlesource.com/chromium/blink/+/b7d7d26cb8be0e9277fa7872f657273770463ad8

I looked into this but the test case appears to be busted so I'm not sure if it applies to WebKit or not.
Comment 1 Eric Carlson 2013-11-07 08:34:05 PST
(In reply to comment #0)
> Consider merging https://chromium.googlesource.com/chromium/blink/+/b7d7d26cb8be0e9277fa7872f657273770463ad8
> 
> I looked into this but the test case appears to be busted so I'm not sure if it applies to WebKit or not.

Their test case doesn't work because it uses a webm file. 

I wasn't able to reproduce the crash, but the problem is extremely timing sensitive so I think it is worth merging the small patch.
Comment 2 Eric Carlson 2013-11-07 09:44:08 PST
(In reply to comment #1)
> (In reply to comment #0)
> > Consider merging https://chromium.googlesource.com/chromium/blink/+/b7d7d26cb8be0e9277fa7872f657273770463ad8
> > 
> > I looked into this but the test case appears to be busted so I'm not sure if it applies to WebKit or not.
> 
> Their test case doesn't work because it uses a webm file. 
> 
> I wasn't able to reproduce the crash, but the problem is extremely timing sensitive so I think it is worth merging the small patch.

Actually this patch isn't necessary. The crash in Chromium is caused by a hack they added to HTMLMediaElement::didMoveToNewDocument for an issue that we don't have.