Closed Bug 405105 Opened 12 years ago Closed 11 years ago

Error: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIObserverService.removeObserver] when closing the pageInfo window.

Categories

(SeaMonkey :: Page Info, defect)

defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED
seamonkey2.0a1

People

(Reporter: philip.chee, Assigned: sgautherie)

References

Details

(Keywords: regression)

Attachments

(2 files)

Error: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIObserverService.removeObserver]

Steps to reproduce:
1. Navigate to about:blank.
2. Open the pageInfo window.
3. Close the pageInfo window.

Reason: The observer doesn't get added on pages with no images. So there might not be an observer to remove on exit.
[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9pre) Gecko/2008060507 Minefield/3.0pre] (nightly) (W2Ksp4)

No bug, as the Media tab is not shown in this case.


[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9pre) Gecko/2008060802 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4)

{{
Error: uncaught exception: [Exception... "Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIObserverService.removeObserver]"  nsresult: "0x80004005 (NS_ERROR_FAILURE)"  location: "JS frame :: chrome://navigator/content/pageinfo/pageInfo.js :: onUnloadPageInfo :: line 348"  data: no]
}}
<http://mxr.mozilla.org/seamonkey/find?text=&kind=text&string=%2Fpageinfo%2Fpageinfo.>

Do we want to simply use |if (gImageView.data.length == 1) {| ?
Or to port the |mediaTab.hidden| behavior ?

Maybe even port bug 388504 ?
(In reply to comment #2)
> Do we want to simply use |if (gImageView.data.length == 1) {| ?
Sounds good to me.
[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9pre) Gecko/2008060802 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4)

*Added test.
*Added comments.
*Trivial rewrite.
*Added blank lines.
Assignee: db48x → sgautherie.bz
Status: NEW → ASSIGNED
Attachment #324228 - Flags: superreview?(neil)
Attachment #324228 - Flags: review?(neil)
Keywords: regression
Target Milestone: --- → seamonkey2.0alpha
> *Added blank lines.

Um any particular reason for the extra blank lines? Just asking.
> *Added test.
I don't see anything added to the unit tests???
Comment on attachment 324228 [details] [diff] [review]
(Av1) <pageInfo.js>
[Checkin: Comment 15]

We've been trying to keep the suite and browser versions roughly in sync so over to Daniel to see if he wants to port the other changes.
Attachment #324228 - Flags: review?(neil) → review?(db48x)
(In reply to comment #5)
> Um any particular reason for the extra blank lines? Just asking.

I find it easier to notice the |return|s...

(In reply to comment #6)
> I don't see anything added to the unit tests???

Bad wording: I meant the |if|.
(In reply to comment #8)
>> Um any particular reason for the extra blank lines? Just asking.
> I find it easier to notice the |return|s...

General rule for mozilla.org patch submissions: don't alter whitespace in areas you aren't touching (unless it's a whitespace cleanup patch of course). This messes up CVS Blame. I've no idea if this still holds true for Hg/mozilla-central.
Comment on attachment 324228 [details] [diff] [review]
(Av1) <pageInfo.js>
[Checkin: Comment 15]

yes, we need to keep them as close to in sync as possible.
Attachment #324228 - Flags: review?(db48x) → review+
Comment on attachment 324228 [details] [diff] [review]
(Av1) <pageInfo.js>
[Checkin: Comment 15]

by which my understanding is that he'd like a patch to port the other changes (e.g. removal of the "os" variable) to browser.
Attachment #324228 - Flags: superreview?(neil) → superreview+
Port Av1 patch to FF, per comment 11.
Attachment #324791 - Flags: review?
Attachment #324791 - Flags: review? → review?(mano)
Keywords: checkin-needed
Whiteboard: [c-n: Av1 // Leave opened]
Attachment #324791 - Flags: review?(mano) → review+
Whiteboard: [c-n: Av1 // Leave opened] → [c-n: Av1, Bv1-FF]
Comment on attachment 324791 [details] [diff] [review]
(Bv1-FF) <pageInfo.js>
[Checkin: Comment 14]

"approval1.9.0.1=?":
Trivial sync with </suite/>, "no" code change.
Attachment #324791 - Flags: approval1.9.0.1?
Attachment #324791 - Flags: approval1.9.0.1? → approval1.9.0.2?
Attachment #324791 - Flags: approval1.9.0.2?
Comment on attachment 324791 [details] [diff] [review]
(Bv1-FF) <pageInfo.js>
[Checkin: Comment 14]

http://hg.mozilla.org/index.cgi/mozilla-central/rev/1237fb220e76
Attachment #324791 - Attachment description: (Bv1-FF) <pageInfo.js> → (Bv1-FF) <pageInfo.js> (checked in)
Whiteboard: [c-n: Av1, Bv1-FF]
Av1:

Checking in suite/browser/pageinfo/pageInfo.js;
/cvsroot/mozilla/suite/browser/pageinfo/pageInfo.js,v  <--  pageInfo.js
new revision: 1.13; previous revision: 1.12
done
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Attachment #324228 - Attachment description: (Av1) <pageInfo.js> → (Av1) <pageInfo.js> [Checkin: Comment 15]
Attachment #324791 - Attachment description: (Bv1-FF) <pageInfo.js> (checked in) → (Bv1-FF) <pageInfo.js> [Checkin: Comment 14]
[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.0.2pre) Gecko/2008071206 SeaMonkey/2.0a1pre] (SEA-WIN32-TBOX-trunk) (W2Ksp4)

V.Fixed

*****

(In reply to comment #2)
> Maybe even port bug 388504 ?

I filed bug 444917.
Status: RESOLVED → VERIFIED
QA Contact: page-info
You need to log in before you can comment on or make changes to this bug.