From 17ee5dbc05424b24377b3ee326de39184cfce290 Mon Sep 17 00:00:00 2001 From: Neil Fuller Date: Tue, 18 Jul 2017 14:02:01 +0100 Subject: [PATCH] Remove a TODO associate with starting a thread Switching to an alternative method to run something off thread. Bug: 31008728 Test: Internal automated testing / manual tests Change-Id: Ia33ca29e8d83028a21af1dca22de75dd43eee119 --- .../server/timezone/RulesManagerServiceHelperImpl.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/services/core/java/com/android/server/timezone/RulesManagerServiceHelperImpl.java b/services/core/java/com/android/server/timezone/RulesManagerServiceHelperImpl.java index 767f0e0993ef4..b89ce1cd0f894 100644 --- a/services/core/java/com/android/server/timezone/RulesManagerServiceHelperImpl.java +++ b/services/core/java/com/android/server/timezone/RulesManagerServiceHelperImpl.java @@ -18,6 +18,7 @@ package com.android.server.timezone; import android.content.Context; import android.content.pm.PackageManager; +import android.os.AsyncTask; import android.os.Binder; import android.os.ParcelFileDescriptor; @@ -56,10 +57,9 @@ final class RulesManagerServiceHelperImpl implements PermissionHelper, Executor return true; } - // TODO Wake lock required? + // TODO(nfuller): Wake lock required while running in background thread? @Override public void execute(Runnable runnable) { - // TODO Is there a better way? - new Thread(runnable).start(); + AsyncTask.execute(runnable); } }