Closed Bug 99504 Opened 24 years ago Closed 23 years ago

javascript strict warning in addressbook.js, setWebShellWindow changed to setDocShellWindow a long time ago.

Categories

(SeaMonkey :: MailNews: Address Book & Contacts, defect)

Product:
SeaMonkey
Component:
MailNews: Address Book & Contacts
Platform:
x86
Windows 2000
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED DUPLICATE of bug 58696
Milestone:
mozilla0.9.9

People

(Reporter: sspitzer, Assigned: racham)

References

(
URL
)

Details

JavaScript strict warning: chrome://messenger/content/addressbook/addressbook.js line 66: reference to unde fined property top.addressbook.setWebShellWindow back in may 2000, setWebShellWindow changed to setDocShellWindow and the js was never fixed. looking at nsAddressBook.cpp, the thing that would be broken would be printing, but according to esther, printing works in today's build.
I don't think any of that code is used for printing. once I confirm that, I'll clean up the .idl, .cpp and .js and remove it.
Status: NEW → ASSIGNED
reassigning to racham. Could this be related to the bug we have where clicking on a url in the card view opens the link in the addressbook (thereby making the ab unusable) rather than a new window?
Assignee: sspitzer → racham
Status: ASSIGNED → NEW
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.7
Target Milestone: mozilla0.9.7 → mozilla0.9.9
*** This bug has been marked as a duplicate of 58696 ***
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
Summary: js strict warning in addressbook.js, setWebShellWindow changed to setDocShellWindow a long time ago. → javascript strict warning in addressbook.js, setWebShellWindow changed to setDocShellWindow a long time ago.
verified; this warning is covered in the duplicate bug
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.