LGTM

http://gwt-code-reviews.appspot.com/92802/diff/1/5
File dev/core/src/com/google/gwt/dev/util/BrowserLauncher.java (right):

http://gwt-code-reviews.appspot.com/92802/diff/1/5#newcode110
Line 110: + " in your browser");
This might be even more annoying that doing nothing. Have you checked to
see if the text in that dialog can be copied and pasted?

http://gwt-code-reviews.appspot.com/92802/diff/1/5#newcode203
Line 203: Runtime.getRuntime().exec("rundll32
url.dll,FileProtocolHandler " + url);
You could do something similar on mac:
    exec(new String[] {"/usr/bin/open", url})

http://gwt-code-reviews.appspot.com/92802

--~--~---------~--~----~------------~-------~--~----~
http://groups.google.com/group/Google-Web-Toolkit-Contributors
-~----------~----~----~----~------~----~------~--~---

Reply via email to