Patch from Michael Scott <mscott@ti.com>, Texas Instruments:

> I work in a group that re-packages the Rhino Debugger as part of a set
> of "C"/JavaScript-based configuration tools for developing embedded
> software. We have had a long-standing problem with setting
> breakOnException, breakOnEnter, and breakOnReturn flags within the debug
> engine and not having them in sync with the gui debug menu checkboxes. I
> have fixed this for our use and have attached the updated Main.java
> file. The change simply sets the checkbox selected state along with the
> dim flags.

js/rhino/toolsrc/org/mozilla/javascript/tools/debugger/Main.java

@@ -99,6 +99,7 @@ public class Main implements ContextListener
     */
     public void setBreakOnExceptions(boolean value) {
         dim.breakOnExceptions = value;
+        debugGui.menubar.breakOnExceptions.setSelected(value);
     }
 
    /**
@@ -106,6 +107,7 @@ public class Main implements ContextListener
     */
     public void setBreakOnEnter(boolean value) {
         dim.breakOnEnter = value;
+        debugGui.menubar.breakOnEnter.setSelected(value);
     }
 
    /**
@@ -113,6 +115,7 @@ public class Main implements ContextListener
     */
     public void setBreakOnReturn(boolean value) {
         dim.breakOnReturn = value;
+        debugGui.menubar.breakOnReturn.setSelected(value);
     }
 
     /**