199985 - parent window is always null in nsICookiePromptDialog::CookieDialog

Status: VERIFIED DUPLICATE of bug 173844

(Core Graveyard :: Embedding: APIs, defect)

Description

[Mike Pinkerton (not reading bugmail)]

when implementing nsICookiePromptDialog::CookieDialog() in camino (and for
cocoaEmbed), I noticed that the parent window param is always null. This means
we can't know what window the cookie was destined for so the dialog cannot be a
sheet.

It appears to be hardcoded to nsnull at:
 http://lxr.mozilla.org/seamonkey/source/extensions/cookie/nsCookies.cpp#1776

This is a pretty major regression in functionality from the branch and needs to
be addressed before this interface is frozen. we need this not only for camino,
but for any other mac embed clients. Application modal dialogs are bad.

Comment 1

[Michiel van Leeuwen (email: mvl+moz@)]

Related to bug 173844.

Comment 2

[Conrad Carlen (not reading bugmail)]

Yes, I think so. Here,
http://lxr.mozilla.org/seamonkey/source/extensions/cookie/nsCookiePermission.cpp#75,
the method calling CookieDialog is being passed a parent window. Why isn't it
being used?

Comment 3

[Mike Pinkerton (not reading bugmail)]

argh, dupe.

*** This bug has been marked as a duplicate of 173844 ***

Comment 4

[Charles Rosendahl]

v