[Webkit-unassigned] [Bug 29847] [Qt] Add support for automatically creating new windows in QWebView

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=29847





--- Comment #3 from Simon Hausmann <hausmann at webkit.org>  2009-11-24 07:46:10 PST ---
(In reply to comment #2)
> > --- Comments ---
> > 
> > Does this still occur when setting the property
> > QWebSettings::JavascriptCanOpenWindows ?
> 
> I just tested the sample application pasted here, and setting the property has
> no effect (and it ready shouldn't).
> 
> so one who wants to make JavaScript to open new windows by default should set
> the 'JavascriptCanOpenWindows' to 'true' AND reimplement QWebPage's
> createWindow method ...
> 
> it should get mentioned in DOCS in a clear way (if it is not yet there).
> 
> (In reply to comment #1)
> > I disagree that by default we should create new windows. I think this is part
> > of the application integrate that needs to be taken care of by the application
> > developer. Allowing new top-level  windows coming up by default (that can be
> > controlled via JavaScript) sounds like the wrong default to me from a security
> > POV.
> 
> INVALID or WONTFIX then, simon ?

I'd say WONTFIX, but ideally RESOLVED/FIXED with a patch to the documentation
;-)

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.