Clean up code

aws · Feb 27, 2025 · 9c28500 · 9c28500
1 parent eeb5664
commit 9c28500
Show file tree

Hide file tree

Showing 7 changed files with 8 additions and 7 deletions.
diff --git a/plugin/src/software/aws/toolkits/eclipse/amazonq/broker/events/AmazonQViewType.java b/plugin/src/software/aws/toolkits/eclipse/amazonq/broker/events/AmazonQViewType.java
@@ -4,6 +4,8 @@
 package software.aws.toolkits.eclipse.amazonq.broker.events;
 
 public enum AmazonQViewType {
+
     TOOLKIT_LOGIN_VIEW, CHAT_VIEW, DEPENDENCY_MISSING_VIEW, RE_AUTHENTICATE_VIEW, CHAT_ASSET_MISSING_VIEW,
     LSP_STARTUP_FAILED_VIEW
+
 }
diff --git a/plugin/src/software/aws/toolkits/eclipse/amazonq/lsp/LspStartupActivity.java b/plugin/src/software/aws/toolkits/eclipse/amazonq/lsp/LspStartupActivity.java
@@ -19,6 +19,7 @@
 import org.eclipse.ui.PlatformUI;
 
 import software.aws.toolkits.eclipse.amazonq.plugin.Activator;
+import software.aws.toolkits.eclipse.amazonq.providers.browser.AmazonQBrowserProvider;
 import software.aws.toolkits.eclipse.amazonq.telemetry.ToolkitTelemetryProvider;
 import software.aws.toolkits.eclipse.amazonq.telemetry.metadata.ExceptionMetadata;
 import software.aws.toolkits.eclipse.amazonq.util.AutoTriggerDocumentListener;
@@ -69,6 +70,8 @@ protected IStatus run(final IProgressMonitor monitor) {
 
         updateCheckJob.setPriority(Job.DECORATE);
         updateCheckJob.schedule();
+
+        new AmazonQBrowserProvider().publishBrowserCompatibilityState();
     }
 
     private void launchWebview() {

diff --git a/plugin/src/software/aws/toolkits/eclipse/amazonq/plugin/Activator.java b/plugin/src/software/aws/toolkits/eclipse/amazonq/plugin/Activator.java
@@ -12,7 +12,6 @@
 import software.aws.toolkits.eclipse.amazonq.configuration.PluginStore;
 import software.aws.toolkits.eclipse.amazonq.lsp.auth.DefaultLoginService;
 import software.aws.toolkits.eclipse.amazonq.lsp.auth.LoginService;
-import software.aws.toolkits.eclipse.amazonq.providers.browser.AmazonQBrowserProvider;
 import software.aws.toolkits.eclipse.amazonq.providers.lsp.LspProvider;
 import software.aws.toolkits.eclipse.amazonq.providers.lsp.LspProviderImpl;
 import software.aws.toolkits.eclipse.amazonq.telemetry.service.DefaultTelemetryService;
@@ -50,7 +49,6 @@ public Activator() {
                 .initializeOnStartUp()
                 .build();
         codeReferenceLoggingService = DefaultCodeReferenceLoggingService.getInstance();
-        new AmazonQBrowserProvider().publishBrowserCompatibilityState();
     }
 
     @Override

diff --git a/.../src/software/aws/toolkits/eclipse/amazonq/providers/assets/ChatWebViewAssetProvider.java b/.../src/software/aws/toolkits/eclipse/amazonq/providers/assets/ChatWebViewAssetProvider.java
@@ -103,7 +103,7 @@ public void completed(final ProgressEvent event) {
     }
 
     @Override
-    protected Optional<String> getContent() {
+    public Optional<String> getContent() {
         Optional<String> content = resolveContent();
         Activator.getEventBroker().post(ChatWebViewAssetState.class,
                 content.isPresent() ? ChatWebViewAssetState.RESOLVED : ChatWebViewAssetState.DEPENDENCY_MISSING);

diff --git a/...tware/aws/toolkits/eclipse/amazonq/providers/assets/ToolkitLoginWebViewAssetProvider.java b/...tware/aws/toolkits/eclipse/amazonq/providers/assets/ToolkitLoginWebViewAssetProvider.java
@@ -61,7 +61,7 @@ public Object function(final Object[] arguments) {
     }
 
     @Override
-    protected Optional<String> getContent() {
+    public Optional<String> getContent() {
         Optional<String> content = resolveContent();
         Activator.getEventBroker().post(ToolkitLoginWebViewAssetState.class,
                 content.isPresent() ? ToolkitLoginWebViewAssetState.RESOLVED

diff --git a/plugin/src/software/aws/toolkits/eclipse/amazonq/providers/assets/WebViewAssetProvider.java b/plugin/src/software/aws/toolkits/eclipse/amazonq/providers/assets/WebViewAssetProvider.java
@@ -9,7 +9,7 @@
 
 public abstract class WebViewAssetProvider {
 
-    public abstract void injectAssets(final Browser browser);
+    public abstract void injectAssets(Browser browser);
 
     public abstract Optional<String> getContent();
 

diff --git a/plugin/src/software/aws/toolkits/eclipse/amazonq/views/ToolkitLoginWebview.java b/plugin/src/software/aws/toolkits/eclipse/amazonq/views/ToolkitLoginWebview.java
@@ -20,8 +20,6 @@ public final class ToolkitLoginWebview extends AmazonQView {
 
     private final WebViewAssetProvider webViewAssetProvider;
 
-    private boolean isViewVisible = false;
-
     public ToolkitLoginWebview() {
         super();
         webViewAssetProvider = new ToolkitLoginWebViewAssetProvider();