Don't auto-register IDLE when gaining connection.

We request a sync in this case anyway, which in turn starts the IDLE
connection anyway. If we're starting IDLE earlier, we do unnecessary
work as the sync will tear it down just a few moments later.

Change-Id: I9b99023fbe1aaab72638f6ef9d29223d9a325b7b

1 files changed, 0 insertions, 14 deletions

diff --git a/provider_src/com/android/email/service/LegacyImapSyncAdapterService.java b/provider_src/com/android/email/service/LegacyImapSyncAdapterService.java
index 1b5a36b61..2cb0e3743 100644
--- a/provider_src/com/android/email/service/LegacyImapSyncAdapterService.java
+++ b/provider_src/com/android/email/service/LegacyImapSyncAdapterService.java
@@ -18,9 +18,6 @@ package com.android.email.service;
 
 import static com.android.emailcommon.Logging.LOG_TAG;
 
-import java.util.concurrent.ExecutorService;
-import java.util.concurrent.Executors;
-
 import android.content.AbstractThreadedSyncAdapter;
 import android.content.ComponentName;
 import android.content.ContentProviderClient;
@@ -48,8 +45,6 @@ public class LegacyImapSyncAdapterService extends PopImapSyncAdapterService {
     // seconds for it to appear. If it takes longer than that, we will fail the sync.
     private static final long MAX_WAIT_FOR_SERVICE_MS = 10 * DateUtils.SECOND_IN_MILLIS;
 
-    private static final ExecutorService sExecutor = Executors.newCachedThreadPool();
-
     private IEmailService mImapService;
 
     private final ServiceConnection mConnection = new ServiceConnection() {
@@ -61,15 +56,6 @@ public class LegacyImapSyncAdapterService extends PopImapSyncAdapterService {
             synchronized (mConnection) {
                 mImapService = IEmailService.Stub.asInterface(binder);
                 mConnection.notify();
-
-                // We need to run this task in the background (not in UI-Thread)
-                sExecutor.execute(new Runnable() {
-                    @Override
-                    public void run() {
-                        final Context context = LegacyImapSyncAdapterService.this;
-                        ImapService.registerAllImapIdleMailboxes(context, mImapService);
-                    }
-                });
             }
         }