92373 - WRMB: commercialsource.com - page loads, then vanishes

Status: VERIFIED WORKSFORME

(Core :: DOM: Core & HTML, defect, P3)

Description

[Chris Waterson]

It looks like there's an IFRAME or that's ``accidentally'' reloading in the main 
content area. I found this page from looking at bugscape bug number 7514.

Comment 1

[Chris Waterson]

Attachment: somewhat minimized test case

Comment 2

[Chris Waterson]

The relevant part of the test case here is:

  function MM_setTextOfLayer(objName,x,newText) { //v3.0
    if ((obj=MM_findObj(objName))!=null) with (obj)
      if (navigator.appName=='Netscape') {
        document.write(unescape(newText)); document.close();
      }
      else innerHTML = unescape(newText);
  }
The problem here is that it's testing the |appName| property, and picking up
``Mozilla'' instead of ``Netscape''. (If I hack the test case to look for
``Mozilla'' as the appname, it works.)

I'm not familiar enough with the |innertHTML| property to know if the <div>
element (which is what's returned from |MM_findObject()|) should have this
property. (I suspect not, since I couldn't find the string in either the DOM1 or
DOM2 specs.) If it _is_ supposed to have this property, then perhaps we have a
scoping problem with the |with| keyword?

Reassigning to evangelism for now, cc'ing brendan and jst who may be able to
comment on the DOM and JS better than I. (Should we just implement |innertHTML|
on <div> elements?)

Comment 3

[Chris Waterson]

Make that ``innerHTML'', as opposed to inner_t_HTML. ;-)

Comment 4

[Bob Clary [:bc] (inactive)]

innerHTML is an IE extension that was added to Mozilla in response to very high
interested by web developers who are much more confortable with creating and
assigning strings of HTML rather than using the raw DOM.

Comment 5

[Chris Waterson]

Okay, in that case, I'll kick this over to jst to triage. jst, is it reasonable
to be setting |innerHTML| on a <div> element?

Comment 6

[Bob Clary [:bc] (inactive)]

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

Comment 7

[rubydoo123]

seems like this is not an evang bug, moving it out of evang product group

Comment 8

[rubydoo123]

since this is not an evang bug, readding WRMB to summary

Comment 9

[timeless]

i think this is an evang candidate. iirc you aren't supposed to call 
document.close()

Comment 10

[Zach Lipton [:zach]]

since this is not an evang bug, setting qa to default

Comment 11

[Nisheeth Ranjan]

The page no longer vanishes.  It now loads up but displays the "Member
Directory", Commercial Properties", etc. text headings stacked one underneath
the other on the top left of the page rather than showing them in the red
backgrounded banner across the page.  This is more a layout problem now and Beth
tells me that that is being tracked in another bug.  Marking this one as worksforme.

Comment 12

[Sivakiran Tummala]

the page does not render properly. i am investigating this bug. the menu items
do not display properly looks more like a evangalism to me.

Comment 13

[Marek Z. Jeziorek]

removing topembed

Comment 14

[Fabian Guisset]

Page looks the same to me in Mozilla0.9.7 and in IE5 on win2k. Worksforme?

Comment 15

[Johnny Stenback (:jst)]

WORKSFORME.

Comment 16

[Sivakiran Tummala]

verified worksforme build 2002-03-18-09