Closed
Bug 1979643
Opened 4 months ago
Closed 4 months ago
Minor cleanups in grid sizing code
Categories
(Core :: Layout: Grid, task)
Core
Layout: Grid
Tracking
()
RESOLVED
FIXED
143 Branch
| Tracking | Status | |
|---|---|---|
| firefox143 | --- | fixed |
People
(Reporter: jfkthame, Assigned: jfkthame)
References
(Blocks 1 open bug)
Details
Attachments
(3 files)
Filing this for a few minor code cleanup issues I noticed while reading grid sizing code related to bug 1951829. None of this should affect behavior in any way.
|
Assignee | |
Comment 1•4 months ago
|
||
|
||
Updated•4 months ago
|
Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 2•4 months ago
|
||
|
|
Assignee | |
Comment 3•4 months ago
|
||
Pushed by jkew@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/db6a9bcfb015
https://hg.mozilla.org/integration/autoland/rev/0a150f54cb21
Use MaxSizingFor(...) as a more idiomatic form than SizingFor(...).GetMax() r=dholbert
https://github.com/mozilla-firefox/firefox/commit/aa04b5f08dd8
https://hg.mozilla.org/integration/autoland/rev/3c2de31e85c8
Modernize a for-loop in nsGridContainerFrame::Tracks::FindFrUnitSize r=dholbert
https://github.com/mozilla-firefox/firefox/commit/0c318b19faf4
https://hg.mozilla.org/integration/autoland/rev/4b203e5f6830
Make a few more things 'const' in the grid sizing code. r=dholbert
|
||
Comment 5•4 months ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/0a150f54cb21
https://hg.mozilla.org/mozilla-central/rev/3c2de31e85c8
https://hg.mozilla.org/mozilla-central/rev/4b203e5f6830
Status: ASSIGNED → RESOLVED
Closed: 4 months ago
status-firefox143:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 143 Branch
|
||
Updated•3 months ago
|
QA Whiteboard: [qa-triage-done-c144/b143]
You need to log in
before you can comment on or make changes to this bug.
Description
•