Review of attachment 8561373:
-----------------------------------------------------------------
::: mailnews/compose/test/unit/test_messageHeaders.js
@@ +91,5 @@
> fields.organization = "World Salvation Committee";
> fields.subject = "This is an obscure reference";
> yield richCreateMessage(fields, [], identity);
> checkDraftHeaders({
> + // As of bug 87987 the identity does not override the from header.
Comment on attachment 8561373
Test fix
Review of attachment 8561373: ------- ------- ------- ------- ------- ------- ------- ------- --
-------
::: mailnews/ compose/ test/unit/ test_messageHea ders.js ge(fields, [], identity);
@@ +91,5 @@
> fields.organization = "World Salvation Committee";
> fields.subject = "This is an obscure reference";
> yield richCreateMessa
> checkDraftHeaders({
> + // As of bug 87987 the identity does not override the from header.
Nit: comma after 'bug 87987'
@@ +96,1 @@
> "From": "Me <email address hidden>",
I assume you meant to also change the expected result here? Otherwise, this patch does nothing.