fixing bug and adding some comment

Author: d28601581

src/backend/linkFiber.ts

@@ -571,17 +571,37 @@ export default (snap: Snapshot, mode: Mode): (() => void) => {
     document.addEventListener('visibilitychange', onVisibilityChange);
 
     if (reactInstance && reactInstance.version) {
+      console.log(devTools);
+      console.log(devTools.onCommitFiberRoot);
       fiberRoot = devTools.getFiberRoots(1).values().next().value;
-      devTools.onCommitFiberRoot = (function (original) {
+      // original is a function
+      // devTools.onCommitFiberRoot = (function (original) {
+      //   return function (...args) {
+      //     // eslint-disable-next-line prefer-destructuring
+      //     fiberRoot = args[1];
+      //     if (doWork) {
+      //       throttledUpdateSnapshot();
+      //     }
+      //     return original(...args);
+      //   };
+      // }(devTools.onCommitFiberRoot));
+
+      // React has inherent methods that are called with react fiber
+      // we attach new functionality without compromising the original work that onCommitFiberRoot does
+      const addOneMoreStep = function (original) {
         return function (...args) {
           // eslint-disable-next-line prefer-destructuring
           fiberRoot = args[1];
+          // this is the additional functionality we added
           if (doWork) {
             throttledUpdateSnapshot();
           }
+          // after our added work is completed we invoke the original function
           return original(...args);
         };
-      }(devTools.onCommitFiberRoot));
+      };
+      devTools.onCommitFiberRoot = addOneMoreStep(devTools.onCommitFiberRoot);
+
       throttledUpdateSnapshot(); // only runs on start up
     }
   };

src/extension/background.js

@@ -122,7 +122,7 @@ function changeCurrLocation(tabObj, rootNode, index, name) {
   }
 }
 
-// Establishing incoming connection with devtools.
+// Establishing incoming connection with Reactime.
 chrome.runtime.onConnect.addListener(port => {
   // port is one end of the connection - an object
   // push every port connected to the ports array
@@ -222,7 +222,7 @@ chrome.runtime.onMessage.addListener((request, sender, sendResponse) => {
   if (request.type === 'SIGN_CONNECT') {
     return true;
   }
-
+  console.log(sender, '<-- sender');
   const tabTitle = sender.tab.title;
   const tabId = sender.tab.id;
   const {
@@ -286,7 +286,7 @@ chrome.runtime.onMessage.addListener((request, sender, sendResponse) => {
         script.setAttribute('type', 'text/javascript');
         script.setAttribute('src', file);
         // eslint-disable-next-line prefer-template
-        document.title = tab + '-' + document.title;
+        // document.title = tab + '-' + document.title; // error of injecting random number
         htmlBody.appendChild(script);
       };
 

src/extension/build/manifest.json

@@ -1,6 +1,6 @@
 {
   "name": "Reactime",
-  "version": "14.0.0",
+  "version": "15.0.0",
   "devtools_page": "devtools.html",
   "description": "A Chrome extension that helps debug React applications by memorizing the state of components with every render.",
   "manifest_version": 3,