[log] simplify

Remove onSuccess, just make sure loading indicator is hidden after
setting up new table model.

platform/vcs-log/impl/src/com/intellij/vcs/log/data/VcsLogFilterer.java

@@ -90,13 +90,12 @@ public class VcsLogFilterer {
     });
   }
 
-  public void requestVcs(@NotNull Collection<VcsLogFilter> filters, final Runnable onSuccess) {
+  public void requestVcs(@NotNull Collection<VcsLogFilter> filters) {
     myLogDataHolder.getFilteredDetailsFromTheVcs(filters, new Consumer<List<VcsFullCommitDetails>>() {
       @Override
       public void consume(List<VcsFullCommitDetails> details) {
         myUI.setModel(new NoGraphTableModel(myUI, details, myLogDataHolder.getDataPack().getRefsModel(), false));
         myUI.updateUI();
-        onSuccess.run();
       }
     });
   }

platform/vcs-log/impl/src/com/intellij/vcs/log/ui/VcsLogUI.java

@@ -95,6 +95,7 @@ public class VcsLogUI {
     if (previousModel instanceof AbstractVcsLogTableModel) { // initially it is an empty DefaultTableModel
       restoreSelection(table, (AbstractVcsLogTableModel)previousModel, selectedRows, model);
     }
+    table.setPaintBusy(false);
   }
 
   private static void restoreSelection(@NotNull VcsLogGraphTable table, @NotNull AbstractVcsLogTableModel previousModel,

platform/vcs-log/impl/src/com/intellij/vcs/log/ui/tables/NoGraphTableModel.java

@@ -67,12 +67,7 @@ public class NoGraphTableModel extends AbstractVcsLogTableModel<CommitCell, Hash
     }
 
     myUi.getTable().setPaintBusy(true);
-    myUi.getFilterer().requestVcs(myUi.collectFilters(), new Runnable() {
-      @Override
-      public void run() {
-        myUi.getTable().setPaintBusy(false);
-      }
-    });
+    myUi.getFilterer().requestVcs(myUi.collectFilters());
     myAllowLoadingMoreRequest = false; // Don't send the request to VCS twice
   }