2003/12/30 01:41:37
[org.ibex.core.git] / src / org / xwt / Surface.java
index fe5844f..e241742 100644 (file)
@@ -29,7 +29,9 @@ public abstract class Surface extends PixelBuffer implements Scheduler.Task {
     public static Vec allSurfaces = new Vec();
     
     /** When set to true, render() should abort as soon as possible and restart the rendering process */
-    static volatile boolean abort = false;
+    volatile boolean abort = false;
+    volatile int pendingWidth = -1;
+    volatile int pendingHeight = -1;
 
     public static boolean alt = false;          ///< true iff the alt button is pressed down, in real time
     public static boolean control = false;      ///< true iff the control button is pressed down, in real time
@@ -46,11 +48,13 @@ public abstract class Surface extends PixelBuffer implements Scheduler.Task {
     public String cursor = "default";                  ///< The active cursor to switch to when syncCursor() is called
     public int mousex;                                 ///< x position of the mouse, in Scheduler-time
     public int mousey;                                 ///< y position of the mouse, in Scheduler-time
+    public int _mousex;                                ///< x position of the mouse, in Scheduler-time
+    public int _mousey;                                ///< y position of the mouse, in Scheduler-time
     public int newmousex = -1;                         ///< x position of the mouse, in realtime
     public int newmousey = -1;                         ///< y position of the mouse, in realtime
     public boolean minimized = false;                  ///< True iff this surface is minimized, in real time
     public boolean maximized = false;                  ///< True iff this surface is maximized, in real time
-    private DirtyList dirtyRegions = new DirtyList();  ///< Dirty regions on the *screen*
+    DirtyList dirtyRegions = new DirtyList();  ///< Dirty regions on the *screen*
 
     // Used For Simulating Clicks and DoubleClicks /////////////////////////////////////////////////
 
@@ -129,7 +133,6 @@ public abstract class Surface extends PixelBuffer implements Scheduler.Task {
 
     /** we enqueue ourselves in the Scheduler when we have a Move message to deal with */
     public void perform() {
-        inqueue = false;
         if (mousex == newmousex && mousey == newmousey) return;
         int oldmousex = mousex;     mousex = newmousex;
         int oldmousey = mousey;     mousey = newmousey;
@@ -139,8 +142,8 @@ public abstract class Surface extends PixelBuffer implements Scheduler.Task {
             try {
                 root.putAndTriggerTraps("_Move", T);
             } catch (JSExn e) {
-                Log.log(Surface.class, "Exception thrown from Move message handler");
-                Log.logJS(e);
+                Log.info(Surface.class, "Exception thrown from Move message handler");
+                JS.log(e);
             }
         if (!cursor.equals(oldcursor)) syncCursor();
     }
@@ -154,31 +157,15 @@ public abstract class Surface extends PixelBuffer implements Scheduler.Task {
     protected final void Move(final int newmousex, final int newmousey) {
         this.newmousex = newmousex;
         this.newmousey = newmousey;
-        if (!inqueue) Scheduler.add(this);
-        inqueue = true;
+        Scheduler.add(this);
     }
-    boolean inqueue = false;
 
-    Scheduler.Task nextSizeChange = null;
-    int nextWidth = 0;
-    int nextHeight = 0;
-    // FEATURE: can we avoid creating objects here?
     /** subclasses should invoke this method when the user resizes the window */
     protected final void SizeChange(final int width, final int height) {
-        nextWidth = width;
-        nextHeight = height;
-        if (nextSizeChange != null) return;
-        if (root.maxwidth == width && root.maxheight == height) return;
-        nextSizeChange = new Scheduler.Task() { public void perform() {
-            if (nextSizeChange != this) return;
-            // dirty the place where the scar used to be
-            root.dirty(0, root.maxheight - Main.scarImage.height, Main.scarImage.width, Main.scarImage.height);
-            root.setMaxWidth(JS.N(nextWidth));
-            root.setMaxHeight(JS.N(nextHeight));
-            nextSizeChange = null;
-        }};
-        Scheduler.add(nextSizeChange);
-        abort = true;
+        pendingWidth = width;
+        pendingHeight = height;
+        Refresh();
+        Scheduler.add(new Scheduler.Task() { public void perform() { }});
     }
 
     // FEATURE: can we avoid creating objects here?
@@ -198,12 +185,7 @@ public abstract class Surface extends PixelBuffer implements Scheduler.Task {
     protected final void Minimized(boolean b) { minimized = b; new SimpleMessage("Minimized", b ? T : F, root); }
     protected final void Maximized(boolean b) { maximized = b; new SimpleMessage("Maximized", b ? T : F, root); }
     protected final void Focused(boolean b) { new SimpleMessage("Focused", b ? T : F, root); }
-    public static void Refresh() { /*Scheduler.add(renderAll);*/ }
-
-    public static final Scheduler.Task renderAll = new Scheduler.Task() { public void perform() {
-        for(int i=0; i<allSurfaces.size(); i++)
-            ((Surface)allSurfaces.elementAt(i)).render();
-    } };
+    public void Refresh() { abort = true; }
 
     public final void setMaximized(boolean b) { if (b != maximized) _setMaximized(maximized = b); }
     public final void setMinimized(boolean b) { if (b != minimized) _setMinimized(minimized = b); }
@@ -213,11 +195,11 @@ public abstract class Surface extends PixelBuffer implements Scheduler.Task {
 
     /** Indicates that the Surface is no longer needed */
     public final void dispose(boolean quitIfAllSurfacesGone) {
-        if (Log.on) Log.log(this, "disposing " + this);
+        if (Log.on) Log.info(this, "disposing " + this);
         allSurfaces.removeElement(this);
         _dispose();
         if (allSurfaces.size() == 0) {
-            if (Log.on) Log.log(this, "exiting because last surface was destroyed");
+            if (Log.on) Log.info(this, "exiting because last surface was destroyed");
             System.exit(0);
         }
     }
@@ -249,18 +231,18 @@ public abstract class Surface extends PixelBuffer implements Scheduler.Task {
 
     /** runs the prerender() and render() pipelines in the root Box to regenerate the backbuffer, then blits it to the screen */
     public synchronized void render() {
-
-        // dirty the place where the scar used to be in case the root window size was programmatically changed
-        if (root.maxwidth != root.width || root.maxheight != root.height)
-            root.dirty(0, root.height - Main.scarImage.height, Main.scarImage.width, Main.scarImage.height);
-
         // make sure the root is properly sized
         do {
             abort = false;
             root.repack();
+            if (pendingWidth != -1) root.setMaxWidth(JS.N(pendingWidth));
+            if (pendingHeight != -1) root.setMaxHeight(JS.N(pendingHeight));
+            // dirty the place where the scar used to be in case the root window size was programmatically changed
+            if (root.maxwidth != root.width || root.maxheight != root.height)
+                root.dirty(0, root.height - Main.scarImage.height, Main.scarImage.width, Main.scarImage.height);
             root.resize(root.x, root.y, root.maxwidth, root.maxheight);
             root.resize_children();
-            //_setSize(root.width, root.height);
+            _setSize(root.width, root.height);
             String oldcursor = cursor;
             cursor = "default";
             root.putAndTriggerTrapsAndCatchExceptions("_Move", JS.T);
@@ -290,8 +272,6 @@ public abstract class Surface extends PixelBuffer implements Scheduler.Task {
                     if (dirt[j] != null)
                         dirtyRegions.dirty(dirt[j][0], dirt[j][1], dirt[j][2], dirt[j][3]);
 
-                // tail-recurse
-                render();
                 return;
             }
         }
@@ -327,8 +307,8 @@ public abstract class Surface extends PixelBuffer implements Scheduler.Task {
             try {
                 boxContainingMouse.putAndTriggerTraps(name, value);
             } catch (JSExn e) {
-                Log.log(Surface.class, "Exception thrown from "+name+" handler");
-                Log.logJS(e);
+                Log.info(Surface.class, "Exception thrown from "+name+" handler");
+                JS.log(e);
             } finally {
                 Platform.clipboardReadEnabled = false;
             }
@@ -362,12 +342,8 @@ public abstract class Surface extends PixelBuffer implements Scheduler.Task {
         }
 
         public void render() {
-            Dirty(0, 0, root.width, root.height);
             super.render();
-            render_();
-        }
-
-        public void render_() {
+            if (abort) return;
             int[][] dirt = screenDirtyRegions.flush();
             for(int i = 0; dirt != null && i < dirt.length; i++) {
                 if (dirt[i] == null) continue;
@@ -380,6 +356,7 @@ public abstract class Surface extends PixelBuffer implements Scheduler.Task {
                 if (x+w > root.width) w = root.width - x;
                 if (y+h > root.height) h = root.height - y;
                 if (w <= 0 || h <= 0) continue;
+                if (abort) return;
                 blit(backbuffer, x, y, x, y, w + x, h + y);
             }
         }
@@ -387,7 +364,12 @@ public abstract class Surface extends PixelBuffer implements Scheduler.Task {
         /** This is how subclasses signal a 'shallow dirty', indicating that although the backbuffer is valid, the screen is not */
         public final void Dirty(int x, int y, int w, int h) {
             screenDirtyRegions.dirty(x, y, w, h);
-            Refresh();
+            Scheduler.renderAll();
+        }
+
+        public void dirty(int x, int y, int w, int h) {
+            screenDirtyRegions.dirty(x, y, w, h);
+            super.dirty(x, y, w, h);
         }
 
         /** copies a region from the doublebuffer to this surface */