Attachment 232093 Details for Bug 410157 – quickfix

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

[patch] quickfix

Fix-ServerPush-in-IE9.patch (text/plain), 1.51 KB, created by Tim Buschtoens

on 2013-06-07 07:24:45 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Tim Buschtoens

Created: 2013-06-07 07:24:45 EDT

Size: 1.51 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.rap.rwt
>diff --git js/rwt/client/ServerPush.js js/rwt/client/ServerPush.js
>index 0c9a1dc..fd33693 100644
>--- js/rwt/client/ServerPush.js
>+++ js/rwt/client/ServerPush.js
>@@ -59,7 +59,8 @@
>     var request = new rwt.remote.Request( server.getUrl(), "GET", "application/javascript" );
>     request.setSuccessHandler( this._handleSuccess, this );
>     request.setErrorHandler( this._handleError, this );
>-    request.setData( "servicehandler=org.eclipse.rap.serverpush&cid=" + server.getConnectionId() );
>+    var timestamp = new Date().getTime();
>+    request.setData( "servicehandler=org.eclipse.rap.serverpush&cid=" + server.getConnectionId() + "&no-cache=" + timestamp );
>     return request;
>   },
> 
>#P org.eclipse.rap.rwt.jstest
>diff --git js/org/eclipse/rwt/test/tests/ServerPushTest.js js/org/eclipse/rwt/test/tests/ServerPushTest.js
>index ecd814c..1dfca72 100644
>--- js/org/eclipse/rwt/test/tests/ServerPushTest.js
>+++ js/org/eclipse/rwt/test/tests/ServerPushTest.js
>@@ -56,8 +56,8 @@
> 
>       var request = serverPush._createRequest();
> 
>-      var expected = "servicehandler=org.eclipse.rap.serverpush&cid=" + server.getConnectionId();
>-      assertEquals( expected, request.getData() );
>+      var expected = "servicehandler=org.eclipse.rap.serverpush&cid=" + server.getConnectionId() + "&no-cache=";
>+      assertTrue( request.getData().indexOf( expected ) !== -1 );
>       assertEquals( "GET", request._method );
>       assertEquals( "application/javascript", request._responseType );
>     },

Flags:

tbuschto: review?

Actions: View | Diff

Attachments on bug 410157: 232093 | 232095