fixed bug when tab id is stored as a string

Author: rydang

src/app/reducers/mainReducer.js

@@ -130,7 +130,7 @@ export default function mainReducer(state, action) {
       });
 
       // only set first tab if current tab is non existent
-      const firstTab = Object.keys(payload)[0];
+      const firstTab = parseInt(Object.keys(payload)[0], 10);
       return {
         ...state,
         currentTab: (currentTab === null) ? firstTab : currentTab,

src/extension/background.js

@@ -16,9 +16,8 @@ function createTabObj() {
 chrome.runtime.onConnect.addListener((port) => {
   bg = port;
 
-  // if snapshots were saved in the snapshots,
   // send it to devtools as soon as connection to devtools is made
-  if (Object.values(tabsObj)[0].snapshots.length > 0) {
+  if (Object.keys(tabsObj).length > 0) {
     bg.postMessage({
       action: 'initialConnectSnapshots',
       payload: tabsObj,
@@ -55,12 +54,7 @@ chrome.runtime.onConnect.addListener((port) => {
       default:
     }
 
-    // Instead of sending the message to the active tab,
-    // now we can send messages to specific tabs that we specify
-    // using tabId
-    // ---------------------------------------------------------------
     chrome.tabs.sendMessage(tabId, msg);
-    // ---------------------------------------------------------------
   });
 });