Closed Bug 1923541 Opened 1 year ago Closed 1 year ago

Create a storeToken attribute on nsIMsgDBHdr

Categories

(MailNews Core :: Backend, enhancement)

Product:
MailNews Core
Component:
Backend
Type:
enhancement

Tracking

(thunderbird_esr128 fixed, thunderbird132 fixed)

Status:
RESOLVED FIXED
Milestone:
133 Branch
Tracking Flags:
Tracking Status
thunderbird_esr128 --- fixed
thunderbird132 --- fixed

People

(Reporter: darktrojan, Assigned: darktrojan)

References

Details

Attachments

(2 files)

Bug 1923541 - Create a storeToken attribute on nsIMsgDBHdr. r=BenC,#thunderbird-reviewers
48 bytes, text/x-phabricator-request
corey
: approval-comm-beta+
corey
: approval-comm-esr128+
Details | Review
Bug 1923541 - Use storeToken attribute instead of string property where appropriate. r=BenC,#thunderbird-reviewers
48 bytes, text/x-phabricator-request
corey
: approval-comm-beta+
corey
: approval-comm-esr128+
Details | Review

I'm adding this attribute to aid with the removal of messageOffset. See bug 1720047 comment 21.

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/1539420bc34c
Create a storeToken attribute on nsIMsgDBHdr. r=BenC
https://hg.mozilla.org/comm-central/rev/088c06e9d567
Use storeToken attribute instead of string property where appropriate. r=BenC

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
See Also: → 1923765

Comment on attachment 9429772 [details]
Bug 1923541 - Create a storeToken attribute on nsIMsgDBHdr. r=BenC,#thunderbird-reviewers

Patch 1 of a pile of 6 for the corrupted message issue.

[Approval Request Comment]
Regression caused by (bug #): not a regression
User impact if declined: none
Testing completed (on c-c, etc.): 4 days on central
Risk to taking this patch (and alternatives if risky): very low

Attachment #9429772 - Flags: approval-comm-beta?

Comment on attachment 9429776 [details]
Bug 1923541 - Use storeToken attribute instead of string property where appropriate. r=BenC,#thunderbird-reviewers

Patch 2 of a pile of 6 for the corrupted message issue.

[Approval Request Comment]
Regression caused by (bug #): not a regression
User impact if declined: none
Testing completed (on c-c, etc.): 4 days on central
Risk to taking this patch (and alternatives if risky): low

Attachment #9429776 - Flags: approval-comm-beta?

Comment on attachment 9429772 [details]
Bug 1923541 - Create a storeToken attribute on nsIMsgDBHdr. r=BenC,#thunderbird-reviewers

[Triage Comment]
Approved for beta

Attachment #9429772 - Flags: approval-comm-beta? → approval-comm-beta+

Comment on attachment 9429776 [details]
Bug 1923541 - Use storeToken attribute instead of string property where appropriate. r=BenC,#thunderbird-reviewers

[Triage Comment]
Approved for beta

Attachment #9429776 - Flags: approval-comm-beta? → approval-comm-beta+

Comment on attachment 9429772 [details]
Bug 1923541 - Create a storeToken attribute on nsIMsgDBHdr. r=BenC,#thunderbird-reviewers

[Approval Request Comment]
See comment 4, don't make me repeat myself. :-)

Attachment #9429772 - Flags: approval-comm-esr128?

Comment on attachment 9429776 [details]
Bug 1923541 - Use storeToken attribute instead of string property where appropriate. r=BenC,#thunderbird-reviewers

[Approval Request Comment]
See comment 5.

Attachment #9429776 - Flags: approval-comm-esr128?

Comment on attachment 9429772 [details]
Bug 1923541 - Create a storeToken attribute on nsIMsgDBHdr. r=BenC,#thunderbird-reviewers

[Triage Comment]
Approved for esr128

Attachment #9429772 - Flags: approval-comm-esr128? → approval-comm-esr128+

Comment on attachment 9429776 [details]
Bug 1923541 - Use storeToken attribute instead of string property where appropriate. r=BenC,#thunderbird-reviewers

[Triage Comment]
Approved for esr128

Attachment #9429776 - Flags: approval-comm-esr128? → approval-comm-esr128+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: