netsurf: branch master updated. release/3.0-192-g1f14790

NetSurf Browser Project (Commit Mailer) no-reply at netsurf-browser.org
Sat Jun 8 09:52:47 BST 2013


Gitweb links:

...log http://git.netsurf-browser.org/netsurf.git/shortlog/1f14790d47ffb81095d778cbef80cc638518a685
...commit http://git.netsurf-browser.org/netsurf.git/commit/1f14790d47ffb81095d778cbef80cc638518a685
...tree http://git.netsurf-browser.org/netsurf.git/tree/1f14790d47ffb81095d778cbef80cc638518a685

The branch, master has been updated
       via  1f14790d47ffb81095d778cbef80cc638518a685 (commit)
      from  4bd65d429cb68bb1927fe4f3e6de2112d61f8114 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commitdiff http://git.netsurf-browser.org/netsurf.git/commit/?id=1f14790d47ffb81095d778cbef80cc638518a685
commit 1f14790d47ffb81095d778cbef80cc638518a685
Author: Vincent Sanders <vince at kyllikki.org>
Commit: Vincent Sanders <vince at kyllikki.org>

    add handling for recived but no action redirect messages and log before abort. (closes #3614409)

diff --git a/css/css.c b/css/css.c
index 0410f18..8f20504 100644
--- a/css/css.c
+++ b/css/css.c
@@ -643,13 +643,10 @@ nserror nscss_import(hlcache_handle *handle,
 	assert(ctx->css->imports[ctx->index].c == handle);
 
 	switch (event->type) {
-	case CONTENT_MSG_LOADING:
-		break;
-	case CONTENT_MSG_READY:
-		break;
 	case CONTENT_MSG_DONE:
 		error = nscss_import_complete(ctx);
 		break;
+
 	case CONTENT_MSG_ERROR:
 		hlcache_handle_release(handle);
 		ctx->css->imports[ctx->index].c = NULL;
@@ -657,9 +654,19 @@ nserror nscss_import(hlcache_handle *handle,
 		error = nscss_import_complete(ctx);
 		/* Already released handle */
 		break;
+
+	case CONTENT_MSG_LOADING:
+	case CONTENT_MSG_READY:
 	case CONTENT_MSG_STATUS:
+	case CONTENT_MSG_REDIRECT:
+		/* messages content handler will legitamately recive
+		 * but does not need to handle
+		 */
 		break;
+
 	default:
+		/* all other messages are unexpected and fatal */
+		LOG(("Unhandled message type %d", event->type));
 		assert(0);
 	}
 


-----------------------------------------------------------------------

Summary of changes:
 css/css.c |   15 +++++++++++----
 1 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/css/css.c b/css/css.c
index 0410f18..8f20504 100644
--- a/css/css.c
+++ b/css/css.c
@@ -643,13 +643,10 @@ nserror nscss_import(hlcache_handle *handle,
 	assert(ctx->css->imports[ctx->index].c == handle);
 
 	switch (event->type) {
-	case CONTENT_MSG_LOADING:
-		break;
-	case CONTENT_MSG_READY:
-		break;
 	case CONTENT_MSG_DONE:
 		error = nscss_import_complete(ctx);
 		break;
+
 	case CONTENT_MSG_ERROR:
 		hlcache_handle_release(handle);
 		ctx->css->imports[ctx->index].c = NULL;
@@ -657,9 +654,19 @@ nserror nscss_import(hlcache_handle *handle,
 		error = nscss_import_complete(ctx);
 		/* Already released handle */
 		break;
+
+	case CONTENT_MSG_LOADING:
+	case CONTENT_MSG_READY:
 	case CONTENT_MSG_STATUS:
+	case CONTENT_MSG_REDIRECT:
+		/* messages content handler will legitamately recive
+		 * but does not need to handle
+		 */
 		break;
+
 	default:
+		/* all other messages are unexpected and fatal */
+		LOG(("Unhandled message type %d", event->type));
 		assert(0);
 	}
 


-- 
NetSurf Browser



More information about the netsurf-commits mailing list