1610999 - Port |Bug 1392221 - FeedWriter doesn't implement nsINavHistoryObserver nor does it need to - so remove it from the QI definition| to SeaMonkey

Status: RESOLVED FIXED

(SeaMonkey :: Feed Discovery and Preview, task)

Description

[Ian Neal]

FeedWriter is declaring that it implements Ci.nsINavHistoryObserver, but it doesn't need to, nor does it implement it.
Bug 627416 stopped FeedWriter being an observer of Places history, and removed the implementation of that observer, but didn't remove the declaration line.
We can simply remove the line from the QueryInterface definition.

Comment 1

[Ian Neal]

Attachment: Port simple removal

Applies cleanly on all repos

Comment 2

[Ian Neal]

Comment on attachment 9122512 [details] [diff] [review]
Port simple removal

[Approval Request Comment]
Regression caused by (bug #): 627416
User impact if declined: None
Testing completed (on m-c, etc.): all
Risk to taking this patch (and alternatives if risky): none
String changes made by this patch: none

Comment 3

[Frank-Rainer Grahl (:frg)]

Comment on attachment 9122512 [details] [diff] [review]
Port simple removal

LGTM

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/2bd348484d80
Port |Bug 1392221 - FeedWriter doesn't implement nsINavHistoryObserver nor does it need to - so remove it from the QI definition| to SeaMonkey. r=frg