Bug 1684236 Comment 4 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 
I see that you basically handle the fragmentation case as I suggested here:
>+      || IsTrueOverflowContainer() // nothing to shift
>+      || GetPrevInFlow() || !aState.mReflowStatus.IsFullyComplete()) {
>+    return;

FYI, I think the `IsTrueOverflowContainer()` is redundant since `GetPrevInFlow()` is always true for those.
Revision 1 by
on 
I see that you basically handle the fragmentation case as I suggested here:
```
      || IsTrueOverflowContainer() // nothing to shift
      || GetPrevInFlow() || !aState.mReflowStatus.IsFullyComplete()) {
    return;
```

FYI, I think the `IsTrueOverflowContainer()` is redundant since `GetPrevInFlow()` is always true for those.

Back to Bug 1684236 Comment 4