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.

Original comment by
on 
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.
Revision 1 by
on 
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.

Back to Bug 1565384 Comment 6