Bug 1658198 Comment 18 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

I load the test case in my local firefox debug build with this commend `MOZ_LOG=ColumnSet:5 ./mach run --layoutdebug https://www.picuki.com/profile/justinchapple`, and see log like the following:

```
[Child 31817: Main Thread]: D/ColumnSet FindBestBalanceBSize: Choosing next guess=444401, iteration=893
...
[Child 31817: Main Thread]: D/ColumnSet FindBestBalanceBSize: Choosing next guess=444402, iteration=894
...
[Child 31817: Main Thread]: D/ColumnSet FindBestBalanceBSize: Choosing next guess=444403, iteration=895
...
```

Apparently, the URL made firefox goes into the code path in [nsColumnSetFrame::FindBestBalanceBSize](https://searchfox.org/mozilla-central/rev/50cb0892948fb4291b9a6b1b30122100ec7d4ef2/layout/generic/nsColumnSetFrame.cpp#981) where it linear search the column balancing size.
I load the test case in my local firefox debug build with this commend `MOZ_LOG=ColumnSet:5 ./mach run --layoutdebug https://www.picuki.com/profile/justinchapple`, and see log like the following:

```
[Child 31817: Main Thread]: D/ColumnSet FindBestBalanceBSize: Choosing next guess=444401, iteration=893
...
[Child 31817: Main Thread]: D/ColumnSet FindBestBalanceBSize: Choosing next guess=444402, iteration=894
...
[Child 31817: Main Thread]: D/ColumnSet FindBestBalanceBSize: Choosing next guess=444403, iteration=895
...
```

Apparently, the URL made firefox goes into the code path in [nsColumnSetFrame::FindBestBalanceBSize](https://searchfox.org/mozilla-central/rev/50cb0892948fb4291b9a6b1b30122100ec7d4ef2/layout/generic/nsColumnSetFrame.cpp#981) where it uses a linear search for the column balancing size, which it shouldn't fall into the path.

Back to Bug 1658198 Comment 18