Closed Bug 784041 Opened 8 years ago Closed 8 years ago

TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/browser/test/browser_bug413915.js | Exception thrown at chrome://mochitests/content/browser/suite/browser/test/browser_bug413915.js:7 - TypeError: secman.getCodebasePrincipal is not a functi

Categories

(SeaMonkey :: Feed Discovery and Preview, defect, P2, major)

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.14

People

(Reporter: philip.chee, Assigned: philip.chee)

References

(Blocks 1 open bug)

Details

(Whiteboard: [perma-orange])

Attachments

(1 file, 1 obsolete file)

TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/browser/test/browser_bug413915.js | Exception thrown at chrome://mochitests/content/browser/suite/browser/test/browser_bug413915.js:7 - TypeError: secman.getCodebasePrincipal is not a function

q.v. Firefox Bug 774585: Rename getCodebasePrincipal to getSimpleCodebasePrincipal since the behavior has semantically changed.
Blocks: 776577
$ TEST_PATH=suite/browser/test/browser_bug413915.js pymake -C ../objdir-sm/ mochitest-browser-chrome

Before:
INFO TEST-START | Shutdown
Browser Chrome Test Summary
        Passed: 0
        Failed: 1
        Todo: 0

After:
INFO TEST-START | Shutdown
Browser Chrome Test Summary
        Passed: 16
        Failed: 0
        Todo: 0
Attachment #653396 - Flags: review?(sgautherie.bz)
Blocks: SmTestFail
Comment on attachment 653396 [details] [diff] [review]
Patch v1.0 Fix test and FeedWriter.js

This ports
http://hg.mozilla.org/mozilla-central/rev/2328647c5d6d
from bug 774585.

Ftr, you attached FeedWriter.js part to bug 776577 (where I expected all this to be fixed) too :-|

r+ (as copying FF code).
NB: I assume you need an additional review for FeedWriter.js.
Attachment #653396 - Flags: review?(sgautherie.bz) → review+
(In reply to Serge Gautherie (:sgautherie) from comment #2)
> Ftr, you attached FeedWriter.js part to bug 776577 (where I expected all
> this to be fixed) too :-|

s/you/Neil/ ;->
Severity: normal → major
Priority: -- → P2
Whiteboard: [perma-orange]
Target Milestone: --- → seamonkey2.14
>> Ftr, you attached FeedWriter.js part to bug 776577 (where I expected all
>> this to be fixed) too :-|
Oops. I'll file a new patch without the changes to FeedWriter.js
>> Ftr, you attached FeedWriter.js part to bug 776577 (where I expected all
>> this to be fixed) too :-|
>Oops. I'll file a new patch without the changes to FeedWriter.js
Attachment #653396 - Attachment is obsolete: true
Attachment #653685 - Flags: review+
Pushed to comm-central
http://hg.mozilla.org/comm-central/rev/ac438606c3a6
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Depends on: 774585
Flags: in-testsuite+
(In reply to Philip Chee from comment #4)
> > Ftr, you attached FeedWriter.js part to bug 776577 (where I expected all
> > this to be fixed) too :-|
> Oops. I'll file a new patch without the changes to FeedWriter.js
Bah, I didn't realise you did this and went to great steps to check in bug 776577 without the FeedWriter.js change only to discover that you hadn't actually patched it after all :-P
You need to log in before you can comment on or make changes to this bug.