From 5959ec6b84d66592a77a3e5e2d2aedc1b3e7d4c5 Mon Sep 17 00:00:00 2001 From: Ting-Yu Lin Date: Tue, 3 Sep 2024 04:25:43 +0000 Subject: [PATCH] Bug 1909761 Part 6 - Delete or improve obsolete comments in layout. r=layout-reviewers,emilio Differential Revision: https://phabricator.services.mozilla.com/D219525 --- layout/generic/ReflowInput.h | 5 +---- layout/generic/nsHTMLCanvasFrame.cpp | 2 -- layout/generic/nsImageFrame.cpp | 2 -- 3 files changed, 1 insertion(+), 8 deletions(-) diff --git a/layout/generic/ReflowInput.h b/layout/generic/ReflowInput.h index e35245c79804..e0473540b808 100644 --- a/layout/generic/ReflowInput.h +++ b/layout/generic/ReflowInput.h @@ -225,10 +225,7 @@ struct SizeComputationInput { }; /** - * State passed to a frame during reflow or intrinsic size calculation. - * - * XXX Refactor so only a base class (nsSizingState?) is used for intrinsic - * size calculation. + * State passed to a frame during reflow. * * @see nsIFrame#Reflow() */ diff --git a/layout/generic/nsHTMLCanvasFrame.cpp b/layout/generic/nsHTMLCanvasFrame.cpp index 857ec7301cf1..e6497a81a6ce 100644 --- a/layout/generic/nsHTMLCanvasFrame.cpp +++ b/layout/generic/nsHTMLCanvasFrame.cpp @@ -367,8 +367,6 @@ CSSIntSize nsHTMLCanvasFrame::GetCanvasSize() const { nscoord nsHTMLCanvasFrame::IntrinsicISize(const IntrinsicSizeInput& aInput, IntrinsicISizeType aType) { - // XXX The caller doesn't account for constraints of the height, - // min-height, and max-height properties. if (Maybe containISize = ContainIntrinsicISize()) { return *containISize; } diff --git a/layout/generic/nsImageFrame.cpp b/layout/generic/nsImageFrame.cpp index 4a6abc445ef5..f3e1586d9137 100644 --- a/layout/generic/nsImageFrame.cpp +++ b/layout/generic/nsImageFrame.cpp @@ -1469,8 +1469,6 @@ nscoord nsImageFrame::GetContinuationOffset() const { nscoord nsImageFrame::IntrinsicISize(const IntrinsicSizeInput& aInput, IntrinsicISizeType aType) { - // XXX The caller doesn't account for constraints of the block-size, - // min-block-size, and max-block-size properties. EnsureIntrinsicSizeAndRatio(); return mIntrinsicSize.ISize(GetWritingMode()).valueOr(0); }