corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1888702 Part 2 - Remove PrependPageBreakItem() and simplify AppendPageBreakItem(). r=layout-reviewers,emilio

Browse Source 
The last caller of PrependPageBreakItem() is removed in
https://hg.mozilla.org/mozilla-central/rev/97613a0619b2

Since we only need to append page break items, we don't need
InsertPageBreakItem() and InsertPageBreakLocation enum. We can just implement
AppendPageBreakItem directly.

Differential Revision: https://phabricator.services.mozilla.com/D206150
This commit is contained in:
Ting-Yu Lin
2024-03-29 23:16:46 +00:00
parent a299dff332
commit 14561b86e0
2 changed files with 6 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
layout/base/nsCSSFrameConstructor.cpp
View File

@@ -4985,9 +4985,8 @@ nsCSSFrameConstructor::FindSVGData(const Element& aElement,
return data;
}
void nsCSSFrameConstructor::InsertPageBreakItem(
nsIContent* aContent, FrameConstructionItemList& aItems,
InsertPageBreakLocation location) {
void nsCSSFrameConstructor::AppendPageBreakItem(
nsIContent* aContent, FrameConstructionItemList& aItems) {
RefPtr<ComputedStyle> pseudoStyle =
mPresShell->StyleSet()->ResolveNonInheritingAnonymousBoxStyle(
PseudoStyleType::pageBreak);
@@ -4997,13 +4996,8 @@ void nsCSSFrameConstructor::InsertPageBreakItem(
static constexpr FrameConstructionData sPageBreakData(NS_NewPageBreakFrame,
FCDATA_SKIP_FRAMESET);
if (location == InsertPageBreakLocation::eBefore) {
aItems.PrependItem(this, &sPageBreakData, aContent, pseudoStyle.forget(),
true);
} else {
aItems.AppendItem(this, &sPageBreakData, aContent, pseudoStyle.forget(),
true);
}
aItems.AppendItem(this, &sPageBreakData, aContent, pseudoStyle.forget(),
true);
}
bool nsCSSFrameConstructor::ShouldCreateItemsForChild(