83107 - Javascript table sort broken

Status: VERIFIED FIXED

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

Description

[Rick DeBay]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:0.9+) Gecko/20010523
BuildID:    2001052304

Javascript table sort demo works in IE, and in DOM 1 compliant brosers,
but doesn't work on this build.  I think it was working on 0.8.1.

Reproducible: Always
Steps to Reproduce:
1.Go to http://webfx.eae.net/dhtml/tablesort/demo.html
2.Click on the column head
3.Rows don't sort, and column title is incorrect

Actual Results:  Column title changed to variable name.

Expected Results:  Rows should sort, and column title should change to show sort
order.

Comment 1

[Rick DeBay]

Also fails at http://mkruse.netexpress.net/javascript/sorttable/
IE and NS 4.77 work.

Comment 2

[Boris Zbarsky [:bzbarsky]]

Rick, are there any errors in the JS console?

Comment 3

[Phil Schwartau]

Using Mozilla nightly binary 2001052710 WinNT. Boris has the right idea:
in the JS Console, you get this error every time you try to do a table sort:

  Error: uncaught exception: Permission denied to create wrapper for object


Reassigning to Security:General. cc'ing jst, jband in case this is more
of an XPConnected DOM issue than a Security one - 

Also happens with the Linux nightly, same date. Changing OS --> All.

Comment 4

[John Bandhauer]

jst: The problem here is that nsTableCellCollection does not expose 
nsIClassInfo. Is should, right?

Comment 5

[Johnny Stenback (:jst)]

Yup, that's the problem, thanks for tracking this down, John.

Comment 6

[Boris Zbarsky [:bzbarsky]]

The patch in bug 85560 fixes this bug as well by adding classinfo to an ancestor
class of nsTableCellCollection.

Comment 7

[Boris Zbarsky [:bzbarsky]]

Fix checked in (reviews and so forth are in bug 85560)

Comment 8

[Boris Zbarsky [:bzbarsky]]

Oops. Forgot to mark fixed.

Comment 9

[ckritzer (gone)]

Marking VERIFIED FIXED on:
-MacOS91 2001-07-10-03-0.9.2
-LinRH62 2001-07-10-08-0.9.2
-Win98SE 2001-07-10-05-0.9.2