Attachment 210744 Details for Bug 370976 – Corrected patch

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

[patch] Corrected patch

browser.execute.patch (text/plain), 1.06 KB, created by Graham Mising name

on 2012-02-08 12:13:18 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Graham Mising name

Created: 2012-02-08 12:13:18 EST

Size: 1.06 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.rap
>Index: runtime.rwt/org.eclipse.rap.rwt/src/org/eclipse/swt/browser/Browser.java
>===================================================================
>RCS file: /cvsroot/rt/org.eclipse.rap/runtime.rwt/org.eclipse.rap.rwt/src/org/eclipse/swt/browser/Browser.java,v
>retrieving revision 1.35
>diff -u -r1.35 Browser.java
>--- runtime.rwt/org.eclipse.rap.rwt/src/org/eclipse/swt/browser/Browser.java	26 Dec 2011 14:56:49 -0000	1.35
>+++ runtime.rwt/org.eclipse.rap.rwt/src/org/eclipse/swt/browser/Browser.java	8 Feb 2012 17:11:32 -0000
>@@ -283,12 +283,14 @@
>     if( executeScript == null ) {
>       executeScript = script;
>       executeResult = null;
>-      while( executeResult == null ) {
>+      while( executeResult == null  && !isDisposed() ) {
>         Display display = getDisplay();
>         if( !display.readAndDispatch() )  {
>           display.sleep();
>         }
>       }
>+      if ( isDisposed() )
>+        return false;
>       executeScript = null;
>       executePending = false;
>       result = executeResult.booleanValue();

Actions: View | Diff

Attachments on bug 370976: 210741 | 210744 | 210782