add message type parameter to notifyDevtools function to categorize messages

Author: localjo

background.js

@@ -36,10 +36,12 @@ function processBackgroundIncomingMessage(msg) {
 }
 
 // Function to send a message to main.js
-function notifyDevtools(msg) {
-  console.log('Background.js Sending Message', msg);
+function notifyDevtools(msgType, msg) {
+  var packagedMessage = new Object();
+  packagedMessage.msgType = msgType;
+  packagedMessage.msg = msg;
   ports.forEach(function (port) {
-    port.postMessage(msg);
+    port.postMessage(packagedMessage);
   });
 }
 
@@ -54,18 +56,19 @@ function getJsonResource(tabID) {
       if (xhr.readyState === 4 && isSolidus) { // Is complete Solidus response?
         if(xhr.status !== 200){ // Check that Solidus response didn't fail
           errMsg = 'Failed to get Solidus context. Status: ' + xhr.status;
-          notifyDevtools(JSON.parse('{"error":"' + errMsg + '"}'));
+          notifyDevtools('error', errMsg);
         } else {
           try {
             // Send Solidus JSON to devpanel
-            notifyDevtools(JSON.parse(xhr.responseText));
+            notifyDevtools('context', JSON.parse(xhr.responseText));
+            notifyDevtools('status', 'The x-powered-by header is' + xhr.getResponseHeader('X-Powered-By'));
           } catch  (e) {
-            notifyDevtools(JSON.parse('{"error":"' + e + '"}'));
+            notifyDevtools('error', e);
           }
         }
       } else {
         errMsg = 'Looks like you\'re not inspecting a Solidus Page.';
-        notifyDevtools(JSON.parse('{"error":"' + errMsg + '"}'));
+        notifyDevtools('error', errMsg);
       }
     };
     xhr.send();

views/devpanel.js

@@ -12,23 +12,25 @@ var inspector;
 
 function processMainIncomingMessage(msg) {
   console.log('Devpanel Processing Message', msg);
-  if (msg.hasOwnProperty('page')) {
+  if (msg.msgType === 'context') {
     displayMessage('Looks like a Solidus page!');
     //Check if there is an initialized InspectorJSON that hasn't been destroyed
     if ((inspector instanceof InspectorJSON) && (inspector.page)) {
-      inspector.view(msg);
+      inspector.view(msg.msg);
     } else {
       inspector = new InspectorJSON({
         element: 'pagecontext',
-        url: msg.url.path,
-        json: msg
+        url: msg.msg.url.path,
+        json: msg.msg
       });
     }
-  } else if (msg.hasOwnProperty('error')) {
+  } else if (msg.msgType === 'error') {
     if (inspector instanceof InspectorJSON) {
       inspector.destroy();
     }
-    displayMessage(msg.error);
+    displayMessage(msg.msg);
+  } else if (msg.msgType === 'status') {
+    displayMessage(msg.msg);
   } else {
     console.log('Message Not Processed', msg);
   }