diff options
author | Teddy Wing | 2023-11-25 14:34:24 +0100 |
---|---|---|
committer | Teddy Wing | 2023-11-25 14:34:24 +0100 |
commit | 344cd163e15382fb884e266bb986fd195c8cc1c5 (patch) | |
tree | 2333191bf9be7d358a639d26dc9ebd99c3805ba4 | |
parent | 34268da6c018a6c191836fe8c26576030cfde3e6 (diff) | |
download | swextreload-344cd163e15382fb884e266bb986fd195c8cc1c5.tar.bz2 |
swextreload: Remove finished TODOs
-rw-r--r-- | internal/swextreload.go | 9 |
1 files changed, 0 insertions, 9 deletions
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 |