User Details
- User Since
- Oct 11 2017, 9:39 AM (422 w, 21 h)
- Availability
- Available
- Review Queue
- 1
Today
Try result of the patch stack:
Yesterday
This patch has been landed.
Tue, Nov 11
Mon, Nov 10
Added a colon to improve the log.
To see this log, execute:
Rebased
Rebased
Rebased
Fri, Nov 7
Wed, Nov 5
Removing the pref declaration is not enough, we should remove the usage as well. For example, mFontSizeInflationDisabledInMasterProcess in PresShell.
https://searchfox.org/firefox-main/rev/cb52781342cc905eda923d009fc0b678f3a8c8c6/layout/base/PresShell.cpp#912-913
Tue, Nov 4
Mon, Nov 3
Sun, Nov 2
Can we test this patch?
This revision requires a Testing Policy Project Tag to be set before landing. Please apply one of testing-approved, testing-exception-unchanged, testing-exception-ui, testing-exception-elsewhere, testing-exception-other. Tip: this Firefox add-on makes it easy!
mPresShell is set, but never used. Nice find! (It appears the last usage was removed by bug 1624819 in https://hg-edge.mozilla.org/mozilla-central/diff/324cc7d374d1860a1743fa7f8a33a782e3bd4177/layout/base/nsCaret.cpp)
Sat, Nov 1
Fri, Oct 31
R+ for layout/.
Thu, Oct 30
Yeah, I've always wanted to look into CalculateContainingBlockSizeForAbsolutes() to see if all the complexity is needed. If the try result looks good, let's land this.
I think we'll need to replace trim with inset in this patch.
Wed, Oct 29
Thanks for improving this patch!