Remove "URL Protocol" from some non-protocol keys

RESOLVED FIXED in Thunderbird 25.0

Status

RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: emk, Assigned: emk)

Tracking

(Depends on: 1 bug)

unspecified
Thunderbird 25.0
x86_64
Windows 8
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Created attachment 764132 [details] [diff] [review]
patch

This patch depends on the m-c part of bug 883855.
Attachment #764132 - Flags: review?(mbanner)

Updated

5 years ago
Assignee: nobody → VYV03354
Status: NEW → ASSIGNED
Comment on attachment 764132 [details] [diff] [review]
patch

Looks fine, though I've not tested it. Thanks for the patch.
Attachment #764132 - Flags: review?(mbanner) → review+
(Assignee)

Comment 2

5 years ago
Will c-c be reopened in future? Or do I have to ask an approval to land the fix forever?
(Assignee)

Comment 3

5 years ago
Bug 883855 has been landed. I would like to land this bug as soon as possible for consistency. How can I ask an approval? ("Flags" did not contain "approval‑comm‑central".)
(Assignee)

Updated

5 years ago
Flags: needinfo?(mbanner)
Generally people can ping me on irc for asking approval. However, here is also fine. I'd prefer this not to land at the moment, as our Windows builds are busted so we have no way of tracking if it causes a regression or not. The good news is releng are now working on bug 869974, so I'm hopeful this would be fixed later this week.

You can always use checkin-needed if you want to ensure that we don't forget it.
Flags: needinfo?(mbanner)
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/f5f4d4b6a815
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 25.0
Depends on: 921018
You need to log in before you can comment on or make changes to this bug.