2002/07/16 00:39:22
[org.ibex.core.git] / src / org / xwt / Box.java
index 1fc2e82..e7d63cb 100644 (file)
@@ -225,13 +225,6 @@ public final class Box extends JSObject {
     /** Holds four "strip images" -- 0=top, 1=bottom, 2=left, 3=right, 4=all */
     Picture[] border = null;
         
-    /**
-     *  The dirty() method walks up the tree, clipping the region being dirtied to each parent -- if this value is less than or
-     *  equal to surface.dirtiedTimeStamp, it means that the entire screen region covered by this box has been dirtied, so dirty()'s on
-     *  children can short-circuit.
-     */
-    int dirtiedTimeStamp = 0;
-
     /** true iff the box's background image should be tiled */
     boolean tile = false;
 
@@ -241,6 +234,9 @@ public final class Box extends JSObject {
     /** If true, the Box will force its own size to the natural size of its background image */
     boolean sizetoimage = false;
 
+    /** If true and tile is false, the background of this image will never be stretched */
+    boolean fixedaspect = false;
+
     /** If true, the box will shrink to the smallest vertical size possible */
     boolean vshrink = false;
 
@@ -296,6 +292,9 @@ public final class Box extends JSObject {
         if (getParent() == null && surface != null && which == size)
             surface._setSize(axis == 0 ? newvalue : size(0), axis == 1 ? newvalue : size(1));
 
+        if (getParent() == null && surface != null && (which == dmin || which == dmax))
+            surface.setLimits(dmin(0), dmin(1), dmax(0), dmax(1));
+
         switch(which) {
         case dmin: if (dmin(axis) == newvalue) return; if (axis == 0) _dmin_0 = newvalue; else _dmin_1 = newvalue; break;
         case dmax: if (dmax(axis) == newvalue) return; if (axis == 0) _dmax_0 = newvalue; else _dmax_1 = newvalue; break;
@@ -332,6 +331,9 @@ public final class Box extends JSObject {
         // if the pad changes, update cmin
         if (which == pad) sync_cmin_to_children();
 
+        // needed in the shrink case, since dmin may have been the deciding factor in calculating cmin
+        if ((vshrink || hshrink) && (which == dmin || which == textdim || which == pad)) sync_cmin_to_children();
+
         // if the cmin changes, we need to be re-prerendered
         if (which == cmin) mark_for_prerender(); 
 
@@ -411,7 +413,7 @@ public final class Box extends JSObject {
     }
 
     /** loads the image described by string str, possibly blocking for a network load */
-    private static ImageDecoder getImage(String str) {
+    static ImageDecoder getImage(String str, final Function callback) {
         ImageDecoder ret = null;
         boolean ispng = false;
 
@@ -427,12 +429,47 @@ public final class Box extends JSObject {
                 if (Log.on) Log.log(Box.class, "HTTP images can not be loaded from the foreground thread");
                 return null;
             }
+            // FIXME: use primitives here
             ThreadMessage mythread = (ThreadMessage)thread;
             mythread.setPriority(Thread.MIN_PRIORITY);
             mythread.done.release();
             try {
-                if (str.endsWith(".png")) ret = PNG.decode(Platform.urlToInputStream(new URL(str)), str);
-                else ret = GIF.decode(Platform.urlToInputStream(new URL(str)), str);
+                // FIXME use mime types here, not extensions
+                if (str.endsWith(".jpeg") || str.endsWith(".jpg"))
+                    str = "http://xmlrpc.xwt.org/jpeg2png/" + str.substring(str.indexOf("//") + 2);
+
+                final HTTP http = new HTTP(str);
+                final int contentLength = http.getContentLength();
+                InputStream is = new FilterInputStream(http.getInputStream()) {
+                        int bytesDownloaded = 0;
+                        boolean clear = true;
+                        public int read() throws IOException {
+                            bytesDownloaded++;
+                            return super.read();
+                        }
+                        public int read(byte[] b, int off, int len) throws IOException {
+                            int ret = super.read(b, off, len);
+                            if (ret != -1) bytesDownloaded += ret;
+                            if (clear && callback != null) {
+                                clear = false;
+                                ThreadMessage.newthread(new JSObject.JSFunction() {
+                                        public Object call(Context cx, Scriptable thisObj, Scriptable ctorObj, Object[] args) throws JavaScriptException {
+                                            try {
+                                                callback.call(cx, null, null, new Object[] {
+                                                    new Double(bytesDownloaded), new Double(contentLength) });
+                                            } finally {
+                                                clear = true;
+                                            }
+                                            return null;
+                                        }
+                                    });
+                            }
+                            return ret;
+                        }
+                    };
+
+                if (str.endsWith(".gif")) ret = GIF.decode(is, str);
+                else ret = PNG.decode(is, str);
                 return ret;
 
             } catch (IOException e) {
@@ -453,7 +490,7 @@ public final class Box extends JSObject {
         Picture ret = null;
         ret = (Picture)pictureCache.get(os);
         if (ret != null) return ret;
-        ImageDecoder id = getImage(os);
+        ImageDecoder id = getImage(os, null);
         if (id == null) return null;
         ret = Platform.createPicture(id);
         pictureCache.put(os, ret);
@@ -491,7 +528,7 @@ public final class Box extends JSObject {
         } else {
             border = (Picture[])bordercache.get(s);
             if (border == null) {
-                ImageDecoder id = getImage(s);
+                ImageDecoder id = getImage(s, null);
                 if (id == null) {
                     if (Log.on) Log.log(this, "unable to load border image " + s + " at " +
                                     Context.enter().interpreterSourceFile + ":" + Context.enter().interpreterLine);
@@ -553,13 +590,11 @@ public final class Box extends JSObject {
     /** Adds the node's current actual geometry to the Surface's dirty list */
     void dirty() {
         dirty(pos(0), pos(1), size(0), size(1));
-        if (surface != null) dirtiedTimeStamp = surface.dirtiedTimeStamp + 1;
     }
 
     /** Adds the intersection of (x,y,w,h) and the node's current actual geometry to the Surface's dirty list */
-    final void dirty(int x, int y, int w, int h) {
+    public final void dirty(int x, int y, int w, int h) {
         for(Box cur = this; cur != null; cur = cur.getParent()) {
-            if (surface != null && cur.dirtiedTimeStamp > surface.dirtiedTimeStamp && cur.dirtiedTimeStamp < Integer.MAX_VALUE - 1) return;
             w = min(x + w, cur.pos(0) + cur.size(0)) - max(x, cur.pos(0));
             h = min(y + h, cur.pos(1) + cur.size(1)) - max(y, cur.pos(1));
             x = max(x, cur.pos(0));
@@ -602,26 +637,31 @@ public final class Box extends JSObject {
     }
 
     /** creates a new box from an anonymous template; <tt>ids</tt> is passed through to Template.apply() */
-    Box(Template anonymous, Vec pboxes, Vec ptemplates) {
+    Box(Template anonymous, Vec pboxes, Vec ptemplates, Function callback, int numerator, int denominator) {
         super(true);
         set(dmax, 0, Short.MAX_VALUE);
         set(dmax, 1, Short.MAX_VALUE);
         template = anonymous;
-        template.apply(this, pboxes, ptemplates);
+        template.apply(this, pboxes, ptemplates, callback, numerator, denominator);
         templatename = null;
         importlist = null;
     }
 
     /** creates a new box from an unresolved templatename and an importlist; use "box" for an untemplatized box */
-    public Box(String templatename, String[] importlist) {
+    public Box(String templatename, String[] importlist) { this(templatename, importlist, null); }
+    public Box(String templatename, String[] importlist, Function callback) {
         super(true);
         set(dmax, 0, Short.MAX_VALUE);
         set(dmax, 1, Short.MAX_VALUE);
         this.importlist = importlist;
-        template = "box".equals(templatename) ? null : Template.getTemplate(templatename, importlist);
-        this.templatename = templatename;
+        if (!"box".equals(templatename)) {
+            template = Template.getTemplate(templatename, importlist);
+            if (template == null)
+                if (Log.on) Log.log(this, "couldn't find template \"" + templatename + "\"");
+        }
         if (template != null) {
-            template.apply(this, null, null);
+            this.templatename = templatename;
+            template.apply(this, null, null, callback, 0, template.numUnits());
             if (redirect == this && !"self".equals(template.redirect)) redirect = null;
         }
     }
@@ -632,6 +672,8 @@ public final class Box extends JSObject {
     /** Checks if the Box's size has changed, dirties it if necessary, and makes sure childrens' sizes are up to date */
     void prerender() {
 
+        if (invisible) return;
+
         if (getParent() == null) {
             set(pos, 0, 0);
             set(pos, 1, 0);
@@ -663,10 +705,11 @@ public final class Box extends JSObject {
 
         // FASTPATH: if we haven't moved position (just changed size), and we're not a stretched image:
         if (oldpos(0) == pos(0) && oldpos(1) == pos(1) && (image == null || tile)) {
-            
-            int bw = border == null ? 0 : border[2].getWidth();
-            int bh = border == null ? 0 : border[0].getHeight();
-            
+
+            // we use the max(border, pad) since because of the pad we might be revealing an abs-pos child
+            int bw = max(border == null ? 0 : border[2].getWidth(), pad(0));
+            int bh = max(border == null ? 0 : border[0].getHeight(), pad(1));
+
             // dirty only the *change* in the area we cover, both on ourselves and on our parent
             for(Box cur = this; cur != null && (cur == this || cur == this.getParent()); cur = cur.getParent()) {
                 cur.dirty(pos(0) + min(oldsize(0) - bw, size(0) - bw),
@@ -696,21 +739,25 @@ public final class Box extends JSObject {
         set(oldpos, 0, pos(0));
         set(oldpos, 1, pos(1));
 
-        if (sizechange || poschange)
-            if (surface.sizePosChangesSinceLastRender++ > 500) {
+        if (!sizechange && !poschange) return;
+
+        if (++surface.sizePosChangesSinceLastRender >= 500) {
+            if (surface.sizePosChangesSinceLastRender == 500) {
                 if (Log.on) Log.log(this, "Warning, more than 500 SizeChange/PosChange traps triggered since last complete render");
+                if (Log.on) Log.log(this, "    interpreter is at " + Context.enter().interpreterSourceFile + ":" + Context.enter().interpreterLine);
                 try {
                     Trap t = sizechange ? Trap.getTrap(this, "SizeChange") : Trap.getTrap(this, "PosChange");
                     InterpretedFunction f = (InterpretedFunction)t.f;
                     if (Log.on) Log.log(this, "Current trap is at " + f.getSourceName() + ":" + f.getLineNumbers()[0]);
                 } catch (Throwable t) { }
             }
-
-        if (sizechange) put("SizeChange", null, Boolean.TRUE);
-        if (poschange) put("PosChange", null, Boolean.TRUE);
-        if (sizechange || poschange) {
-            surface.abort = true;
-            return;
+        } else {
+            if (sizechange) put("SizeChange", null, Boolean.TRUE);
+            if (poschange) put("PosChange", null, Boolean.TRUE);
+            if (sizechange || poschange) {
+                surface.abort = true;
+                return;
+            }
         }
     }
     
@@ -817,7 +864,6 @@ public final class Box extends JSObject {
 
     /** Renders self and children within the specified region. All rendering operations are clipped to xIn,yIn,wIn,hIn */
     void render(int xIn, int yIn, int wIn, int hIn, DoubleBuffer buf) {
-        dirtiedTimeStamp = surface.dirtiedTimeStamp;
         if (surface.abort || invisible) return;
 
         // intersect the x,y,w,h rendering window with ourselves; quit if it's empty
@@ -828,16 +874,16 @@ public final class Box extends JSObject {
         if (w <= 0 || h <= 0) return;
 
         if (border != null) renderBorder(x, y, w, h, buf);
-
         if ((color & 0xFF000000) != 0x00000000 || getParent() == null) {
             int bw = border == null ? 0 : border[2].getWidth();
             int bh = border == null ? 0 : border[0].getHeight();
+            int x1 = max(x, pos(0) + bw);
+            int y1 = max(y, pos(1) + bh);
+            int x2 = min(x + w, pos(0) + size(0) - bw);
+            int y2 = min(y + h, pos(1) + size(1) - bh);
             buf.setClip(0, 0, buf.getWidth(), buf.getHeight());
-            buf.fillRect(max(x, pos(0) + bw),
-                         max(y, pos(1) + bh),
-                         min(x + w, pos(0) + size(0) - bw),
-                         min(y + h, pos(1) + size(1) - bh),
-                         (color & 0xFF000000) != 0 ? color : SpecialBoxProperty.lightGray);
+            if (y2 - y1 > 0 && x2 - x1 > 0)
+                buf.fillRect(x1,y1,x2,y2,(color & 0xFF000000) != 0 ? color : SpecialBoxProperty.lightGray);
         }
 
         if (image != null) {
@@ -863,9 +909,8 @@ public final class Box extends JSObject {
         buf.setClip(x, y, w + x, h + y);
 
         if ((color & 0xFF000000) != 0xFF000000) {
-
             // if the color is null, we have to be very careful about drawing the corners
-            if (Log.verbose) Log.log(this, "WARNING: (color == null && border != null) on box with border " + imageToNameMap.get(border[4]));
+            //if (Log.verbose) Log.log(this, "WARNING: (color == null && border != null) on box with border " + imageToNameMap.get(border[4]));
 
             // upper left corner
             buf.drawPicture(border[4],
@@ -920,11 +965,28 @@ public final class Box extends JSObject {
         buf.setClip(x, y, w + x, h + y);
         int bw = border == null ? 0 : border[4].getHeight();
         int bh = border == null ? 0 : border[4].getWidth();
+
+        int width = pos(0) + size(0) - bw / 2 - pos(0) + bw / 2;
+        int height = pos(1) + size(1) - bh / 2 - pos(1) + bh / 2;
+
+        if (fixedaspect) {
+            int hstretch = width / image.getWidth();
+            if (hstretch == 0) hstretch = -1 * image.getWidth() / width;
+            int vstretch = height / image.getHeight();
+            if (vstretch == 0) vstretch = -1 * image.getHeight() / height;
+
+            if (hstretch < vstretch) {
+                height = image.getHeight() * width / image.getWidth();
+            } else {
+                width = image.getWidth() * height / image.getHeight();
+            }
+        }
+
         buf.drawPicture(image,
                         pos(0) + bw / 2,
                         pos(1) + bh / 2,
-                        pos(0) + size(0) - bw / 2,
-                        pos(1) + size(1) - bh / 2,
+                        pos(0) + bw / 2 + width,
+                        pos(1) + bh / 2 + height,
                         0, 0, image.getWidth(), image.getHeight());
         buf.setClip(0, 0, buf.getWidth(), buf.getHeight());
     }
@@ -979,15 +1041,15 @@ public final class Box extends JSObject {
 
         if (font.lastIndexOf('d') > i) {
             for(int j = pos(0) + pad(0); j < pos(0) + pad(0) + textdim(0); j += 2)
-                buf.fillRect(j, pos(1) + pad(1) + (xwf == null ? Platform.getMaxAscent(font) : xwf.getMaxAscent()) + 3,
-                             j + 1, pos(1) + pad(1) + (xwf == null ? Platform.getMaxAscent(font) : xwf.getMaxAscent()) + 3 + 1,
+                buf.fillRect(j, pos(1) + pad(1) + (xwf == null ? Platform.getMaxAscent(font) : xwf.getMaxAscent()) + 2,
+                             j + 1, pos(1) + pad(1) + (xwf == null ? Platform.getMaxAscent(font) : xwf.getMaxAscent()) + 2 + 1,
                              textcolor);
 
         } else if (font.lastIndexOf('u') > i) {
             buf.fillRect(pos(0) + pad(0),
-                        pos(1) + pad(1) + (xwf == null ? Platform.getMaxAscent(font) : xwf.getMaxAscent()) + 3,
+                        pos(1) + pad(1) + (xwf == null ? Platform.getMaxAscent(font) : xwf.getMaxAscent()) + 2,
                         pos(0) + pad(0) + textdim(0),
-                        pos(1) + pad(1) + (xwf == null ? Platform.getMaxAscent(font) : xwf.getMaxAscent()) + 3 + 1,
+                        pos(1) + pad(1) + (xwf == null ? Platform.getMaxAscent(font) : xwf.getMaxAscent()) + 2 + 1,
                         textcolor);
         }
 
@@ -1011,7 +1073,7 @@ public final class Box extends JSObject {
      */
     public void put(int i, Scriptable start, Object value) {
         if (value == null) {
-            if (i > 0 && i < numChildren()) getChild(i).remove();
+            if (i >= 0 && i < numChildren()) getChild(i).remove();
             return;
         }
         if (value instanceof RootProxy) {
@@ -1024,6 +1086,14 @@ public final class Box extends JSObject {
             return;
         }
         Box newnode = (Box)value;
+
+        for(Box cur = this; cur != null; cur = cur.getParent())
+            if (cur == newnode) {
+                if (Log.on) Log.log(this, "attempt to make a node a parent of its own ancestor at " + 
+                                    Context.enter().interpreterSourceFile + ":" + Context.enter().interpreterLine);
+                return;
+            }
+
         if (redirect == null) {
             if (Log.on) Log.log(this, "attempt to add a child to a node with a null redirect at " + 
                                 Context.enter().interpreterSourceFile + ":" + Context.enter().interpreterLine);
@@ -1089,6 +1159,7 @@ public final class Box extends JSObject {
         if (name.equals("Array")) return JSObject.defaultObjects.get("Array", null);
         if (name.equals("Function")) return JSObject.defaultObjects.get("Function", null);
         if (name.equals("TypeError")) return JSObject.defaultObjects.get("TypeError", null);
+        if (name.equals("ConversionError")) return JSObject.defaultObjects.get("ConversionError", null);
 
         // See if we're reading back the function value of a trap
         if (name.charAt(0) == '_') {
@@ -1106,7 +1177,11 @@ public final class Box extends JSObject {
         SpecialBoxProperty gph = (SpecialBoxProperty)SpecialBoxProperty.specialBoxProperties.get(name);
         if (gph != null) return gph.get(this);
 
-        return super.get(name, start);
+        Object ret = super.get(name, start);
+        if (name.startsWith("$") && ret == null)
+            if (Log.on) Log.log(this, "WARNING: attempt to access " + name + ", but no child with id=\"" + name.substring(1) + "\" found; " +
+                                Context.enter().interpreterSourceFile + ":" + Context.enter().interpreterLine);
+        return ret;
     }
 
     /** indicate that we don't want JSObject trying to handle these */
@@ -1115,7 +1190,8 @@ public final class Box extends JSObject {
         if (traps != null && traps.get(name) != null) return true;
         if (name.charAt(0) == '_') return true;
         if (SpecialBoxProperty.specialBoxProperties.get(name) != null) return true;
-        if (name.equals("Function") || name.equals("Array") || name.equals("Object") || name.equals("TypeError")) return true;
+        if (name.equals("Function") || name.equals("Array") || name.equals("Object") ||
+            name.equals("TypeError") || name.equals("ConversionError")) return true;
         return super.has(name, start);
     }
 
@@ -1413,6 +1489,10 @@ public final class Box extends JSObject {
 
     /** returns numerator/denominator, but rounds <i>up</i> instead of down */
     static final int divide_round_up(int numerator, int denominator) {
+
+        // cope with bozos who use flex==0.0
+        if (denominator == 0) return Integer.MAX_VALUE;
+
         int ret = numerator / denominator;
         if (ret * denominator < numerator) return ret + 1;
         return ret;