Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 315230 - Browser bars occasionally lead to multiple instance warning
Summary: Browser bars occasionally lead to multiple instance warning
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: 1.3   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.4 M1   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard: sr131
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-01 10:48 EDT by Stefan Röck CLA
Modified: 2010-07-05 04:51 EDT (History)
0 users

See Also:

Attachments
Sample html page (851 bytes, text/plain)
2010-06-01 10:49 EDT, Stefan Röck CLA 		no flags Details
Patch proposal (1.02 KB, patch)
2010-06-01 11:17 EDT, Stefan Röck CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Röck CLA 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 CLA 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 CLA 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 CLA 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 CLA 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 CLA 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 CLA 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 CLA 2010-07-01 12:48:46 EDT 
Fixed in CVS HEAD.
Comment 8 Benjamin Muskalla CLA 2010-07-05 04:51:23 EDT 
Backported fix to v13_maintance.