Closed
Bug 1431021
Opened 7 years ago
Closed 7 years ago
Remove unused isContentFrame from contentAreaUtils.js
Categories
(Toolkit :: General, enhancement)
Toolkit
General
Tracking
()
RESOLVED
FIXED
mozilla59
Tracking | Status | |
---|---|---|
firefox59 | --- | fixed |
People
(Reporter: standard8, Assigned: standard8)
References
Details
Attachments
(1 file)
As part of cleaning up browser window globals, we should remove isContentFrame from contentAreaUtils.js as it is not used.
Comment hidden (mozreview-request) |
Comment 2•7 years ago
|
||
mozreview-review |
Comment on attachment 8943156 [details]
Bug 1431021 - Remove unused isContentFrame from contentAreaUtils.js.
https://reviewboard.mozilla.org/r/213504/#review219262
I agree we should remove this, but it's used in:
editor/ui/composer/content/editorApplicationOverlay.js
suite/browser/navigator.js
maybe needinfo someone from SeaMonkey to take care of copying that function where they want it?
Attachment #8943156 -
Flags: review?(florian) → review+
Assignee | ||
Comment 3•7 years ago
|
||
fgr: fyi, this will be landing soon. You may want to move the function to SeaMonkey somewhere.
Flags: needinfo?(frgrahl)
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2768c5b6f287
Remove unused isContentFrame from contentAreaUtils.js. r=florian
Comment 6•7 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox59:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
You need to log in
before you can comment on or make changes to this bug.
Description
•