58399 - [document.all/document.layers] DMenu: Unknown DOM.

Status: VERIFIED WORKSFORME

(Tech Evangelism Graveyard :: English US, defect, P3)

Description

[Peter Adolphs]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux 2.2.17 i686; en-US; m18) Gecko/20001026
BuildID:    2000102621

If you load this site, a modal dialog appears (title: "Alert", message: "DMenu:
Unknown DOM."). Clicking this dialog away does not help as it reappears
instantly. Only way to get rid of it is a kill command.
A menu that should be shown on the left frame does not appear.

Reproducible: Always

Actual Results:
1. the menu that should be displayed on the left side won't be shown
2. a modal dialog appears saying "DMenu: Unknown DOM.". After clicking
   it away a new window will apear (again and again).

Expected Results:
1. the menu on this site should be displayed correctly
2. if an error occurs, the alert dialog should only be shown once

Additional Comments:
same on Windows...

Comment 1

[robin shaw]

i think this is just bad code. opened it with netscape 4.x and immediately
received 14 javascript error boxes, either "missing or invalid menuid" or
"missing or invalid submenu name" 

Comment 2

[Peter Adolphs]

I tried it with Netscape 4.6 on linux and new Netscape 4.75 on Windows and it works.

Comment 3

[Boris Zbarsky [:bzbarsky]]

Looking at the source, it is full of things like:

if (document.layers) {
  // do stuff
} else if (document.all) {
  // do stuff
} else {
  alert("FunctionName: Unknown DOM/");
}

Where FunctionName depends on the function this is found in.

Since Mozilla supports neither the document.layers nor the document.all
constructs (which are nonstandard), you get all those alerts.  There is
basically one alert for each function call made (57 in all).

This sounds like an evangelism bug; we should at least ask the owner of the site
to fail silently instead of spewing alerts...

Comment 4

[Boris Zbarsky [:bzbarsky]]

I have sent the site maintainers mail explaining the problem.

Comment 5

[Alec Flett]

this is no a DOM viewer bug. reassigning to default owner of the DOM

Comment 6

[Johnny Stenback (:jst)]

Invalid, we're not 100% backwards compatible and that's not gonna change, the
site needs to be fixed.

Comment 7

[Boris Zbarsky [:bzbarsky]]

This should be in evangelism

Comment 8

[Boris Zbarsky [:bzbarsky]]

Over to evangelism; changing summary to reflect problem.

Comment 9

[Peter Adolphs]

It works now....

Comment 10

[Andreas M. "Clarence" Schneider]

Verified wfm.

Comment 11

[Bob Clary [:bc] (inactive)]

All Evangelism Bugs are now in the Product Tech Evangelism. See bug 86997 for
details.