(no bug) Update ResolveFlexibleLengths() documentation & drop whitespace on blank line. comment/whitespace-only

(Prior to this commit, the documentation mentioned long-since-removed params aTotalFreeSpace and aItems.)
This commit is contained in:
Daniel Holbert 2014-04-11 14:22:10 -07:00
parent d4dd4d5972
commit 44fd755c5a

View File

@ -686,8 +686,8 @@ public:
return mBaselineOffset;
}
// Runs the "resolve the flexible lengths" algorithm, distributing
// |aFlexContainerMainSize| among the |aItems| and freezing them.
// Runs the "Resolving Flexible Lengths" algorithm from section 9.7 of the
// CSS flexbox spec to distribute aFlexContainerMainSize among our flex items.
void ResolveFlexibleLengths(nscoord aFlexContainerMainSize);
void PositionItemsInMainAxis(uint8_t aJustifyContent,
@ -698,7 +698,7 @@ public:
const FlexboxAxisTracker& aAxisTracker);
friend class AutoFlexLineListClearer; // (needs access to mItems)
private:
// Helper for ResolveFlexibleLengths():
void FreezeOrRestoreEachFlexibleSize(const nscoord aTotalViolation,
@ -1617,12 +1617,6 @@ FlexLine::FreezeOrRestoreEachFlexibleSize(const nscoord aTotalViolation,
}
}
// Implementation of flexbox spec's "resolve the flexible lengths" algorithm.
// NOTE: aTotalFreeSpace should already have the flex items' margin, border,
// & padding values subtracted out, so that all we need to do is distribute the
// remaining free space among content-box sizes. (The spec deals with
// margin-box sizes, but we can have fewer values in play & a simpler algorithm
// if we subtract margin/border/padding up front.)
void
FlexLine::ResolveFlexibleLengths(nscoord aFlexContainerMainSize)
{