Closed Bug 155638 Opened 22 years ago Closed 22 years ago

Need to back out additional fix for bug 137071on branch

Categories

(MailNews Core :: MIME, defect, P1)

Product:
MailNews Core
Component:
MIME
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
mozilla1.0.1

People

(Reporter: bugzilla, Assigned: bugzilla)

Details

(Whiteboard: [adt1 rtm])

Attachments

(1 file, 1 obsolete file)

Proposed fix, v2
3.54 KB, patch
nhottanscp
: review+
Bienvenu
: superreview+
chofmann
: approval+
Details | Diff | Splinter Review 
The additional fix for bug 137071 to allow users to correctly forward inline
encrypted messages has cause several regressions (bug 155253, bug 154734).
Attamp to fix the firt level of regression has caused other regressions (bug
155476, bug 155629)

Therefore, we need to backup part of the fix for bug 137071 as well the fix for
regression 154734 and maybe 155253 until we are able to get the right fix.
nominating nsbeta1
Keywords: nsbeta1
Please add the patch and get reviews.
Keywords: nsbeta1 → nsbeta1+
Summary: Need to backup additional fix for bug 137071on branch → Need to back out additional fix for bug 137071on branch
Whiteboard: [adt1 rtm]
Severity: normal → major
Status: NEW → ASSIGNED
Priority: -- → P1
Target Milestone: --- → mozilla1.0.1
Attached patch Proposed fix, v1 (obsolete) — — Splinter Review 
This backup the additional fix for bug 137071 as well the regression fix for
bug 154734
QA Contact: gayatri → meehansqa
When verifying this defect, we need to re-verify bug 137071 (Charles) and bug
154734 (Stephen).
Attached patch Proposed fix, v2 — — Splinter Review 
More complete backout, I forget to remove the extra parameter originalMsgURI in
the function -CreateTheComposeWindow
Attachment #90106 - Attachment is obsolete: true
Comment on attachment 90113 [details] [diff] [review]
Proposed fix, v2

r=nhotta
Attachment #90113 - Flags: review+
Attachment #90113 - Flags: superreview+
Comment on attachment 90113 [details] [diff] [review]
Proposed fix, v2

sr=bienvenu
adding adt1.0.1+.  Please get drivers approval before checking into the branch.
Keywords: adt1.0.1+, mozilla1.0.1
Comment on attachment 90113 [details] [diff] [review]
Proposed fix, v2

a=chofmann for 1.0.1
Attachment #90113 - Flags: approval+
Fix checked in the branch.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Keywords: mozilla1.0.1 → fixed1.0.1
Resolution: --- → FIXED
Verified FIXED for the branch -  See the separate verifications of

myself in bug 154734 - http://bugzilla.mozilla.org/show_bug.cgi?id=154734#c40
Charles in bug 137071 - http://bugzilla.mozilla.org/show_bug.cgi?id=137071#c53
Status: RESOLVED → VERIFIED
Keywords: fixed1.0.1 → verified1.0.1
Also verified other two regressions related to this fix:

bug 155253 - verified
bug 155176 - dup of bug 58724, not a regression of this fix
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: