Closed Bug 1304399 Opened 8 years ago Closed 8 years ago

Port bug bug 1301249 to Mailnews - Check return value of nsIDocument::GetDocumentURI()

Categories

(MailNews Core :: Composition, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 52.0

People

(Reporter: jorgk-bmo, Assigned: jorgk-bmo)

Details

Summary: Check return value of nsIDocument::GetDocumentURI() → Port bug bug 1301249 to Mailnews - Check return value of nsIDocument::GetDocumentURI()
Assignee: nobody → jorgk
Status: NEW → ASSIGNED
Component: General → Composition
OS: Unspecified → All
Product: Thunderbird → MailNews Core
Hardware: Unspecified → All
https://hg.mozilla.org/comm-central/rev/b377951bcd31

Landed trivial fix. Positive post-landing reviews welcome ;-)

<off-topic>
We had this discussion about landing patches without review. I'm not a great friend of that, but it's OK of it's really trivial and obvious and has been done many times before the very same way.

And hey, nobody complained about this:
https://hg.mozilla.org/comm-central/rev/a09c05db1261
https://hg.mozilla.org/comm-central/rev/66ffb3c06576
;-)
</off-topic>
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 52.0
In this case I was around so I could review it quickly :)
You have my r+ now, but there is no patch to attach it to :)
You need to log in before you can comment on or make changes to this bug.