From dd8e3c0f5496990bb3626a3f0448063a0b5f9b72 Mon Sep 17 00:00:00 2001
From: Jim Robinson <jrobinso@broadinstitute.org>
Date: Fri, 12 Jun 2020 19:24:33 -0700
Subject: [PATCH] Remove oauth check on session load -- this will happen
 downstream, if neccessary

---
 src/main/java/org/broad/igv/ui/IGV.java | 21 +++++++++------------
 1 file changed, 9 insertions(+), 12 deletions(-)

diff --git a/src/main/java/org/broad/igv/ui/IGV.java b/src/main/java/org/broad/igv/ui/IGV.java
index f6b61307cb..ea536d8d7b 100644
--- a/src/main/java/org/broad/igv/ui/IGV.java
+++ b/src/main/java/org/broad/igv/ui/IGV.java
@@ -48,7 +48,6 @@
 import org.broad.igv.feature.Range;
 import org.broad.igv.feature.*;
 import org.broad.igv.feature.genome.*;
-import org.broad.igv.google.OAuthUtils;
 import org.broad.igv.lists.GeneList;
 import org.broad.igv.prefs.IGVPreferences;
 import org.broad.igv.prefs.PreferencesEditor;
@@ -477,6 +476,7 @@ public void run() {
                     showLoadedTrackCount();
                     IGV.this.repaint();
                 }
+
                 public String getName() {
                     return "Load Tracks";
                 }
@@ -1082,10 +1082,6 @@ public void doRestoreSession(final String sessionPath,
                                  final String locus,
                                  final boolean merge) {
 
-        // check to see if any files in session file are on protected (oauth) server. If
-        // so, make sure user is logged into server before -proceeding
-
-        OAuthUtils.getInstance().getProvider().checkServerLogin(sessionPath);
         Runnable runnable = () -> restoreSessionSynchronous(sessionPath, locus, merge);
         LongRunningTask.submit(runnable);
     }
@@ -2371,7 +2367,6 @@ private synchronized void repaint(final JComponent component) {
                     if (track.isReadyToPaint(frame) == false) {
                         if (Globals.isBatch()) {
                             track.load(frame);
-                            component.paintImmediately(component.getBounds());
                         } else {
                             futures.add(CompletableFuture.runAsync(() -> {
                                 track.load(frame);
@@ -2382,7 +2377,14 @@ private synchronized void repaint(final JComponent component) {
             }
         }
 
-        if (futures.size() > 0) {
+        if (Globals.isBatch()) {
+            component.paintImmediately(component.getBounds());
+        } else if (futures.size() == 0) {
+            UIUtilities.invokeOnEventThread(() -> {
+                component.repaint();
+
+            });
+        } else {
             final CompletableFuture[] futureArray = futures.toArray(new CompletableFuture[futures.size()]);
             WaitCursorManager.CursorToken token = WaitCursorManager.showWaitCursor();
             CompletableFuture.allOf(futureArray).thenApplyAsync(future -> {
@@ -2394,11 +2396,6 @@ private synchronized void repaint(final JComponent component) {
                 });
                 return null;
             });
-        } else {
-            UIUtilities.invokeOnEventThread(() -> {
-                component.repaint();
-
-            });
         }
     }