Closed
Bug 1698680
Opened 4 years ago
Closed 4 years ago
Add nsLayoutUtils::GetCrossDocParentFrameInProcess()
Categories
(Core :: Layout, task)
Core
Layout
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox88 | --- | fixed |
People
(Reporter: botond, Assigned: botond)
References
Details
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
Details |
To make bug 1599913 (Audit usage of nsLayoutUtils::GetCrossDocParentFrame) easier to do, I propose adding a new function, nsLayoutUtils::GetCrossDocParentFrameInProcess().
The two functions would do the same thing (with one dispatching to the other), but we would convert call sites to the InProcess version as we audit them, so it's easy to tell which ones haven't been audited yet.
|
||
Comment 1•4 years ago
|
||
Sounds like a good plan! Thanks, Botond.
|
Assignee | |
Comment 2•4 years ago
|
||
|
|
Assignee | |
Comment 3•4 years ago
|
||
One question here is: what to do about call sites in code that's only used on Android? There may be some of those that need fixing in the future, but not for M7.
Pushed by bballo@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f1ae93d282f6
Add nsLayoutUtils::GetCrossDocParentFrameInProcess(). r=dholbert
|
||
Comment 5•4 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox88:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 88 Branch
|
||
Comment 6•4 years ago
|
||
Setting Fission Milestone to M7a (the current Beta milestone) because this bug is blocking meta bug 1599913 which is a blocker for Fission M7a.
Fission Milestone: --- → M7a
You need to log in
before you can comment on or make changes to this bug.
Description
•