Closed Bug 1691304 Opened 4 years ago Closed 4 years ago

Tidy up specialTabs.js: remove the chromeTab type and use "url" as the argument to contentTab.openTab

Categories

(Thunderbird :: Toolbars and Tabs, enhancement)

enhancement

Tracking

(thunderbird_esr78 wontfix, thunderbird86 wontfix)

RESOLVED FIXED
87 Branch
Tracking Status
thunderbird_esr78 --- wontfix
thunderbird86 --- wontfix

People

(Reporter: darktrojan, Assigned: darktrojan)

References

Details

Attachments

(2 files)

The chrome tab type is practically the same as the content tab type. It's a browser that displays a document. The duplication is unnecessary.

These tab types take an URL argument for the document to display. Inexplicably it's called contentPage (and chromePage) instead of just url. I'm going to fix that too.

Depends on D104333

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/2baeb3a0f287
Rename "contentPage" and "chromePage" labels to "url". r=mkmelin
https://hg.mozilla.org/comm-central/rev/28dbc09063ce
Remove chromeTab tab type. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 87 Branch
Regressions: 1692057
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: