Applied code to fix the PHP Browser hang issue reported in bug #1374420. It was teste...
authorEdward Mann <phpeclipse.dev@edmann.com>
Thu, 17 May 2007 03:11:14 +0000 (03:11 +0000)
committerEdward Mann <phpeclipse.dev@edmann.com>
Thu, 17 May 2007 03:11:14 +0000 (03:11 +0000)
This commit also changes the behavior of the browser url bar.
If you had typed in php.net  and hit enter it would have saved a copy to
your browser history. Then if you wanted to go to phpeclipse.net, you would
not get past php before it would auto take you to php.net. This is changed.
you can now type in the url you would like to visit before any action is
taken.

net.sourceforge.phpeclipse.webbrowser/src/net/sourceforge/phpeclipse/webbrowser/internal/WebBrowser.java

index 961fd5d..c4e6e52 100644 (file)
@@ -139,7 +139,8 @@ public class WebBrowser extends Composite {
                                public void widgetSelected(SelectionEvent we) {
                                        try {
                                                if (combo.getSelectionIndex() != -1)
-                                                       setURL(combo.getItem(combo.getSelectionIndex()));
+                                                       setURL(combo.getItem(combo.getSelectionIndex()),
+                                                                       false);
                                        } catch (Exception e) {
                                        }
                                }
@@ -238,7 +239,8 @@ public class WebBrowser extends Composite {
                                        shell.dispose();
                                else {
                                        // #1365431 (toshihiro) editor.closeEditor(); causes NPE
-                               if (editor != null) editor.closeEditor();
+                                       if (editor != null)
+                                               editor.closeEditor();
                                }
                        }
                });
@@ -379,11 +381,12 @@ public class WebBrowser extends Composite {
                if (newURL != null)
                        url = newURL;
 
-               if (browse)
+               if (browse) {
                        navigate(url);
 
-               addToHistory(url);
-               updateHistory();
+                       addToHistory(url);
+                       updateHistory();
+               }
        }
 
        protected void addToHistory(String url) {