Closed
Bug 1990884
Opened 9 months ago
Closed 9 months ago
Rename and move `AnchorPosReferencedAnchors` from `nsStyleStructs.h`
Categories
(Core :: CSS Parsing and Computation, enhancement)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
145 Branch
| Tracking | Status | |
|---|---|---|
| firefox145 | --- | fixed |
People
(Reporter: dshin, Assigned: dshin)
References
(Blocks 1 open bug)
Details
(Whiteboard: [anchorpositioning:m2])
Attachments
(2 files)
That header is crowded, AnchorPosReferencedAnchors et al. should really live in AnchorPositioningUtils.h.
It should also be renamed since we'll be keeping track of scroll-related data.
Separating it out of bug 1968745 to land this first since it can have repercussions to others. This will take 1 point of work from bug 1968745.
| Assignee | ||
Comment 1•9 months ago
|
||
| Assignee | ||
Comment 2•9 months ago
|
||
Pushed by dshin@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/299d6ecd10c5
https://hg.mozilla.org/integration/autoland/rev/03d0776c5576
Rename AnchorPosReferencedAnchors to AnchorPosReferenceData. r=layout-anchor-positioning-reviewers,firefox-style-system-reviewers,layout-reviewers,emilio
https://github.com/mozilla-firefox/firefox/commit/850a073fceb1
https://hg.mozilla.org/integration/autoland/rev/ff8d942f94ec
Move AnchorPosReferenceData from nsStyleStructs.h to AnchorPositioningUtils.h. r=layout-anchor-positioning-reviewers,firefox-style-system-reviewers,layout-reviewers,emilio
Updated•9 months ago
|
Status: NEW → ASSIGNED
Points: --- → 1
Whiteboard: [anchorpositioning:triage] → [anchorpositioning:m2]
Pushed by sstanca@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/d73470d01e6c
https://hg.mozilla.org/integration/autoland/rev/852c14399af2
Revert "Bug 1990884: Move AnchorPosReferenceData from nsStyleStructs.h to AnchorPositioningUtils.h. r=layout-anchor-positioning-reviewers,firefox-style-system-reviewers,layout-reviewers,emilio" for causing build bustages in nsAbsoluteContainingBlock.h.
Comment 5•9 months ago
|
||
Reverted this because it was causing build bustages in nsAbsoluteContainingBlock.h.
- Revert link
- Push with failures
- Failure Log
- Failure line: /builds/worker/checkouts/gecko/layout/generic/nsAbsoluteContainingBlock.h(122,7): error: unknown type name 'AnchorPosReferenceData'; did you mean 'mozilla::AnchorPosReferenceData'?
Flags: needinfo?(dshin)
| Assignee | ||
Comment 6•9 months ago
|
||
Woops, missed a spot after separating the patch - fixed, and trying again
Flags: needinfo?(dshin)
Pushed by dshin@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/37dcaf1be4ae
https://hg.mozilla.org/integration/autoland/rev/0d13e317d111
Rename AnchorPosReferencedAnchors to AnchorPosReferenceData. r=layout-anchor-positioning-reviewers,firefox-style-system-reviewers,layout-reviewers,emilio
https://github.com/mozilla-firefox/firefox/commit/c43b58897e79
https://hg.mozilla.org/integration/autoland/rev/d5b0b668d63b
Move AnchorPosReferenceData from nsStyleStructs.h to AnchorPositioningUtils.h. r=layout-anchor-positioning-reviewers,firefox-style-system-reviewers,layout-reviewers,emilio
Comment 8•9 months ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/0d13e317d111
https://hg.mozilla.org/mozilla-central/rev/d5b0b668d63b
Status: ASSIGNED → RESOLVED
Closed: 9 months ago
status-firefox145:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 145 Branch
Updated•8 months ago
|
QA Whiteboard: [qa-triage-done-c146/b145]
You need to log in
before you can comment on or make changes to this bug.
Description
•