Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 301236 - Background-image with rounded borders kills session in Chrome
Summary: Background-image with rounded borders kills session in Chrome
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 M5   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 332850 333858 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-01-29 06:12 EST by Tim Buschtoens CLA
Modified: 2011-01-10 08:04 EST (History)
3 users (show)

See Also:

Attachments
test snippet (1.18 KB, text/x-java)
2010-01-29 06:12 EST, Tim Buschtoens CLA 		no flags Details
fix / workaround (4.10 KB, patch)
2010-01-29 06:39 EST, Tim Buschtoens CLA 		ivan: iplog+
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 Tim Buschtoens CLA 2010-01-29 06:12:59 EST 
Created attachment 157606 [details]
test snippet

To reproduce, execute snippet using fancy-design, click button.
Comment 1 Tim Buschtoens CLA 2010-01-29 06:30:22 EST 
What we know so far:
- The image is set as a source on the svg-image node.
- Chrome sends a succesfull request to load the image.
- So far there is no difference to loading the image using HTML/DOM.
- Now Chrome sends a SECOND request.
- This request looks identical to the initial request (like pressing F5).
- This causes the session to be killed. 
- With the next AJAX-request, the response is the index.html

Noteworthy:
- Safari does not have this problem, even though it also uses webkit.
- The url is set on the svg-image using xLink (there is no other way):
node.setAttributeNS( "http://www.w3.org/1999/xlink", "href", value );
- The second (bad) request is not logged by the developer tools of Chrome.
Comment 2 Tim Buschtoens CLA 2010-01-29 06:39:12 EST 
Created attachment 157609 [details]
fix / workaround

Prevents the problem by loading the image using HTML/DOM first.
Comment 3 Ivan Furnadjiev CLA 2010-01-29 09:39:30 EST 
Applied patch to CVS HEAD.
Comment 4 Tim Buschtoens CLA 2010-12-17 10:23:19 EST 
This issues resurfaced in 1.4M4.
Comment 5 Tim Buschtoens CLA 2010-12-17 10:24:50 EST 
*** Bug 332850 has been marked as a duplicate of this bug. ***
Comment 6 Tim Buschtoens CLA 2010-12-17 11:15:28 EST 
The workaround still does what it should do, but in some cases a second request is created anyway. 

Also, i found the chrome bug entry for this:
http://code.google.com/p/chromium/issues/detail?id=53581
Comment 7 Tim Buschtoens CLA 2010-12-17 12:17:01 EST 
Workaround isn't effective because the shape is not in dom when setting the background-image.
Comment 8 Ivan Furnadjiev CLA 2010-12-17 12:44:41 EST 
Fixed in CVS HEAD.
Comment 9 Ivan Furnadjiev CLA 2011-01-10 08:04:35 EST 
*** Bug 333858 has been marked as a duplicate of this bug. ***