Remove nsCertVerificationThread.h from includes after bug 1284946

RESOLVED FIXED in Thunderbird 50.0

Status

MailNews Core
Security
--
blocker
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: Paenglab, Assigned: Paenglab)

Tracking

({dogfood})

unspecified
Thunderbird 50.0
dogfood

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
The build fails with: z:/Mozilla/comm-central/mailnews/mime/src/nsCMS.cpp(12): fatal error C1083: Cannot open include file: 'nsCertVerificationThread.h': No such file or directory
(Assignee)

Comment 1

2 years ago
Created attachment 8772429 [details] [diff] [review]
Bug1287825.patch

With only removing the include it builds again and TB is working with no issue for me. I don't know if something else is needed.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8772429 - Flags: review?(mkmelin+mozilla)
(Assignee)

Updated

2 years ago
Severity: normal → blocker
Sorry - I forgot to check for instances of what bug 1284946 was going to remove in comm-central. Along those lines, it looks like the nsIX509Cert constants VERIFIED_OK, NOT_VERIFIED_UNKNOWN, etc. will have to be dealt with here: https://dxr.mozilla.org/comm-central/rev/752a2122c521c5115049f4fbb4229484a05b2c2c/mailnews/extensions/smime/content/msgCompSecurityInfo.js#24
As far as I can tell, that function is only being used from line 204 of that file. The values passed to it come from the call to nsISMimeJSHelper.getRecipientCertsInfo on line 89. Looking at the implementation, those values appear to all just be 0: https://dxr.mozilla.org/comm-central/source/mailnews/extensions/smime/src/nsSMimeJSHelper.cpp#97 so this implementation can be simplified somewhat.

Comment 3

2 years ago
Comment on attachment 8772429 [details] [diff] [review]
Bug1287825.patch

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

rs=mkmelin
Attachment #8772429 - Flags: review?(mkmelin+mozilla) → review+
(Assignee)

Comment 4

2 years ago
https://hg.mozilla.org/comm-central/rev/e8d79693a47a
Target Milestone: --- → Thunderbird 50.0
(Assignee)

Comment 5

2 years ago
(In reply to David Keeler [:keeler] (use needinfo?) from comment #2)
> Sorry - I forgot to check for instances of what bug 1284946 was going to
> remove in comm-central. Along those lines, it looks like the nsIX509Cert
> constants VERIFIED_OK, NOT_VERIFIED_UNKNOWN, etc. will have to be dealt with
> here:
> https://dxr.mozilla.org/comm-central/rev/
> 752a2122c521c5115049f4fbb4229484a05b2c2c/mailnews/extensions/smime/content/
> msgCompSecurityInfo.js#24
> As far as I can tell, that function is only being used from line 204 of that
> file. The values passed to it come from the call to
> nsISMimeJSHelper.getRecipientCertsInfo on line 89. Looking at the
> implementation, those values appear to all just be 0:
> https://dxr.mozilla.org/comm-central/source/mailnews/extensions/smime/src/
> nsSMimeJSHelper.cpp#97 so this implementation can be simplified somewhat.

Thank you, David.

Magnus, should I file a new bug for this cleanup or do we this in this bug? My knowledge is too low to fix this.
Flags: needinfo?(mkmelin+mozilla)
(Assignee)

Updated

2 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED

Comment 6

2 years ago
Filed bug 1293378.
Flags: needinfo?(mkmelin+mozilla)
You need to log in before you can comment on or make changes to this bug.