Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 369369 - exitConfirmation not showing up on Chrome browser
Summary: exitConfirmation not showing up on Chrome browser
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P2 major with 1 vote (vote)
Target Milestone: 1.5 M7   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-23 05:30 EST by Leandro Prusch CLA
Modified: 2012-08-24 14:52 EDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Leandro Prusch CLA 2012-01-23 05:30:50 EST
Build Identifier: Version: Indigo Service Release 1 Build id: 20110916-0149

Hello,

My app exitConfirmation message shows up fine in both Firefox 9 and IE 8, but not on Chrome 16 and 17.

It seems something has changed in Chrome that caused this new bug, as this was working on older versions (as reported on the forums in April 2011).

Reproducible: Always

Steps to Reproduce:
Just run the demo app, or any other RAP app, and try reloading the page or closing the tab, or clicking the back button. It should aks for confirmation, but it doesnt.
Comment 1 Ralf Sternberg CLA 2012-01-25 05:05:25 EST
If there's anything we can do about this issue, I think we should do it for 1.5.
Comment 2 Ronald So CLA 2012-03-14 19:07:24 EDT
The Exit Confirmation does not work in the following scenarios too.

IE Update Versions: 9.0.5 on Windows 7
Firefox version 3.6.26 (Mozilla/5.0) on CentOS 6.2 (Linux x86_64)
Google 17 on Windows 7

Is there a plan to fix this problem soon?
Comment 3 Ralf Sternberg CLA 2012-03-15 06:31:27 EDT
Thanks for this update. Because it's a regression, the bug is marked as P2, this means that we're going to fix it for the release.
Comment 4 Ivan Furnadjiev CLA 2012-03-20 05:17:48 EDT
Ronald, IE 9.0.5 on Windows 7 works for me. The problem still exists with Chrome 18.0.1025.113.
Comment 5 Tim Buschtoens CLA 2012-04-04 09:49:57 EDT
Fixed in CVS HEAD.

In chrome, the exit message has to be the returnValue of the listener, not of the event. (Confusing, i know. Not e.returnValue = ...; but return ...;).
Comment 6 Leandro Prusch CLA 2012-04-04 16:39:48 EDT
Thank you
Comment 7 Ivan Furnadjiev CLA 2012-04-05 03:17:34 EDT
There is a problem with the fix in IE. Start Control Demo in IE9 (no exit confirmation message set ). Close the browser - an exit confirmation dialog popup with a message null.
Comment 8 Tim Buschtoens CLA 2012-04-05 04:42:36 EDT
Fixed ie9 issue in CVS HEAD. Apparently ie9 is the only browser to make a difference between null and undefined.
Comment 9 Ronald So CLA 2012-08-24 14:52:34 EDT
The confirmation dialog box now shows up in all three browsers that we are testing: 

IE Update Versions: 9.0.5 on Windows 7
Firefox version EST 10.0.5 (Mozilla/5.0) on CentOS 6.2 (Linux x86_64)
Google Chrome 17 on Windows 7

IE and Chrome show the message that I set in the "getExitConfirmationText()".  Firefox shows its own message but not showing my message.  Is this a known issue?