Only run the scheduler on null poll events.

Ensure the status bar widget treats its text as const.

Fix erroneous free of status bar text in status bar widget destructor. 
The status bar text is now owned by the core.

Schedule a callback to request a redraw from the Wimp when setting status bar 
text. The previous approach could lead to the Wimp having a long queue of 
redraw events to issue to us, thus preventing any other application being 
scheduled.

The above changes are based on a patch from Rik Griffin.

svn path=/trunk/netsurf/; revision=10126

1 files changed, 10 insertions, 1 deletions

diff --git a/riscos/gui.c b/riscos/gui.c
index fa1158ad5..64276b36b 100644
--- a/riscos/gui.c
+++ b/riscos/gui.c
@@ -959,10 +959,19 @@ void gui_poll(bool active)
 	} else {
 		event = wimp_poll(wimp_MASK_NULL | mask, &block, 0);
 	}
+
 	xhourglass_on();
 	gui_last_poll = clock();
 	ro_gui_handle_event(event, &block);
-	schedule_run();
+
+	/* Only run scheduled callbacks on a null poll 
+	 * We cannot do this in the null event handler, as that may be called 
+	 * from gui_multitask(). Scheduled callbacks must only be run from the 
+	 * top-level.
+	 */
+	if (event == wimp_NULL_REASON_CODE)
+		schedule_run();
+
 	ro_gui_window_update_boxes();
 
 	if (browser_reformat_pending && event == wimp_NULL_REASON_CODE)