Closed Bug 784041 Opened 12 years ago Closed 12 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)

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
Attached patch Patch v1.0 Fix test and FeedWriter.js (obsolete) (deleted) — Splinter Review
$ 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+
Status: ASSIGNED → RESOLVED
Closed: 12 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.

Attachment

General

Creator:
Created:
Updated:
Size: