Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 340193

Summary: org.eclipse.swt.browser.IE sending out double LocationEvents when loading page with frames
Product: [Eclipse Project] Platform Reporter: Ian Tewksbury <itewksbu>
Component: SWTAssignee: Platform-SWT-Inbox <platform-swt-inbox>
Status: CLOSED WONTFIX QA Contact: Grant Gayed <grant_gayed>
Severity: normal    
Priority: P3 CC: eclipse.felipe
Version: 3.6.2Keywords: triaged
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard: stalebug

Description Ian Tewksbury CLA 2011-03-16 13:13:36 EDT
In an adopter product we have noticed that org.eclipse.swt.browser.IE is sending out two events for the page included in the first frame of a page with two frames.

So if this was my page:
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Frameset//EN" "http://www.w3.org/TR/html4/frameset.dtd">
<html>
<head>
<title>index</title>
<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1">
</head>
<frameset rows="50%,50%">
	<frame src="newpage1.html">
	<frame src="newpage2.html">
	<noframes>
		<body>

		</body>
	</noframes>
</frameset>
</html>

I am getting two notifications for the 'newpage1.html' and one notification for the 'newpage2.html'. I would expect to only get one notification for each.
Comment 1 Grant Gayed CLA 2011-03-24 17:09:02 EDT
I don't see the extra events, tried with IE6 and IE8.  I took your example, set its first frame to point at http://www.asdf.com and its second frame to point at http://www.qwerty.com, and the received location events were:

changing: file://C:/workspace-e4/Grant/browser/test.html
changing: http://www.asdf.com/
changing: http://www.qwerty.com/
changed: http://www.asdf.com/
changed: http://www.qwerty.com/
changed: file://C:/workspace-e4/Grant/browser/test.html

asdf.com and qwerty.com were used because they're pretty basic sites (no ajax, etc.).  Perhaps there's something particular about the page you're showing in your first frame that's causing extra events to be received for it?  If you switch the pages between the two frames do you then get an extra event for the other page instead?
Comment 2 Ian Tewksbury CLA 2011-03-24 17:16:45 EDT
(In reply to comment #1)
> I don't see the extra events, tried with IE6 and IE8.  I took your example, set
> its first frame to point at http://www.asdf.com and its second frame to point
> at http://www.qwerty.com, and the received location events were:
> 
> changing: file://C:/workspace-e4/Grant/browser/test.html
> changing: http://www.asdf.com/
> changing: http://www.qwerty.com/
> changed: http://www.asdf.com/
> changed: http://www.qwerty.com/
> changed: file://C:/workspace-e4/Grant/browser/test.html
> 
hrmmm, now you have me thinking that we are getting a changing and changed event but not discriminating.  I will have to check.

> asdf.com and qwerty.com were used because they're pretty basic sites (no ajax,
> etc.).  Perhaps there's something particular about the page you're showing in
> your first frame that's causing extra events to be received for it?  If you
> switch the pages between the two frames do you then get an extra event for the
> other page instead?

My testing is on the exact page I pasted there where newpage1.html and newpage2.html are blank.
Comment 3 Ian Tewksbury CLA 2011-03-24 17:40:55 EDT
I checked and I am seeing two different OleEvents coming into the OleListener on line 377 of org.eclipse.swt.browser.IE.  Both events get into the BeforeNavigate2 case and send out two separate LocationEvents.

I noticed that I don't see this the first time i create a frame page and view it, but only after I close the viewer and open it again.

Could there be something we are doing that is causing this?
Comment 4 Leo Ufimtsev CLA 2017-08-03 12:27:33 EDT
This is a one-off bulk update. (The last one in the triage migration).

Moving bugs from swt-triaged@eclipse to platform-swt-inbox@eclipse.org and adding "triaged" keyword as per new triage process:
https://wiki.eclipse.org/SWT/Devel/Triage

See Bug 518478 for details.

Tag for notification/mail filters:
@TriageBulkUpdate
Comment 5 Eclipse Genie CLA 2020-03-23 16:55:23 EDT
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're closing this bug.

If you have further information on the current state of the bug, please add it and reopen this bug. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.