Closed Bug 979760 Opened 6 years ago Closed 6 years ago

Pull out a BeginTransaction call common to the start of all three branches of if/else logic

Categories

(Core :: Web Painting, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla30

People

(Reporter: dbaron, Assigned: dbaron)

Details

Attachments

(1 file)

I don't see any particular reason for them to logically be in each
branch, and it seems the code was originally written with the
BeginTransaction in one place, but later had to be refactored into its
current form.

Note that this separates the comment from one of the EndEmptyTransaction
calls below it, but the comment was actually associated primarily with
the further EndEmptyTransaction call, and with the if above it, based on
the history pointing to
https://hg.mozilla.org/mozilla-central/rev/b4e9a17e7fe2
Attachment #8385920 - Flags: review?(matt.woodrow) → review+
https://hg.mozilla.org/mozilla-central/rev/539db8ea943a
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
Component: Layout: View Rendering → Layout: Web Painting
You need to log in before you can comment on or make changes to this bug.