Ah, so I bet the problem is that this `FrameNeedsReflow` call is guarded by: ``` if (DependsOnIntrinsicSize(embeddingFrame)) { ``` ...which probably returns "false" for this testcase, because we have a specified px-valued `width` and `height` on the embedding element. So we skip the FrameNeedsReflow call. Really, we probably need to add something like the following (these functions aren't real): ``` else if (CaresAboutIntrinsicRatio(embeddingFrame)) { embeddingFrame->RequestRepaint(...) } ``` ...or somesuch.
Bug 1565384 Comment 6 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
Ah, so I bet the problem is that this `FrameNeedsReflow` call is guarded by: ``` if (DependsOnIntrinsicSize(embeddingFrame)) { ``` ...which probably returns "false" for this testcase, because we have a specified px-valued `width` and `height` on the embedding element. So we skip the FrameNeedsReflow call. Really, we probably need to add something like the following (these functions aren't real): ``` else if (CaresAboutIntrinsicRatio(embeddingFrame)) { embeddingFrame->RequestRepaint(...) } ``` ...or somesuch.