Closed Bug 1627927 Opened 2 years ago Closed 2 years ago

Rewrite Thunderbird callers of chrome "redirects" such as chrome://global/skin/ to chrome://global/skin/global.css

Categories

(Thunderbird :: General, task)

Product:
Thunderbird
Component:
General
Type:
task
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
Thunderbird 77.0

People

(Reporter: mkmelin, Assigned: mkmelin)

References

(Blocks 1 open bug)

Details

Attachments

(3 files)

bug1627927_chrome_redirects_rm.patch
78.02 KB, patch
pmorris
: review+
Details | Diff | Splinter Review
bug1627927_chrome_redirects_rm.messenger.patch
96.72 KB, patch
pmorris
: review+
Details | Diff | Splinter Review
bug1627927_chrome_redir_rm.editor.patch
1.46 KB, patch
pmorris
: review+
Details | Diff | Splinter Review

Bug 1601122 will remove such redirects. Bug 1623992 did some rewriting already.

Summary: Rewrite callers of chrome "redirects" such as chrome://global/skin/ to chrome://global/skin/global.css → Rewrite Thunderbird callers of chrome "redirects" such as chrome://global/skin/ to chrome://global/skin/global.css

global

Attachment #9138798 - Flags: review?(paul)

messenger

Attachment #9138799 - Flags: review?(paul)

editor

Attachment #9138800 - Flags: review?(paul)
Status: NEW → ASSIGNED
Comment on attachment 9138798 [details] [diff] [review]
bug1627927_chrome_redirects_rm.patch

Review of attachment 9138798 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM.
Attachment #9138798 - Flags: review?(paul) → review+
Comment on attachment 9138799 [details] [diff] [review]
bug1627927_chrome_redirects_rm.messenger.patch

Review of attachment 9138799 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM.
Attachment #9138799 - Flags: review?(paul) → review+
Comment on attachment 9138800 [details] [diff] [review]
bug1627927_chrome_redir_rm.editor.patch

Review of attachment 9138800 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM.
Attachment #9138800 - Flags: review?(paul) → review+

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/02acd483616d
Rewrite callers of chrome "redirects" chrome://global/skin/ to chrome://global/skin/global.css. r=pmorris
https://hg.mozilla.org/comm-central/rev/01f9b5a1e9f8
Rewrite callers of chrome "redirects" chrome://messenger/skin/ to chrome://messenger/skin/messenger.css. r=pmorris
https://hg.mozilla.org/comm-central/rev/aab13fd352ab
remove dead chrome://editor/skin/ chrome redirect. r=pmorris

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 77.0
Regressions: 1648585
You need to log in before you can comment on or make changes to this bug.