From 344cd163e15382fb884e266bb986fd195c8cc1c5 Mon Sep 17 00:00:00 2001
From: Teddy Wing
Date: Sat, 25 Nov 2023 14:34:24 +0100
Subject: swextreload: Remove finished TODOs

---
 internal/swextreload.go | 9 ---------
 1 file changed, 9 deletions(-)

(limited to 'internal')

diff --git a/internal/swextreload.go b/internal/swextreload.go
index 98bff1f..803f39e 100644
--- a/internal/swextreload.go
+++ b/internal/swextreload.go
@@ -42,7 +42,6 @@ func Reload(
 	ctx, cancel := chromedp.NewContext(allocatorContext)
 	defer cancel()
 
-	// TODO: I think get targets once first, and reload all extensions using those targets. Rather than getting targets for each extension reload.
 	targets, err := chromedp.Targets(ctx)
 	if err != nil {
 		return fmt.Errorf("swextreload: can't get targets: %v", err)
@@ -60,10 +59,6 @@ func Reload(
 		if err != nil {
 			return err
 		}
-
-		// TODO: Do the reload of the current page after reloading all
-		// extensions. The current system doesn't work well with multiple
-		// extensions.
 	}
 
 	if shouldReloadTab {
@@ -71,10 +66,6 @@ func Reload(
 
 		var firstExtensionTarget *target.Info
 		for _, target := range targets {
-			// for i := len(targets) - 1; i >= 0; i-- {
-
-			// logDebugf("A target: %#v", target)
-
 			if strings.HasPrefix(target.URL, extensionURL) {
 				firstExtensionTarget = target
 
-- 
cgit v1.2.3