Merge https://github.com/jberkman/WebViewJavascriptBridge into jberkman-json-encoding

Author: marcuswestin

WebViewJavascriptBridge/WebViewJavascriptBridge.h

@@ -8,7 +8,6 @@
 
 #import <Foundation/Foundation.h>
 
-#define kMessageSeparator     @"__WVJB_MESSAGE_SEPERATOR__"
 #define kCustomProtocolScheme @"wvjbscheme"
 #define kQueueHasMessage      @"__WVJB_QUEUE_MESSAGE__"
 

WebViewJavascriptBridge/WebViewJavascriptBridge.js.txt

@@ -5,7 +5,6 @@
 	var receiveMessageQueue = []
 	var messageHandlers = {}
 
-	var MESSAGE_SEPARATOR = '__WVJB_MESSAGE_SEPERATOR__'
 	var CUSTOM_PROTOCOL_SCHEME = 'wvjbscheme'
 	var QUEUE_HAS_MESSAGE = '__WVJB_QUEUE_MESSAGE__'
 
@@ -46,12 +45,12 @@
 			responseCallbacks[callbackId] = responseCallback
 			message['callbackId'] = callbackId
 		}
-		sendMessageQueue.push(JSON.stringify(message))
+		sendMessageQueue.push(message)
 		messagingIframe.src = CUSTOM_PROTOCOL_SCHEME + '://' + QUEUE_HAS_MESSAGE
 	}
 
 	function _fetchQueue() {
-		var messageQueueString = sendMessageQueue.join(MESSAGE_SEPARATOR)
+		var messageQueueString = JSON.stringify(sendMessageQueue)
 		sendMessageQueue = []
 		return messageQueueString
 	}

WebViewJavascriptBridge/WebViewJavascriptBridge.m

@@ -92,6 +92,9 @@ - (void)dealloc {
 }
 
 - (void)_sendData:(NSDictionary *)data responseCallback:(WVJBResponseCallback)responseCallback handlerName:(NSString*)handlerName {
+    if (!data) {
+        data = (NSDictionary *)[NSNull null];
+    }
     NSMutableDictionary* message = [NSMutableDictionary dictionaryWithObject:data forKey:@"data"];
 
     if (responseCallback) {
@@ -138,12 +141,18 @@ - (void)_dispatchMessage:(NSDictionary *)message {
 - (void)_flushMessageQueue {
     NSString *messageQueueString = [_webView stringByEvaluatingJavaScriptFromString:@"WebViewJavascriptBridge._fetchQueue();"];
 
-    NSArray* messages = [messageQueueString componentsSeparatedByString:kMessageSeparator];
-    for (NSString *messageJSON in messages) {
-        [self _log:@"receivd" json:messageJSON];
-        
-        NSDictionary* message = [self _deserializeMessageJSON:messageJSON];
-        
+    id messages = [self _deserializeMessageJSON:messageQueueString];
+    if (![messages isKindOfClass:[NSArray class]]) {
+        NSLog(@"WebViewJavascriptBridge: WARNING: Invalid %@ received: %@", [messages class], messages);
+        return;
+    }
+    for (NSDictionary *message in messages) {
+        if (![message isKindOfClass:[NSDictionary class]]) {
+            NSLog(@"WebViewJavascriptBridge: WARNING: Invalid %@ received: %@", [message class], message);
+            continue;
+        }
+        [self _log:@"receivd" json:message];
+
         NSString* responseId = message[@"responseId"];
         if (responseId) {
             WVJBResponseCallback responseCallback = _responseCallbacks[responseId];
@@ -202,10 +211,13 @@ - (NSDictionary *)_deserializeMessageJSON:(NSString *)messageJSON {
 #endif
 }
 
-- (void)_log:(NSString *)action json:(NSString *)json {
+- (void)_log:(NSString *)action json:(id)json {
     if (!logging) { return; }
-    if (json.length > 500) {
-        NSLog(@"WVJB %@: %@", action, [[json substringToIndex:500] stringByAppendingString:@" [...]"]);
+    if (![json isKindOfClass:[NSString class]]) {
+        json = [self _serializeMessage:json];
+    }
+    if ([json length] > 500) {
+        NSLog(@"WVJB %@: %@ [...]", action, [json substringToIndex:500]);
     } else {
         NSLog(@"WVJB %@: %@", action, json);
     }