Committing edits made during code review.

Co-authored-by: Chris LeBrett <100822842+fscgolden@users.noreply.github.com>
Co-authored-by: Robby Tipton <100332566+RobbyTipton@users.noreply.github.com>
Co-authored-by: Joseph Park <10874783+joeepark@users.noreply.github.com>
Co-authored-by: David Kim <101825171+codejunkie7@users.noreply.github.com>
Co-authored-by: Kevin HoEun Lee <14950124+khobread@users.noreply.github.com>

Author: 5 people

src/extension/background.js

@@ -124,7 +124,6 @@ function changeCurrLocation(tabObj, rootNode, index, name) {
 
 // Establishing incoming connection with devtools.
 chrome.runtime.onConnect.addListener(port => {
-  console.log('Hello from line 126');
   // port is one end of the connection - an object
   // push every port connected to the ports array
   portsArr.push(port);
@@ -200,14 +199,10 @@ chrome.runtime.onConnect.addListener(port => {
         tabsObj[tabId].mode.persist = payload;
         return true;
       case 'launchContentScript':
-        // !!! in Manifest Version 3 this will need to be changed to the commented out code below !!!
-        console.log('Youre here');
         chrome.scripting.executeScript({
           target: { tabId },
           files: ['bundles/content.bundle.js'],
         });
-        // This line below will need to be removed
-        // chrome.tabs.executeScript(tabId, { file: 'bundles/content.bundle.js' });
         return true;
       case 'jumpToSnap':
         chrome.tabs.sendMessage(tabId, msg);
@@ -223,7 +218,6 @@ chrome.runtime.onConnect.addListener(port => {
 
 // background.js listening for a message from contentScript.js
 chrome.runtime.onMessage.addListener((request, sender, sendResponse) => {
-  console.log('Hello from line 224');
   // AUTOMATIC MESSAGE SENT BY CHROME WHEN CONTENT SCRIPT IS FIRST LOADED: set Content
   if (request.type === 'SIGN_CONNECT') {
     return true;
@@ -286,15 +280,13 @@ chrome.runtime.onMessage.addListener((request, sender, sendResponse) => {
     // This injects a script into the app that you're testing Reactime on,
     // so that Reactime's backend files can communicate with the app's DOM.
     case 'injectScript': {
-      console.log('Hello from line 287');
-
-      const injectScript = (file, tabId) => {
+      const injectScript = (file, tab) => {
         const htmlBody = document.getElementsByTagName('body')[0];
         const script = document.createElement('script');
         script.setAttribute('type', 'text/javascript');
         script.setAttribute('src', file);
         // eslint-disable-next-line prefer-template
-        document.title = tabId + '-' + document.title;
+        document.title = tab + '-' + document.title;
         htmlBody.appendChild(script);
       };
 
@@ -428,7 +420,6 @@ chrome.runtime.onInstalled.addListener(() => {
     title: 'Reactime',
     contexts: ['page', 'selection', 'image', 'link'],
   });
-  console.log('Context Menu Created');
 });
 
 // when context menu is clicked, listen for the menuItemId,

src/extension/build/manifest.json

@@ -1,6 +1,6 @@
 {
   "name": "Reactime",
-  "version": "13.0.0",
+  "version": "14.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,

src/extension/contentScript.ts

@@ -13,8 +13,6 @@ let firstMessage = true;
 let isRecording = true;
 
 window.addEventListener('message', msg => {
-  console.log('Hello from line 16');
-  console.log('msg is', msg.data);
   // Event listener runs constantly based on actions
   // recorded on the test application from backend files (linkFiber.ts).
   // Background.js has a listener that includes switch cases, depending on
@@ -84,5 +82,4 @@ getCLS(gatherMetrics);
 
 // Send message to background.js for injecting the initial script
 // into the app's DOM.
-console.log('Hello from line 86');
 chrome.runtime.sendMessage({ action: 'injectScript' });