2003/12/07 08:45:03

[org.ibex.core.git] / src / org / xwt / Box.java

diff --git a/src/org/xwt/Box.java b/src/org/xwt/Box.java
index 366b04b..06a5e91 100644 (file)
--- a/src/org/xwt/Box.java
+++ b/src/org/xwt/Box.java
@@ -63,6 +63,31 @@ public final class Box extends JSScope {
     public static final int MAX_LENGTH = Integer.MAX_VALUE;
     static final Font DEFAULT_FONT = Font.getFont((Res)Main.builtin.get("fonts/vera/Vera.ttf"), 10);
 
+    // box properties can not be trapped
+    static final String[] props = new String[] {
+        "fill", "stroke", "image", "tile", "fixedaspect", "text", "path", "font",
+        "shrink", "hshrink", "vshrink", "x", "y", "width", "height", "cols", "rows",
+        "colspan", "rowspan", "align", "invisible", "absolute", "globalx", "globaly",
+        "minwidth", "maxwidth", "minheight", "maxheight",
+        "numchildren", "redirect", "cursor", "mousex", "mousey", "xwt", "static",
+        "mouseinside", "root", "thisbox", "indexof"
+    };
+
+    // events can have write traps, but not read traps
+    static final String[] events = new String[] {
+        "Press1", "Press2", "Press3",
+        "Release1", "Release2", "Release3",
+        "Click1", "Click2", "Click3",
+        "DoubleClick1", "DoubleClick2", "DoubleClick3",
+        "Enter", "Leave", "Move", 
+        "KeyPressed", "KeyReleased", "PosChange", "SizeChange",
+
+        "childadded", "childremoved",
+
+        "Focused", "Maximized", "Minimized", "Close",
+        "icon", "titlebar", "toback", "tofront"
+    };
+
     // Flags //////////////////////////////////////////////////////////////////////
 
     static final int MOUSEINSIDE  = 0x00000001;
@@ -238,10 +263,10 @@ public final class Box extends JSScope {
             boolean poschange = (this.x != x || this.y != y) && getTrap("PosChange") != null;
             this.width = width; this.height = height; this.x = x; this.y = y;
             dirty();
-            try { if (sizechange) putAndTriggerTraps("SizeChange", T); /*Surface.abort = true;*/ }
-            catch (Exception e) { Log.log(this, e); }
-            try { if (poschange) putAndTriggerTraps("PosChange", T); /*Surface.abort = true;*/ }
-            catch (Exception e) { Log.log(this, e); }
+            if (sizechange) try { putAndTriggerTraps("SizeChange", T); /*Surface.abort = true;*/ }
+                catch (Exception e) { Log.log(this, e); }
+            if (poschange) try { putAndTriggerTraps("PosChange", T); /*Surface.abort = true;*/ }
+                catch (Exception e) { Log.log(this, e); }
         }
     }
 
@@ -335,7 +360,7 @@ public final class Box extends JSScope {
                 for(int y = globaly; y < cy2; y += texture.picture.getHeight())
                     buf.drawPicture(texture.picture, x, y, cx1, cy1, cx2, cy2);
 
-       if (text != null && !text.equals("") && font != null)
+        if (text != null && !text.equals("") && font != null)
             if (font.rasterizeGlyphs(text, buf, strokecolor, globalx, globaly, cx1, cy1, cx2, cy2, null) == -1)
                 font.rasterizeGlyphs(text, buf, strokecolor, globalx, globaly, cx1, cy1, cx2, cy2,
                                     new Scheduler.Task() { public void perform() { Box b = Box.this; MARK_REFLOW_b; dirty(); }});
@@ -364,7 +389,18 @@ public final class Box extends JSScope {
 
     public Enumeration keys() { throw new Error("you cannot apply for..in to a " + this.getClass().getName()); }
 
-    protected boolean isTrappable() { return true; }
+    protected boolean isTrappable(Object key, boolean isRead) {
+        if (key == null) return false;
+        else if (key instanceof String) {
+            // not allowed to trap box properties, and no read traps on events
+            String name = (String)key;
+            for (int i=0; i < props.length; i++) if (name.equals(props[i])) return false; 
+            if (isRead) for (int i=0; i < events.length; i++) if (name.equals(events[i])) return false; 
+        }
+
+        return true;
+    }
+
     public Object get(Object name) throws JSExn {
         if (name instanceof Number)
             return redirect == null ? null : redirect == this ? getChild(toInt(name)) : redirect.get(name);