564692 - Sync suite/feeds/ with mozilla-central

Status: RESOLVED FIXED

(SeaMonkey :: Feed Discovery and Preview, defect)

Description

[Philip Chee]

Port over some recent patches from the Firefox RSS feeds component.

Bug 364677 - No preview for sniffed feeds that don't have a channel/link or channel/id
Bug 487511 - nsINavHistoryObserver has no "onBeforeDeleteURI" callback
Bug 524270 - Make the second getChildList parameter optional
Bug 527433 - nsIWebContentConverterRegistrar::getContentHandlers count out param should be optional

* Backout part of Bug 480205 - Implement a wrapper for exposing chrome objects to content (aka COWs)
Part of this bug was backed out of Firefox. The bug is marked as sg: so I can't access it. I've based my patch on the relevant mercurial changeset instead.

I've included the test for Bug 364677 and all the rss feed mochitest-plain tests in suite work.

Comment 1

[Philip Chee]

Attachment: Patch v1.0 with mochitest

Actually attach the patch. I don't suppose Standard8 is doing suite reviews any time soon?

Comment 2

[[e-mail]]

Comment on attachment 444314 [details] [diff] [review]
Patch v1.0 with mochitest

>   init: function init(aWindow) {
>-    // Explicitly wrap |window| in an XPCNativeWrapper to make sure
>-    // it's a real native object! This will throw an exception if we
>-    // get a non-native object.
>-    var window = new XPCNativeWrapper(aWindow);
Asking mrbkap why he removed this in the Firefox version.

Comment 3

[Blake Kaplan (:mrbkap) (inactive)]

Comment on attachment 444314 [details] [diff] [review]
Patch v1.0 with mochitest

It shouldn't be necessarily needed anymore. On the other hand, I realized that it might not have been a great idea to take it out, since it technically increases attack surface. Either way is fine (and safe) though.

Comment 4

[[e-mail]]

Comment on attachment 444314 [details] [diff] [review]
Patch v1.0 with mochitest

r=me with the XPCNativeWrapper guards restored.

Comment 5

[Philip Chee]

Attachment: Patch v1.1 with XPCNativeWrappers r=Neil

Carrying forward r=Neil

>  Blake Kaplan (:mrbkap)      2010-05-10 13:59:56 PDT

> (From update of attachment 444314 [details] [diff] [review])
> It shouldn't be necessarily needed anymore. On the other hand, I realized that
> it might not have been a great idea to take it out, since it technically
> increases attack surface. Either way is fine (and safe) though.

> [e-mail] 2010-05-10 16:16:10 PDT

> (From update of attachment 444314 [details] [diff] [review])
> r=me with the XPCNativeWrapper guards restored.

Fixed.

Comment 6

[Philip Chee]

Checked-in to comm-central
http://hg.mozilla.org/comm-central/rev/7f41e0b5db6b