Closed
Bug 86238
Opened 23 years ago
Closed 23 years ago
Viewing source of "about:" or some chrome XUL files gives JavaScript console error
Categories
(SeaMonkey :: UI Design, defect)
SeaMonkey
UI Design
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9.6
People
(Reporter: doctor__j, Assigned: timeless)
References
Details
Attachments
(1 file)
(deleted),
patch
|
bzbarsky
:
review+
darin.moz
:
superreview+
|
Details | Diff | Splinter Review |
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Win98; en-US; rv:0.9.1+) Gecko/20010615
BuildID: 2001061309
Reproducible: Always
Steps to Reproduce:
1. Select "Help -> About" in browser.
2. View the source of the page.
3. JavaScript Console error is given:
Error: redeclaration of const kIOServiceProgID
Source File: chrome://communicator/content/utilityOverlay.js
Line: 1
Comment 1•23 years ago
|
||
That's odd. It only seems to be declared once.
Comment 2•23 years ago
|
||
over to XPApps GUI Features.
Assignee: asa → blake
Component: Browser-General → XP Apps: GUI Features
QA Contact: doronr → sairuh
Comment 3•23 years ago
|
||
At a guess.... The scripts that viewSource.xul loads are executed in some
existing context for some reason and we get this error as a result.
No clue why something like that would happen, though....
Updated•23 years ago
|
OS: Windows 98 → All
Hardware: PC → All
Summary: Viewing source of "about:" gives JavaScript console error → Viewing source of "about:" or some chrome XUL files gives JavaScript console error
Comment 6•23 years ago
|
||
it it really an error, or should it be a warning? What does this error prevent?
Does it break some functionality?
It's getting worse.
The error message comes up even when viewing a simple HTML file.
Comment 8•23 years ago
|
||
I saw this (or something similar) with today's build
Comment 9•23 years ago
|
||
*** Bug 102415 has been marked as a duplicate of this bug. ***
Comment 10•23 years ago
|
||
*** Bug 102617 has been marked as a duplicate of this bug. ***
Comment 11•23 years ago
|
||
*** Bug 102801 has been marked as a duplicate of this bug. ***
Comment 12•23 years ago
|
||
*** Bug 103253 has been marked as a duplicate of this bug. ***
Comment 13•23 years ago
|
||
This can be fixed by removing a single line from viewSource.xul. It's this line btw:
<script type="application/x-javascript"
src="chrome://communicator/content/utilityOverlay.js"/>
/HJ
Comment 14•23 years ago
|
||
HJ, that would disable half the menuitems and key commands in view source. Not
a good fix.
Comment 15•23 years ago
|
||
actually, this does fix it, as navigatorOverlay includes utilityOverlay.xul,
which then includes the .js
Assignee | ||
Comment 16•23 years ago
|
||
Assignee | ||
Comment 17•23 years ago
|
||
i figured this out yesterday... sorry for the long delay.
Assignee: blakeross → timeless
Comment 18•23 years ago
|
||
Comment on attachment 52264 [details] [diff] [review]
remove manual inclusion of utilityOverlay (it's chain included via xul overlays)
r=bzbarsky. Can we get this in for 0.9.5?
Attachment #52264 -
Flags: review+
Comment 19•23 years ago
|
||
Comment on attachment 52264 [details] [diff] [review]
remove manual inclusion of utilityOverlay (it's chain included via xul overlays)
sr=darin
Attachment #52264 -
Flags: superreview+
Comment 21•23 years ago
|
||
asa says not stop-ship, -> 0.9.6. sorry for the spam
Target Milestone: mozilla0.9.5 → mozilla0.9.6
Comment 22•23 years ago
|
||
Checked in for timeless; he's out of town and asked me to do it.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Comment 23•23 years ago
|
||
*** Bug 105767 has been marked as a duplicate of this bug. ***
Comment 24•23 years ago
|
||
i don't see the error originally reported in this bug. but i now see the following:
Error: popup has no properties
Source File: chrome://messenger/content/mailNavigatorOverlay.xul
Line: 58
Error: popup has no properties
Source File: chrome://editor/content/editorApplicationOverlay.js
Line: 82
using 2001.11.01.12 bits on linux.
reopening. but, if this new issue should be filed seperately, go ahead and
re-resolve this.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 25•23 years ago
|
||
that's a separate issue. File it on Ben (his overlays)
Status: REOPENED → RESOLVED
Closed: 23 years ago → 23 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 26•23 years ago
|
||
*** Bug 108700 has been marked as a duplicate of this bug. ***
Comment 27•23 years ago
|
||
okay, verifying this one.
filed bug 108745 for the newer js errors i see.
Status: RESOLVED → VERIFIED
Updated•20 years ago
|
Product: Core → Mozilla Application Suite
You need to log in
before you can comment on or make changes to this bug.
Description
•