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)

enhancement
Points:
1

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.

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
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.

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)

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
Status: ASSIGNED → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → 145 Branch
QA Whiteboard: [qa-triage-done-c146/b145]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: