Bug 1142350 will remove from m-c the unused nsIUserCertPicker, nsICertPickDialogs, and associated code only used by the implementations of those interfaces. This code *is* used in c-c however, so this bug will track importing the removed code to keep S/MIME cert selection working.
This patch imports the initial batch of nsIUserCertPicker and nsICertPickDialogs related code removed from m-c. The code is directly copied from m-c with at most trivial changes, and trailing spaces removed.
This patch imports the rest of code removed from m-c to c-c. The code is directly copied from m-c with at most trivial changes, and trailing spaces removed.
Attachment #8793286 - Flags: review?(Pidgeot18)
This patch makes the imported code from the previous patches actually build and work.
Attachment #8793287 - Flags: review?(Pidgeot18)
I've separated the changes required to get things working again into three patches for easier review. I'll fold the changes into a single patch before getting it checked in. jcranmer: I only tested the changes by manually testing on Thunderbird and SeaMonkey that the Account Security > Security cert pickers still work. If there are other tests I should run, please let me know.
Comment on attachment 8793285 [details] [diff] [review] bug1297368_part1a_cp-standalone-files_v1.patch Thanks so much for doing this! If jcranmer does not get to this soon, I'll have a look at it (and the other patches).
Comment on attachment 8793285 [details] [diff] [review] bug1297368_part1a_cp-standalone-files_v1.patch Review of attachment 8793285 [details] [diff] [review]: ----------------------------------------------------------------- I've tested this compiles and builds, and confirmed that the certificate picker still works. I also looked this over and did not see any obvious issues, so let's land it after the dependent patch lands.
Parts 1a, 1b and 2 folded together.
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 52.0
You need to log in before you can comment on or make changes to this bug.