Skip to content

Commit

Permalink
Bug 1361580 - Initialize nsContinuingTextFrame::mPrevContinuation. r=…
Browse files Browse the repository at this point in the history
…dholbert

In `nsContinuingTextFrame::Init()` [1], we always call `SetPrevInFlow()` to
initialize `mPrevContinuation`, so we are fine. Still, it is better to
initialize the pointer properly before `Init()`.

[1] https://searchfox.org/mozilla-central/rev/d7f837add602d270f2b2958b3ab5206dc85965c0/layout/generic/nsTextFrame.cpp#4230

Differential Revision: https://phabricator.services.mozilla.com/D194407
  • Loading branch information
aethanyc committed Nov 22, 2023
1 parent 1354dc1 commit 25f18e8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion layout/generic/nsTextFrame.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -4215,7 +4215,7 @@ class nsContinuingTextFrame final : public nsTextFrame {
nsPresContext* aPresContext)
: nsTextFrame(aStyle, aPresContext, kClassID) {}

nsTextFrame* mPrevContinuation;
nsTextFrame* mPrevContinuation = nullptr;
nsTextFrame* mFirstContinuation = nullptr;
};

Expand Down

0 comments on commit 25f18e8

Please sign in to comment.