1383319 - stylo: has_current_style debug assertion on test-case with animation and innerHTML.

Status: RESOLVED DUPLICATE of bug 1383001

(Core :: CSS Parsing and Computation, defect, P2)

Description

[Emilio Cobos Álvarez [:emilio]]

Seen while investigating bug 1383001.

Comment 1

[Emilio Cobos Álvarez [:emilio]]

Attachment: testcase

display: contents is just to disable lazy frame construction.

Comment 2

[Emilio Cobos Álvarez [:emilio]]

(contenteditable also works, and would be nicer for a reftest, since ideally we'd make the display: contents stuff work with lazy frame construction).

Comment 3

[Emilio Cobos Álvarez [:emilio]]

Hiro, I think this is because we restyle, then do a frame constructor pass with the outdated styles... Then update animations.

Is that right? If so, what's the course of action here? Tweaking the assertion would work, I guess...

Comment 4

[Hiroyuki Ikezoe (:hiro)]

(In reply to Emilio Cobos Álvarez [:emilio] from comment #3)
> Hiro, I think this is because we restyle, then do a frame constructor pass
> with the outdated styles... Then update animations.
> 
> Is that right? If so, what's the course of action here? Tweaking the
> assertion would work, I guess...

Yeah, I am also thinking tweaking the assertion is a way we can do here. Let me think about the effective assertion.

Comment 5

[Manish Goregaokar [:manishearth]]

(P2 since it's a debug assertion which seems to be incorrect. Should be fixed but no rush)

Comment 6

[Emilio Cobos Álvarez [:emilio]]

I fixed this in bug 1383001.

Comment 7

[Hiroyuki Ikezoe (:hiro)]

Thanks again and again. :)