Bug 123948
| Summary: | Block load event dispatching on old document when an HTMLMediaElement is moved between documents | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> |
| Component: | Media | Assignee: | Eric Carlson <eric.carlson> |
| Status: | RESOLVED WONTFIX | ||
| Severity: | Normal | CC: | eric.carlson, jer.noble |
| Priority: | P2 | Keywords: | BlinkMergeCandidate |
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Ryosuke Niwa
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.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Eric Carlson
(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.
Eric Carlson
(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.