Index: sync/android/java/src/org/chromium/sync/ModelTypeHelper.java |
diff --git a/sync/android/java/src/org/chromium/sync/ModelTypeHelper.java b/sync/android/java/src/org/chromium/sync/ModelTypeHelper.java |
new file mode 100644 |
index 0000000000000000000000000000000000000000..37daf3f2f8147e68c1cc12d0a46edcb36d5987a9 |
--- /dev/null |
+++ b/sync/android/java/src/org/chromium/sync/ModelTypeHelper.java |
@@ -0,0 +1,111 @@ |
+// Copyright 2015 The Chromium Authors. All rights reserved. |
+// Use of this source code is governed by a BSD-style license that can be |
+// found in the LICENSE file. |
+ |
+package org.chromium.sync; |
+ |
+import com.google.ipc.invalidation.external.client.types.ObjectId; |
+import com.google.protos.ipc.invalidation.Types; |
+ |
+import org.chromium.base.FieldTrialList; |
+import org.chromium.base.VisibleForTesting; |
+import org.chromium.base.annotations.JNINamespace; |
+import org.chromium.base.library_loader.LibraryLoader; |
+ |
+import java.util.Collection; |
+import java.util.HashSet; |
+import java.util.Set; |
+ |
+/** |
+ * Helper methods for dealing with ModelTypes. |
+ */ |
+@JNINamespace("syncer") |
+public class ModelTypeHelper { |
+ private static final String TAG = "ModelTypeHelper"; |
+ |
+ private static final int[] NON_INVALIDATION_TYPES_ARRAY = new int[] { |
+ ModelType.PROXY_TABS |
+ }; |
+ |
+ // Convenience sets for checking whether a type can have invalidations. Some ModelTypes |
+ // such as PROXY_TABS are not real types and can't be registered. Initializing these |
+ // once reduces toString() calls in the isInvalidationType() method. |
+ private static Set<String> sNonInvalidationTypes = null; |
+ private static Set<String> sNonInvalidationTypesExperiment = null; |
+ |
+ /** |
+ * Initializes the non-invalidation sets. Called lazily the first time they're needed. |
+ */ |
+ private static void initNonInvalidationTypes() { |
nyquist
2015/08/03 20:07:10
This doesn't seem thread safe. Could you at least
maxbogue
2015/08/04 19:43:54
Decided to make it actually thread safe; PTAL.
|
+ sNonInvalidationTypes = new HashSet<String>(); |
+ for (int i = 0; i < NON_INVALIDATION_TYPES_ARRAY.length; i++) { |
+ sNonInvalidationTypes.add(toString(NON_INVALIDATION_TYPES_ARRAY[i])); |
+ } |
+ sNonInvalidationTypesExperiment = new HashSet<String>(); |
+ sNonInvalidationTypesExperiment.add(toString(ModelType.SESSIONS)); |
+ sNonInvalidationTypesExperiment.add(toString(ModelType.FAVICON_TRACKING)); |
+ sNonInvalidationTypesExperiment.add(toString(ModelType.FAVICON_IMAGES)); |
+ } |
+ |
+ /** |
+ * Checks whether a type is allowed to register for invalidations. |
+ */ |
+ private static boolean isInvalidationType(String modelType) { |
+ if (sNonInvalidationTypes == null) { |
nyquist
2015/08/03 20:07:10
Same about threading here.
maxbogue
2015/08/04 19:43:54
Done.
|
+ initNonInvalidationTypes(); |
+ } |
+ if (sNonInvalidationTypesExperiment.contains(modelType) && LibraryLoader.isInitialized()) { |
+ return !FieldTrialList.findFullName("AndroidSessionNotifications").equals("Disabled"); |
Nicolas Zea
2015/08/04 17:04:53
To be clear, the experiment being "on" in this cas
maxbogue
2015/08/04 20:06:12
Yep.
|
+ } |
+ return !sNonInvalidationTypes.contains(modelType); |
+ } |
+ |
+ /** |
+ * Converts a model type string into an ObjectId. |
+ * |
+ * If the model type is not an invalidation type, this function uses the string "NULL". |
+ */ |
+ private static ObjectId toObjectId(String modelType) { |
+ String objectIdString = isInvalidationType(modelType) ? modelType : "NULL"; |
+ return ObjectId.newInstance(Types.ObjectSource.CHROME_SYNC, objectIdString.getBytes()); |
+ } |
+ |
+ @VisibleForTesting |
+ public static ObjectId toObjectId(int modelType) { |
+ return toObjectId(toString(modelType)); |
+ } |
+ |
+ /** |
+ * Converts a model type to its string representation using JNI. |
+ * This is the value that is stored in the invalidation preferences. |
+ * |
+ * @param modelType the model type to convert to a string. |
+ * @return the string representation of the model type constant. |
+ */ |
+ public static String toString(Integer modelType) { |
nyquist
2015/08/03 20:07:10
Nit: Could this be 'int'?
maxbogue
2015/08/04 19:43:54
Yes, thanks!
|
+ // Because PROXY_TABS isn't an invalidation type, it doesn't have a string from native, |
+ // but for backwards compatibility we need to keep its pref value the same as the old |
+ // ModelType enum name value. |
+ if (modelType == ModelType.PROXY_TABS) { |
+ return "PROXY_TABS"; |
+ } |
+ return nativeModelTypeToNotificationType(modelType); |
+ } |
+ |
+ /** |
+ * Converts a set of {@link String} model types to a set of {@link ObjectId}. |
+ * |
+ * This strips out any {@link ModelType} that is not an invalidation type. |
+ */ |
+ public static Set<ObjectId> modelTypeStringsToObjectIds(Collection<String> modelTypes) { |
+ Set<ObjectId> objectIds = new HashSet<ObjectId>(); |
+ for (String modelType : modelTypes) { |
+ if (isInvalidationType(modelType)) { |
+ objectIds.add(toObjectId(modelType)); |
+ } |
+ } |
+ return objectIds; |
+ } |
+ |
+ private static native String nativeModelTypeToNotificationType(int modelType); |
+} |