friscofrog Reporter
	Description • 20 years ago

User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X; de-de) AppleWebKit/125.5.5 (KHTML, like Gecko) Safari/125.11
Build Identifier: Camino 2005011008

On the homepage http://www.sfgate.com there is a small image in the right column
with the caption "Day in Pictures" Clicking on the text or the picture launches
a smaller blank popup window and a new regular sized window which contains the
desired content. This problem was first noted by me on the nightlies released on
January 11 and all thereafter. It does not exist with the latest version of
Mozilla (1.8a) or Firefox or versions of Camino prior to Jan 11. The desired
action on clicking on the text or image is to launch the popup with content
which can be navigated within that window.

Reproducible: Always

Steps to Reproduce:
1.Launch any Camino nightly build after Jan 11
2.Go to http://www.sfgate.com
3.click on image or text labelled "Day in Pictures"

Actual Results:  
A blank popup window is launched and then is covered with a regular-sized new
window with the desired content

Expected Results:  
Content should be present in popup window. It should be navigatable. No new
larger window should be created.

	Smokey Ardisson (offline for a while; not following bugs - do not email)
	Comment 1 • 20 years ago

Here's the relevant bit of HTML:

<!-- day in pictures -->




<A HREF="/cgi-bin/object/dip?f=/g/archive/2005/01/18/dip.DTL" TARGET="photos"
onClick="window.open('','photos','directories=no,height=500,width=580,location=no,resizable=yes,screenx=15,screeny=15,toolbar=no,opener=orig')">
<img src="/templates/types/homepage/graphics/sub_dip2.gif" width="118"
height="15" border="0" vspace="3" hspace="0"></a><br>

  
<table border="0" cellpadding="0" cellspacing="0" width="143"><tr><td valign="top">
<A HREF="/cgi-bin/object/dip?f=/g/archive/2005/01/18/dip.DTL" TARGET="photos"
onClick="window.open('','photos','directories=no,height=500,width=580,location=no,resizable=yes,screenx=15,screeny=15,toolbar=no,opener=orig')"><IMG
SRC="/templates/types/object/graphics/dip_t.gif" HEIGHT="64" WIDTH="64"
HSPACE="0" VSPACE="0" border="1"></A></td>
<td width="2">&nbsp;</td>
<td valign="middle" class="text1sm">
<!--DIP-->


Animal tales and human prayers. 
<A HREF="/cgi-bin/object/dip?f=/g/archive/2005/01/18/dip.DTL" TARGET="photos"
onClick="window.open('','photos','directories=no,height=500,width=580,location=no,resizable=yes,screenx=15,screeny=15,toolbar=no,opener=orig')">DIP</A>!<!--DIP--></td></tr></table>



To my ignorant eye, it looks like they're using both a TARGET and an
onClick="window.open(, which seems iffy to my ignorant brain.

From the rest of the initial report, it seems to me that something in the way
Gecko handled those changed on Jan 11 on the trunk.

FriscoFrog, did you test in Mozilla 1.8a6 (which was released on Jan 12 and may
or may not have that code depending on the freeze point) and/or a Firefox
nightly after Jan 11?

	Mike Pinkerton (not reading bugmail)
	Comment 2 • 20 years ago

jst, did any popup blocking stuff change last week that camino might have missed?

	Jasper
	Comment 3 • 20 years ago

Confirming :)

	Johnny Stenback (:jst)
	Comment 4 • 20 years ago

Mike, I bet this changed when security bug 273699 was fixed, that bug was fixed
by bug 103638.

	Jasper
	Comment 5 • 20 years ago

*** Bug 279201 has been marked as a duplicate of this bug. ***

	Jasper
	Comment 6 • 20 years ago

This regressed between 11 and 12 januarie.
I looked and it seems that Johnny Stenback is right in that those two bugs seems
to be the cause when looking at the checkins.

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 7 • 20 years ago

So the problem is clearly that the target is not finding the just-opened window.
 Is the window being found but failing the security checks added in bug 103638?
 Or is it just not being found at all?

Also, does Camino use nsWebBrowser and nsDocShellTreeOwner, or does it use the
appshell goop in xpfe/?  If the former, does it also have its own
nsIDocShellTreeOwner implementation?

	OstGote!
	Comment 8 • 20 years ago

Same problem for Mozilla trunk: Bug 279495.

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 9 • 20 years ago

No, that's a different problem.  I can reproduce that bug with Mozilla trunk,
but this one worksforme.  So this one is likely to be embedding-specific or
Camino-specific.

	Wevah (Nate Weaver)
	Comment 10 • 20 years ago

I've noticed some issues with window.open() when it's called with an empty
location part (e.g., window.open('', 'foo');). I wonder if this is related?

	Wevah (Nate Weaver)
	Comment 11 • 20 years ago

Nevermind, (I think) my issue is unrelated (though this bug did help me narrow
some of my own problems down).

	friscofrog Reporter
	Comment 12 • 20 years ago

 Disabling javascript eliminated the blank popup window leaving only the larger
new page. The desired result is to have the content in the smaller popup window.
Oh well, at least the developers are aware of that bug and are working on a fix
(I hope.) Disabling javascript also "fixed" bug 280390. Thanks to Wevah for
finding the link.

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 13 • 20 years ago

Mike, Simon, could you answer my question from comment 7?  I'm trying to sort
out why this could happen by code inspection, but it'd help to know which code
to inspect...

	Simon Fraser [no longer active]
	Comment 14 • 20 years ago

Would the fix for bug 279495 have fixed this?

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 15 • 20 years ago

Possible, but unlikely -- see comment 9...

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 16 • 20 years ago

OK, I think this should fix it, assuming that Camino uses nsWebBrowser.  I did
test this in TestGtkEmbed (which was also showing this bug).

The basic issue is that the window-finding code in nsDocShellTreeOwner was just
wrong -- it skipped looking in other windows if its requestore was its child
docshell, which is exactly the case when you _do_ want to look in other
windows...

This patch makes the code be pretty similar to that in
nsChrome/ContentTreeOwner, with the modifications I added in bug 279495.

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 17 • 20 years ago

Since it's a general embedding problem, moving to core....

	Johnny Stenback (:jst)
	Comment 18 • 20 years ago

Comment on attachment 173190 [details] [diff] [review]
Proposed patch

sr=jst

	Simon Fraser [no longer active]
	Comment 19 • 20 years ago

This patch fixes Camino -- thanks Boris! (I see another crash when closing the
DIP window on SFGate: will file separately.)

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 20 • 20 years ago

Fixed.

	Christopher Aillon (sabbatical, not receiving bugmail)
	Comment 21 • 20 years ago

Comment on attachment 173190 [details] [diff] [review]
Proposed patch

a=caillon (on behalf of drivers) for 1.0.1 and 1.7.6

	Johnny Stenback (:jst)
	Comment 22 • 20 years ago

Fixed on 1.0.1 and 1.7 branches.

	Jay Patel [:jay]
	Comment 23 • 20 years ago

Verified Fixed on Win32 with latest Aviary 1.0.1 and Mozilla 1.7.6 builds.

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 24 • 17 years ago

Test for this got added in bug 408052.