Attachment 55517 Details for Bug 146894 – patch

[patch] patch

Patch_Bugzilla146894 (text/plain), 1.94 KB, created by Alex Boyko

on 2006-12-12 13:33:04 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Alex Boyko

Created: 2006-12-12 13:33:04 EST

Size: 1.94 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.draw2d
>Index: src/org/eclipse/draw2d/DeferredUpdateManager.java
>===================================================================
>RCS file: /cvsroot/tools/org.eclipse.draw2d/src/org/eclipse/draw2d/DeferredUpdateManager.java,v
>retrieving revision 1.22
>diff -u -r1.22 DeferredUpdateManager.java
>--- src/org/eclipse/draw2d/DeferredUpdateManager.java	5 Apr 2006 14:21:13 -0000	1.22
>+++ src/org/eclipse/draw2d/DeferredUpdateManager.java	12 Dec 2006 18:09:47 -0000
>@@ -55,6 +55,7 @@
> private boolean updateQueued = false;
> 
> private boolean updating;
>+private boolean validating = false;
> private RunnableChain afterUpdate;
> 
> private static class RunnableChain {
>@@ -137,16 +138,28 @@
> 
> void paint(GC gc) {
> 	SWTGraphics graphics = new SWTGraphics(gc);
>+	Rectangle rect = graphics.getClip(new Rectangle());
> 	if (!updating) {
> 		/**
> 		 * If a paint occurs not as part of an update, we should notify that the region
> 		 * is being painted. Otherwise, notification already occurs in repairDamage().
> 		 */
> 		HashMap map = new HashMap();
>-		Rectangle rect = graphics.getClip(new Rectangle());
> 		map.put(root, rect);
> 		firePainting(rect, map);
>+	} else {
>+		/*
>+		 * If figures are being validated (as part of an update) then we can simply
>+		 * add a dirty region here and update will repaint this region with other 
>+		 * dirty regions when it gets to painting. We can't paint if we're not sure
>+		 * that all figures are valid. 
>+		 */
>+		if (validating) {
>+			addDirtyRegion(root, rect);
>+			return;
>+		}
> 	}
>+	performValidation();
> 	root.paint(graphics);
> 	graphics.dispose();
> }
>@@ -185,6 +198,7 @@
> 		return;
> 	try {
> 		IFigure fig;
>+		validating = true;
> 		fireValidating();
> 		for (int i = 0; i < invalidFigures.size(); i++) {
> 			fig = (IFigure) invalidFigures.get(i);
>@@ -193,6 +207,7 @@
> 		}
> 	} finally {
> 		invalidFigures.clear();
>+		validating = false;
> 	}
> }
>

Actions: View | Diff

Attachments on bug 146894: 54360 | 54567 | 54963 | 55517 | 55530 | 55531 | 55559 | 55871 | 56843