commit 01d686b85ebc5f7eea1a43db33a2f3e3f3becc3e
author Neil Fuller <nfuller@google.com> Mon Jul 31 12:51:05 2017 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Mon Jul 31 12:51:05 2017 +0000
tree 5763bd8c52e01bc3b92d6741e7f15a042a143fc2
parent f60a38e152164b1d2df96f6801074b10ff08b3be
parent 6f14377e21367ebf8b248ac3449341f73750ae34

Merge "Remove redundant TODOs from PackageTracker"

services/core/java/com/android/server/timezone/PackageTracker.java

diff --git a/services/core/java/com/android/server/timezone/PackageTracker.java b/services/core/java/com/android/server/timezone/PackageTracker.java
index 9b49996..24e0fe4 100644
--- a/services/core/java/com/android/server/timezone/PackageTracker.java
+++ b/services/core/java/com/android/server/timezone/PackageTracker.java
@@ -173,8 +173,6 @@
             throw logAndThrowRuntimeException("Could not determine update app package details for "
                     + mUpdateAppPackageName, e);
         }
-        // TODO(nfuller) Consider permission checks. While an updated system app retains permissions
-        // obtained by the system version it's not clear how to check them.
         Slog.d(TAG, "Update app " + mUpdateAppPackageName + " is valid.");
 
         // Validate the data application package.
@@ -187,8 +185,6 @@
             throw logAndThrowRuntimeException("Could not determine data app package details for "
                     + mDataAppPackageName, e);
         }
-        // TODO(nfuller) Consider permission checks. While an updated system app retains permissions
-        // obtained by the system version it's not clear how to check them.
         Slog.d(TAG, "Data app " + mDataAppPackageName + " is valid.");
     }
 
@@ -466,7 +462,6 @@
                     + TimeZoneRulesDataContract.AUTHORITY);
             return false;
         }
-        // TODO(nfuller) Add any permissions checks needed.
         return true;
     }