Closed Bug 1565136 Opened 5 years ago Closed 5 years ago

Split WipeContainingBlock to save work for those checks requiring no frame construction items

Categories

(Core :: Layout, task, P3)

task

Tracking

()

RESOLVED FIXED
mozilla70
Tracking Status
firefox70 --- fixed

People

(Reporter: TYLin, Assigned: TYLin)

Details

Attachments

(1 file)

File this bug per from dbaron's comment in https://phabricator.services.mozilla.com/D36529#1121858.

We currently call WipeContainingBlock after constructing frame construction items, but many of the checks in WipeContainingBlock do not need to inspect these items, e.g. reframing ruby-related frames.

It might worth extracting such logic into another function, and call it before constructing frame construction items to save some work.

Assignee: nobody → aethanyc
Status: NEW → ASSIGNED
Pushed by aethanyc@gmail.com: https://hg.mozilla.org/integration/autoland/rev/66d48a0fef6b Move some reframing logic out of WipeContainingBlock to make it done earlier. r=dholbert
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: