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

Bug 315230

Summary:

Browser bars occasionally lead to multiple instance warning

Product:

[RT] RAP

Reporter:

Stefan Röck <stefan.roeck>

Component:

RWT

Assignee:

Project Inbox <rap-inbox>

Status:

RESOLVED FIXED

QA Contact:

Severity:

normal

Priority:

Version:

1.3

Target Milestone:

1.4 M1

Hardware:

All

OS:

All

Whiteboard:

sr131

Attachments:

Description	Flags
Sample html page	none
Patch proposal	none

Description Stefan Röck

2010-06-01 10:48:07 EDT

Use Case:
- A static HTML page is used for login where the user can enter username and passwords
- Modern Browsers (Firefox, Chrome) display a yellow bar at the window's top and ask the user to store the password
- This bar triggers some additional requests during application startup (resize events in Application.js)

Sometimes, this leads to the warning "Multiple browser-instances or browser-tabs per session are not supported...".

This can be reproduced with the attached html page (make sure to enter some text in both input fields) with Chrome for about 3 of 10 attempts for me.

Comment 1 Stefan Röck

2010-06-01 10:49:50 EDT

Created attachment 170640 [details]
Sample html page

This simplified login form redirects to the rap demo application.

Comment 2 Stefan Röck

2010-06-01 10:53:28 EDT

In case of error, the following three requests are send:

Request 1:
w1.bounds.width:1280
w1.bounds.height:955
w1.scrollbar.size:17
w1.dpi.x:96
w1.dpi.y:96
w1.colorDepth:32
w1.cursorLocation.x:358
w1.cursorLocation.y:20
uiRoot:w1

Request 2:
w1.bounds.width:1280
w1.bounds.height:927
w1.cursorLocation.x:358
w1.cursorLocation.y:-8
uiRoot:w1

Request 3:
w1.bounds.width:1280
w1.bounds.height:925
w2.mode:maximized
w1.cursorLocation.x:358
w1.cursorLocation.y:-8
uiRoot:w1
requestCounter:0

If everything is OK, the last request is *not* send.
I wonder why the 2 request succeeds as the requestCounter is missing there...

Comment 3 Stefan Röck

2010-06-01 11:17:21 EDT

Created attachment 170643 [details]
Patch proposal

This patch works for me, however, I'm not sure if this is a good solution as there still some RAP internals I haven't fully understood yet :-)

Comment 4 Ivan Furnadjiev

2010-07-01 09:26:55 EDT

Hi Stefan, I had the same issue before with FF and Firebug, but I can't reproduce it anymore. I can't reproduce the issue with your attached file too and CVS HEAD. Tested on FF 3.6.6, Chrome 6 and IE8 - about 20-30 times each. Nevertheless, your patch looks good for me - I had exactly the same patch prepared, when I had this issue but I've never committed it. What are the browser versions used by you?

Comment 5 Stefan Röck

2010-07-01 09:29:13 EDT

Hi Ivan, I could reproduce this with Google Chrome. In contrast to Firefox, Chrome displays the "remember password"-bar slightly delayed which seems to cause the trouble.

Comment 6 Rüdiger Herrmann

2010-07-01 12:24:17 EDT

I could reproduce it wiht Chrome and Firefox. The patch looks good, I would only replace the access to the internal field with a function call.

Comment 7 Ivan Furnadjiev

2010-07-01 12:48:46 EDT

Fixed in CVS HEAD.

Comment 8 Benjamin Muskalla

2010-07-05 04:51:23 EDT

Backported fix to v13_maintance.