961 - [BLOCKER] Document dimensions not exposed to JS yet

Status: VERIFIED FIXED

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

Description

[Angus Davis]

document.width
window.innerWidth
window.outerWidth

... not implemented yet. I need to get this value for some XPFE stuff I'm
working on. I do not need set yet.

Comment 1

[joki (gone)]

window.innerWidth should work now.

window.outerWidth waiting on api's into widget stuff.

we'll need to decide what to do with document.width

Comment 2

[leger]

Setting all current Open/Normal to M4.

Comment 3

[Paul MacQuiddy]

per leger, assigning QA contacts to all open bugs without QA contacts according
to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser

Comment 4

[gerardok]

QA contact re-assigned according to the product areas we're currently working
on.

Comment 5

[gerardok]

Component updated to DOM Level 0.

Comment 6

[joki (gone)]

Not critical for M4 at this point, moving to M5

Comment 7

[joki (gone)]

Moving out to M7

Comment 8

[joki (gone)]

I believe the window parts of this now work.  Just the document dimensions
don't.  Those are less important and less well defined.  Chaning summary and
pushing farther down.

Comment 9

[Christian Wenz]

Attachment: test case for window.[inner|outer]Width and document.width

Comment 10

[Christian Wenz]

Overview Description: document.width is not implemented yet

Steps to reproduce: load the attachment. Note that window.innerWidth,
window.outerWidth and document.width are read.

Actual Results: You get values for window.innerWidth and window.outerWidth, and
document.width = undefined

Expected Results: document.width = [any number]

Build Date & Platform: M7 and 1999070917 build (Win32)

Additional information: the innerWidth/outerWidth implementation seems to be
buggy, I'll file an extra bug report for that.

Comment 11

[Christian Wenz]

jfyi: the window.[inner|outer]Width bug is #9606. I did  not send a dependency
to this bug, as bug #961 is now limited to document.width

Comment 12

[Steve Lamm]

I need document.height too.

Comment 13

[Steve Lamm]

Upgrading to blocker.

Comment 14

[brendan-obsolete]

Joki said I could have this one; could use advice from troy or kipp for source
of document height and width in pixels.

/be

Comment 15

[Steve Lamm]

I see that the document height and width is computed in nsScrollFrame::Reflow()
and nsGfxScrollFrame::Reflow(). The values are not saved into any member
variables. I am not sure how you would get these values to
nsHTMLDocument::GetWidth() and nsHTMLDocument::GetHeight().

Comment 16

[vidur (gone)]

Fix checked in on 10/6/1999. Reviewed by Troy.

Comment 17

[Prashant Desale]

Tested with 10-09-09. Working fine. Marking verified.