Bug 3837 - Page nearly loads but then returned to default page by javascript issue
Summary: Page nearly loads but then returned to default page by javascript issue
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 412
Hardware: Mac OS X 10.4
: P2 Major
Assignee: Anders Carlsson
URL: http://www.realenz.co.nz/misc/menumap...
Depends on:
Reported: 2005-07-03 04:32 PDT by George McKinlay
Modified: 2005-08-09 15:20 PDT (History)
0 users

See Also:

testcase reduction (212 bytes, text/html)
2005-07-03 15:52 PDT, David Storey
no flags Details
Don't evaluate scripts follwing framesets (2.39 KB, patch)
2005-07-13 09:54 PDT, Anders Carlsson
darin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description George McKinlay 2005-07-03 04:32:08 PDT
CF site seems to generate world leaks... sends one back to the default site. Not sure but strikes me as a 
javascipt issue. Once kicked back to http://www.realenz.co.nz/misc/menuhome_frames.cfm you can 
repeat the isue by selecting another region on the map
Comment 1 Joost de Valk (AlthA) 2005-07-03 08:26:42 PDT
Confirmed. Would be good to have a testcase for this one, since i don't get what's happening.
Comment 2 David Storey 2005-07-03 15:52:45 PDT
Created attachment 2778 [details]
testcase reduction

If the testcase works then the page will remain blank, with a title saying "If
page doesn't redirect then correct" (tested in IE/Mac, Camino and FF),
otherwise it will return to webkit project homepage (current behaviour in
Safari).  This behaves slightly different to the real page, in the IE/Mac now
works.	In the real page it crashes.

If the frameset is remove from the code then it redirects in all browsers
Comment 3 Anders Carlsson 2005-07-13 09:54:50 PDT
Created attachment 2941 [details]
Don't evaluate scripts follwing framesets

It looks like neither Gecko nor WinIE evaluates or even loads scripts following
framesets when loading the document. Here's a patch that makes Safari do the
Comment 4 Darin Adler 2005-08-04 11:23:14 PDT
Comment on attachment 2941 [details]
Don't evaluate scripts follwing framesets

Looks good, r=me.
Comment 5 Justin Garcia 2005-08-09 15:20:39 PDT
Landed this