AG-27583 improve documentation

AdguardTeam · Nov 19, 2024 · 0507368 · 0507368
1 parent 9b7d12e
commit 0507368
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/packages/tswebextension/src/lib/mv2/background/tabs/tabs-cosmetic-injector.ts b/packages/tswebextension/src/lib/mv2/background/tabs/tabs-cosmetic-injector.ts
@@ -107,10 +107,12 @@ export class TabsCosmeticInjector {
              */
             const TAB_PROCESSING_THRESHOLD_MS = 10000;
 
+            // TODO: Instead of this, it’s better to use the runtime.onStartup and runtime.onInstalled
+            // events to inject cosmetics once during the extension's initialization and browser startup.
+            // However, this would require big refactoring of the extension.
             /**
              * This condition prevents applying cosmetic rules to the tab multiple times.
              * Applying them once after the extension's initialization is sufficient.
-             * TODO: it would be better to use chrome.runtime.onStartup and chrome.runtime.onInstalled events.
              */
             if (appContext.startTimeMs && Date.now() - appContext.startTimeMs > TAB_PROCESSING_THRESHOLD_MS) {
                 return;

diff --git a/packages/tswebextension/src/lib/mv3/tabs/tabs-cosmetic-injector.ts b/packages/tswebextension/src/lib/mv3/tabs/tabs-cosmetic-injector.ts
@@ -89,10 +89,12 @@ export class TabsCosmeticInjector {
              */
             const TAB_PROCESSING_THRESHOLD_MS = 10000;
 
+            // TODO: Instead of this, it’s better to use the runtime.onStartup and runtime.onInstalled
+            // events to inject cosmetics once during the extension's initialization and browser startup.
+            // However, this would require big refactoring of the extension.
             /**
              * This condition prevents applying cosmetic rules to the tab multiple times.
              * Applying them once after the extension's initialization is enough.
-             * TODO: it would be better to use chrome.runtime.onStartup and chrome.runtime.onInstalled events.
              */
             if (appContext.startTimeMs && currentTime - appContext.startTimeMs > TAB_PROCESSING_THRESHOLD_MS) {
                 return;