From fac94deeb88966505575d0a06ccbbf856d34476c Mon Sep 17 00:00:00 2001
From: Richard Wilson <rjw@netsurf-browser.org>
Date: Thu, 6 May 2004 22:24:37 +0000
Subject: [project @ 2004-05-06 22:24:37 by rjw] Background redraw bug fix.

svn path=/import/netsurf/; revision=838
---
 riscos/window.c | 74 ++++++++++++++++++++++++++++++++++++---------------------
 1 file changed, 47 insertions(+), 27 deletions(-)

(limited to 'riscos')

diff --git a/riscos/window.c b/riscos/window.c
index f711c8187..c04495b6b 100644
--- a/riscos/window.c
+++ b/riscos/window.c
@@ -14,6 +14,7 @@
 #include <assert.h>
 #include <stdbool.h>
 #include <string.h>
+#include "oslib/colourtrans.h"
 #include "oslib/osspriteop.h"
 #include "oslib/wimp.h"
 #include "oslib/wimpspriteop.h"
@@ -196,34 +197,52 @@ void gui_window_redraw_window(gui_window* g)
 }
 
 
-void ro_gui_window_redraw(gui_window* g, wimp_draw* redraw)
-{
-  osbool more;
-  struct content *c = g->data.browser.bw->current_content;
-
-  if (g->type == GUI_BROWSER_WINDOW && c != NULL)
-  {
-    more = wimp_redraw_window(redraw);
-    wimp_set_font_colours(wimp_COLOUR_WHITE, wimp_COLOUR_BLACK);
+void ro_gui_window_redraw(gui_window* g, wimp_draw* redraw) {
+	osbool more;
+	osbool clear_background = false;
+	struct content *c = g->data.browser.bw->current_content;
+	unsigned long background_colour = 0xffffff;
 
-    while (more)
-    {
-      content_redraw(c,
-	  (int) redraw->box.x0 - (int) redraw->xscroll,
-	  (int) redraw->box.y1 - (int) redraw->yscroll,
-	  c->width * 2, c->height * 2,
-	  redraw->clip.x0, redraw->clip.y0,
-	  redraw->clip.x1 - 1, redraw->clip.y1 - 1,
-	  g->scale);
-      more = wimp_get_rectangle(redraw);
-    }
-  }
-  else
-  {
-    more = wimp_redraw_window(redraw);
-    while (more)
-      more = wimp_get_rectangle(redraw);
-  }
+	if (g->type == GUI_BROWSER_WINDOW && c != NULL) {
+	
+		/*	We should clear the background for GIFs and PNGs
+		*/
+		if ((c->type != CONTENT_HTML) &&
+				(c->type != CONTENT_TEXTPLAIN)) {
+		        clear_background = true;
+		}
+	
+		more = wimp_redraw_window(redraw);
+		wimp_set_font_colours(wimp_COLOUR_WHITE, wimp_COLOUR_BLACK);
+
+    		while (more) {
+			if (clear_background) {
+				colourtrans_set_gcol(background_colour << 8,
+					colourtrans_SET_BG | colourtrans_USE_ECFS,
+					os_ACTION_OVERWRITE, 0);
+				os_clg();
+			}
+		      	content_redraw(c,
+				(int) redraw->box.x0 - (int) redraw->xscroll,
+			  	(int) redraw->box.y1 - (int) redraw->yscroll,
+			  	c->width * 2, c->height * 2,
+			  	redraw->clip.x0, redraw->clip.y0,
+			  	redraw->clip.x1 - 1, redraw->clip.y1 - 1,
+			  	g->scale);
+      			more = wimp_get_rectangle(redraw);
+    		}
+ 	} else {
+    		more = wimp_redraw_window(redraw);
+    		while (more) {
+			if (g->type == GUI_BROWSER_WINDOW && c == NULL) {
+				colourtrans_set_gcol(background_colour << 8,
+					colourtrans_SET_BG | colourtrans_USE_ECFS,
+					os_ACTION_OVERWRITE, 0);
+				os_clg();
+			}
+      			more = wimp_get_rectangle(redraw);
+      		}
+      	}
 }
 
 
@@ -263,6 +282,7 @@ void gui_window_update_box(gui_window *g, const union content_msg_data *data)
 					update.clip.x1 - 1, update.clip.y1 - 1,
 					g->scale);
 		} else {
+
 			assert(data->redraw.object);
 			content_redraw(data->redraw.object,
 					update.box.x0 - update.xscroll +
-- 
cgit v1.2.3