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)
SeaMonkey
Feed Discovery and Preview
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)
(deleted),
patch
|
philip.chee
:
review+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•12 years ago
|
||
$ 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)
Assignee | ||
Updated•12 years ago
|
Blocks: SmTestFail
Comment 2•12 years ago
|
||
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+
Comment 3•12 years ago
|
||
(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
Assignee | ||
Comment 4•12 years ago
|
||
>> 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
Assignee | ||
Comment 5•12 years ago
|
||
>> 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+
Assignee | ||
Comment 6•12 years ago
|
||
Pushed to comm-central
http://hg.mozilla.org/comm-central/rev/ac438606c3a6
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 7•12 years ago
|
||
(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.
Description
•