|Summary:||REGRESSION: link targeting a frame in another window does not work|
|Product:||WebKit||Reporter:||Giuliano Gavazzi <jmail-bugs.webkit.org>|
|Component:||Page Loading||Assignee:||Maxime BRITTO <mbritto>|
|Severity:||Normal||CC:||almaslegacy, ddkilzer, dstorey, mbritto, sroret|
|Priority:||P1||Keywords:||InRadar, NeedsReduction, Regression|
|OS:||OS X 10.4|
Description Giuliano Gavazzi 2006-12-05 08:53:30 PST
a link with a target attribute that targets a frame panel in another window does not perform any action. See for example: http://www.elfo.org/ it will open a pop-up window (so please do not disable pop-up windows!). The first link in the popup window (and most of the others) are of the form: <a href="../somepath..." target="testo">some text</a> where "testo" is the name of the central pane of the main page frame. In the current version of Safari this of course works. I does not in the nightly build of webkit (downloaded today). Perhaps this is connected with the bug http://bugs.webkit.org/show_bug.cgi?id=10828 ?
Comment 1 David Kilzer (:ddkilzer) 2007-06-17 08:00:20 PDT
*** Bug 14194 has been marked as a duplicate of this bug. ***
Comment 2 David Kilzer (:ddkilzer) 2007-06-17 09:24:53 PDT
This works for me with Safari 3.0 (522.11) and a local debug build of WebKit r23546 on Mac OS X 10.4.9 (8P135). Giuliano, do you still see this issue?
Comment 3 Giuliano Gavazzi 2007-06-18 01:29:04 PDT
(In reply to comment #2) > This works for me with Safari 3.0 (522.11) and a local debug build of WebKit > r23546 on Mac OS X 10.4.9 (8P135). Giuliano, do you still see this issue? > David, in webkit r23558 I still see it on http://www.elfo.org/ I have not tried Safari 3.0 yet. Note that I am on PPC, if that makes any difference.
Comment 4 David Kilzer (:ddkilzer) 2007-06-18 06:08:03 PDT
Comment 5 David Kilzer (:ddkilzer) 2007-06-18 06:41:04 PDT
* REGRESSION This is a regression since I managed to test the original Safari 2.0.4 (419.3) with the original WebKit.framework, and this works in that version. Note that this works in Firefox 18.104.22.168 on Mac OS X 10.4.9 (8P135). In Opera 9.21, the pop-up window never appears even with "Pop-ups" preference set to "Open all pop-ups". It would be interesting to know what MSIE 6/7 does.
Comment 6 Giuliano Gavazzi 2007-06-18 07:27:56 PDT
Comment 7 Giuliano Gavazzi 2007-06-20 05:11:40 PDT
(In reply to comment #5) > It would be interesting to know what MSIE 6/7 does. looking at the logs I guess that it works fine (the referrer field in the logs is compatible with this). Although the page corresponding to the popup can also be opened in the main frame, I can distinguish this case in the log. Looking at the logs I noticed that in WebKit the links targeting the main frame simply do not perform any http request to the server. No wonder they do not work..
Comment 9 Maxime BRITTO 2007-06-25 05:58:15 PDT
I tried to copy the files from the website to my HD and then it was working. The only thing I modified was the JS call to open the popup : before : pippo=window.open("/bacheca/bacheca.html","popup",opt); after : pippo=window.open("../bacheca/bacheca.html","popup",opt); I don't really understand how the site can work locally but not remotely...
Comment 10 Brady Eidson 2007-07-24 16:13:13 PDT
"I don't really understand how the site can work locally but not remotely..." Local files are allowed to access any resource they wish - both local and remote. There's little security checking on them. When these pages are on a live, remote server, they are held to a higher security standard.
Comment 11 Maxime BRITTO 2007-07-25 01:47:19 PDT
I have the fix, I'm working now on the test case.
Comment 12 Maxime BRITTO 2007-07-25 05:41:22 PDT
Created attachment 15680 [details] proposed fix Here is the fix, there is no attached test case because I'm not very good at these but I'm sure about the fix. If the test case is essential I can give it another try but I'd prefer to wait for the reviewer opinion.
Comment 13 Maxime BRITTO 2007-07-25 08:26:00 PDT
Created attachment 15682 [details] test case (needs changes in DRT to work) This is the test case mitz, bdash and I made but for now it is useless. It will be usefull if the DRT is changed to handle differently the group name of the frames.
Comment 14 Darin Adler 2007-07-25 08:34:53 PDT
Comment on attachment 15680 [details] proposed fix r=me
Comment 15 mitz 2007-07-25 08:54:38 PDT
Created attachment 15683 [details] Allow testing of cross-page frame lookup Does not regress any existing test.
Comment 16 Darin Adler 2007-07-25 08:57:56 PDT
Please do continue to work on a test case for this. I landed the fix, but we really need a regression test too. Committed revision 24623.
Comment 17 Darin Adler 2007-07-25 09:53:03 PDT
Comment on attachment 15683 [details] Allow testing of cross-page frame lookup r=me
Comment 18 Giuliano Gavazzi 2007-08-11 00:32:38 PDT
(In reply to comment #12) > Created an attachment (id=15680)  > proposed fix > > Here is the fix, there is no attached test case because I'm not very good at > these but I'm sure about the fix. > If the test case is essential I can give it another try but I'd prefer to wait > for the reviewer opinion. > thank you for the fix. I have finally tested r25000 with the original site (although it is a non stationary site...) and it works. Will make a couple of static test pages to put on the site and keep an eye of the next releases. Giuliano