refactor: adds comments and breaks donw code info smaller fucntions

Signed-off-by: hitg00 <[email protected]>
hitgo00 · Apr 13, 2021 · 4526f5e · 4526f5e
1 parent 1ace6c6
commit 4526f5e
Show file tree

Hide file tree

Showing 3 changed files with 67 additions and 13 deletions.
diff --git a/background.js b/background.js
@@ -1,19 +1,68 @@
-chrome.runtime.onConnectExternal.addListener((port) => {
- let strInfo, cpuInfo, memInfo;
- port.onMessage.addListener((message) => {
+//to fetch cpu stats
+const getCpuInfo = async () => {
+ return new Promise((resolve, reject) => {
  chrome.system.cpu.getInfo((info) => {
- cpuInfo = info;
+ if (info) {
+ resolve(info);
+ } else {
+ reject("Unable to get cpu information");
+ }
  });
+ });
+};
+
+//to fetch memory information
+const getMemoryInfo = async () => {
+ return new Promise((resolve, reject) => {
  chrome.system.memory.getInfo((info) => {
- memInfo = info;
+ if (info) {
+ resolve(info);
+ } else {
+ reject("Unable to get memory information");
+ }
  });
+ });
+};
+
+//to fetch storage information
+const getStorageInfo = async () => {
+ return new Promise((resolve, reject) => {
  chrome.system.storage.getInfo((info) => {
- strInfo = info;
- });
- port.postMessage({
- strInfo,
- cpuInfo,
- memInfo,
+ if (info) {
+ resolve(info);
+ } else {
+ reject("Unable to get storage information");
+ }
  });
  });
+};
+
+const fetchDataFromSystem = () => {
+ return new Promise((resolve, reject) => {
+ Promise.all([getCpuInfo(), getMemoryInfo(), getStorageInfo()]).then(
+ (values) => {
+ const cpuInfo = values[0];
+ const memInfo = values[1];
+ const strInfo = values[2];
+ resolve({
+ strInfo,
+ cpuInfo,
+ memInfo,
+ });
+ }
+ );
+ });
+};
+
+//listen to incoming port connection requests
+chrome.runtime.onConnectExternal.addListener((port) => {
+ port.onMessage.addListener(async (message) => {
+ try {
+ const systemInfo = await fetchDataFromSystem();
+ console.log(systemInfo);
+ port.postMessage(systemInfo);
+ } catch (error) {
+ port.postMessage(error);
+ }
+ });
 });
diff --git a/manifest.json b/manifest.json
@@ -10,7 +10,11 @@
  "default_popup": "popup.html"
  },
  "externally_connectable": {
- "matches": ["http://localhost:3000/*", "https://telemetry-pwa.vercel.app/*"]
+ "matches": [
+ "http://localhost:3000/*",
+ "https://telemetry-pwa.vercel.app/*",
+ "http://localhost:3001/*"
+ ]
  },
  "permissions": [
  "system.cpu",

diff --git a/popup.html b/popup.html
@@ -10,7 +10,8 @@
 
 <body>
  <h1 style="border: 1px solid black; width: 512px;">This is a simple Chrome Extension</h1>
- <h3>It is responsible for sending System Info to </br><a href="https://telemetry-pwa.vercel.app/">Telemetry Web
+ <h3>It is responsible for sending System Info to </br><a target="_blank"
+ href="https://telemetry-pwa.vercel.app/">Telemetry Web
  App</a></h3>
 </body>