651750 - Fix emailWizard.js now that document.height and document.width have gone

Status: RESOLVED INVALID

(MailNews Core :: Account Manager, defect)

Description

[Ian Neal]

Attachment: Change to using clientHeight and clientWidth

Bug 585877 removed document.height and document.width, so need to fix emailWizard.js to use something else.

Comment 1

[Mark Banner (:standard8)]

Ok, so I don't know what's going on here.

Without the patch the account wizard is displaying fine, height and width seem to be set appropriately. I've also tried it on aurora and get no warning that these are obsolete.

With the patch I get:

JavaScript strict warning: chrome://messenger/content/accountcreation/emailWizard.js, line 251: reference to undefined property document.body
JavaScript error: chrome://messenger/content/accountcreation/emailWizard.js, line 251: document.body is undefined

My only thought is that there's actually a different set of functionality between chrome and content.

Comment 2

[[e-mail]]

Bug 585877 removed document.height and document.width for HTML documents only.

XUL documents still have height and width, although it appears to be the same as window.innerHeight/Width.