summaryrefslogtreecommitdiff
path: root/desktop/browser.c
diff options
context:
space:
mode:
authorJohn Mark Bell <jmb@netsurf-browser.org>2007-03-11 22:08:57 +0000
committerJohn Mark Bell <jmb@netsurf-browser.org>2007-03-11 22:08:57 +0000
commit9cf51227979c84a9ed74c339290c3f65f0d58c4f (patch)
treea0c2a9fd6b328eca1dfbdf17989b5ef175427124 /desktop/browser.c
parent58c217cbeeca8bac6c5edd3d156b1f5bbefed3d3 (diff)
downloadnetsurf-9cf51227979c84a9ed74c339290c3f65f0d58c4f.tar.gz
netsurf-9cf51227979c84a9ed74c339290c3f65f0d58c4f.tar.bz2
Content handlers should not call warn_user - they should broadcast the
error using content_broadcast and leave it to the content owner(s) to decide what to do about it. Only use warn_user for top-level contents. svn path=/trunk/netsurf/; revision=3204
Diffstat (limited to 'desktop/browser.c')
-rw-r--r--desktop/browser.c6
1 files changed, 5 insertions, 1 deletions
diff --git a/desktop/browser.c b/desktop/browser.c
index 0273917bc..54a112f27 100644
--- a/desktop/browser.c
+++ b/desktop/browser.c
@@ -458,7 +458,11 @@ void browser_window_callback(content_msg msg, struct content *c,
case CONTENT_MSG_ERROR:
browser_window_set_status(bw, data.error);
- warn_user(data.error, 0);
+
+ /* Only warn the user about errors in top-level windows */
+ if (bw->browser_window_type == BROWSER_WINDOW_NORMAL)
+ warn_user(data.error, 0);
+
if (c == bw->loading_content)
bw->loading_content = 0;
else if (c == bw->current_content) {