WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED WONTFIX
123948
Block load event dispatching on old document when an HTMLMediaElement is moved between documents
https://bugs.webkit.org/show_bug.cgi?id=123948
Summary
Block load event dispatching on old document when an HTMLMediaElement is move...
Ryosuke Niwa
Reported
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.
Attachments
Add attachment
proposed patch, testcase, etc.
Eric Carlson
Comment 1
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.
Eric Carlson
Comment 2
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.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug