Bug 115696
| Summary: | Remove enforcement for having the first child as RenderInline. | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> |
| Component: | Media | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED INVALID | ||
| Severity: | Normal | CC: | eric.carlson, mjs |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Ryosuke Niwa
We should consider merging
https://chromium.googlesource.com/chromium/blink/+/a7e0d7b2e1a26326adf18b6d39c15beb960b5f59
Remove enforcement for having the first child as RenderInline.
If the added cue is not inline then most almost certainly an
external factor (e.g. such as an external stylesheet as in
the testcase) has modified the default display: value for the cue.
If this is the case, there's no need or reason to run the WebVTT
algorithm for positioning the as we don't really have to support
these cases.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Eric Carlson
The test case attached to that bug does not crash for me, but I would be surprised if it crashed Chromium even without the patch because of the way text tracks work.
While investigating it I did find another crash, which I fixed in http://trac.webkit.org/changeset/149766.
Eric Carlson
Marking as Invalid for now. We can reopen if I am wrong.