888310 - Use more Services.jsm in SeaMonkey feed discovery and preview.

Status: RESOLVED FIXED

(SeaMonkey :: Feed Discovery and Preview, defect)

Description

[Philip Chee]

Follow up to Bug 735333 (Use Services.prefs in SeaMonkey code: Feeds)

Comment 1

[Philip Chee]

Attachment: Patch v1.0 Proposed fix. [check-in comment 3]

> +++ b/suite/browser/test/mochitest/test_bug364677.html
> +++ b/suite/browser/test/mochitest/test_bug395533.html
Minor fix-ups for tests that I ran to verify that my patch doesn't break anything.

> +++ b/suite/browser/test/mochitest/test_feed_discovery.html
...
>  addLoadEvent(function() {
> -  // Need privs because the feed seems to have an about:feeds principal or some
> -  // such.  It's not same-origin with us in any case.
> -  netscape.security.PrivilegeManager.enablePrivilege("UniversalXPConnect");
> -  is($("testFrame").contentDocument.documentElement.id, "feedHandler",
> +  is(SpecialPowers.wrap($("testFrame")).contentDocument.documentElement.id, "feedHandler",
>       "Feed served as text/xml without a channel/link should have been sniffed");
This test was broken so I ported over the relevant changes from:
Bug 829872 : Fix up tests that depend on contentDocument being non-null

TEST_PATH=suite/browser/test/test_maxSniffing.html  pymake -C ../objdir-sm/ mochitest-chrome

6 INFO Passed: 2
7 INFO Failed: 0
8 INFO Todo:   0

TEST_PATH=suite/browser/test/test_feed_discovery.html  pymake -C ../objdir-sm/ mochitest-plain

Passed: 27
Failed: 0
Todo: 0

TEST_PATH=suite/browser/test/test_bug364677.html pymake -C ../objdir-sm/ mochitest-plain

Passed: 1
Failed: 0
Todo: 0

TEST_PATH=suite/browser/test/test_bug395533.html pymake -C ../objdir-sm/ mochitest-plain

Passed: 1
Failed: 0
Todo: 0

TEST_PATH=suite/browser/test/test_registerHandler.html pymake -C ../objdir-sm/ mochitest-plain

Passed: 24
Failed: 0
Todo: 1

Comment 2

[Ian Neal]

Comment on attachment 768980 [details] [diff] [review]
Patch v1.0 Proposed fix. [check-in comment 3]

r=me does not appear to break anything (and even fixes stuff too!)

Comment 3

[Philip Chee]

Pushed to comm-central a=Callek for CLOSED TREE
http://hg.mozilla.org/comm-central/rev/b1317c3ff66d