summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGeorge Lin <giolin@google.com>2023-09-21 19:41:40 +0000
committerGeorge Lin <giolin@google.com>2023-09-21 21:12:53 +0000
commit64160c68e0a272ff571529a1b3d846f85859255e (patch)
treecb859b8c95ea3c7c89cbbe70f625cc04b9484279
parent14f144a0d128e610399f1ae14b7f0c8c57399a1f (diff)
downloadThemePicker-64160c68e0a272ff571529a1b3d846f85859255e.tar.gz
Move logging related classes to logging package (1/3)
Test: Build succeeded Bug: 299659307 Change-Id: I9698c4e7df5bbb42b175431e6d5c972d929c66d4
-rw-r--r--src/com/android/customization/model/grid/GridOptionsManager.java2
-rw-r--r--src/com/android/customization/module/ThemePickerInjector.kt4
-rw-r--r--src/com/android/customization/module/logging/StatsLogUserEventLogger.java (renamed from src/com/android/customization/module/StatsLogUserEventLogger.java)5
-rw-r--r--src/com/android/customization/module/logging/ThemesUserEventLogger.java (renamed from src/com/android/customization/module/ThemesUserEventLogger.java)6
-rw-r--r--tests/common/src/com/android/customization/testing/TestCustomizationInjector.kt4
-rw-r--r--tests/common/src/com/android/customization/testing/TestThemesUserEventLogger.java2
-rw-r--r--tests/robotests/src/com/android/customization/model/grid/GridOptionsManagerTest.java2
7 files changed, 15 insertions, 10 deletions
diff --git a/src/com/android/customization/model/grid/GridOptionsManager.java b/src/com/android/customization/model/grid/GridOptionsManager.java
index 78dbb5b2..bd24cf5f 100644
--- a/src/com/android/customization/model/grid/GridOptionsManager.java
+++ b/src/com/android/customization/model/grid/GridOptionsManager.java
@@ -27,7 +27,7 @@ import androidx.lifecycle.LiveData;
import com.android.customization.model.CustomizationManager;
import com.android.customization.module.CustomizationInjector;
-import com.android.customization.module.ThemesUserEventLogger;
+import com.android.customization.module.logging.ThemesUserEventLogger;
import com.android.wallpaper.R;
import com.android.wallpaper.module.InjectorProvider;
import com.android.wallpaper.util.PreviewUtils;
diff --git a/src/com/android/customization/module/ThemePickerInjector.kt b/src/com/android/customization/module/ThemePickerInjector.kt
index c7c9c4c9..ec06733f 100644
--- a/src/com/android/customization/module/ThemePickerInjector.kt
+++ b/src/com/android/customization/module/ThemePickerInjector.kt
@@ -36,6 +36,8 @@ import com.android.customization.model.themedicon.ThemedIconSwitchProvider
import com.android.customization.model.themedicon.data.repository.ThemeIconRepository
import com.android.customization.model.themedicon.domain.interactor.ThemedIconInteractor
import com.android.customization.model.themedicon.domain.interactor.ThemedIconSnapshotRestorer
+import com.android.customization.module.logging.StatsLogUserEventLogger
+import com.android.customization.module.logging.ThemesUserEventLogger
import com.android.customization.picker.clock.data.repository.ClockPickerRepositoryImpl
import com.android.customization.picker.clock.data.repository.ClockRegistryProvider
import com.android.customization.picker.clock.domain.interactor.ClockPickerInteractor
@@ -71,8 +73,8 @@ import com.android.wallpaper.dispatchers.MainDispatcher
import com.android.wallpaper.model.WallpaperColorsRepository
import com.android.wallpaper.module.CustomizationSections
import com.android.wallpaper.module.FragmentFactory
-import com.android.wallpaper.module.UserEventLogger
import com.android.wallpaper.module.WallpaperPicker2Injector
+import com.android.wallpaper.module.logging.UserEventLogger
import com.android.wallpaper.picker.CustomizationPickerActivity
import com.android.wallpaper.picker.customization.data.content.WallpaperClientImpl
import com.android.wallpaper.picker.customization.data.repository.WallpaperRepository
diff --git a/src/com/android/customization/module/StatsLogUserEventLogger.java b/src/com/android/customization/module/logging/StatsLogUserEventLogger.java
index 8fbf996c..8ed30ff9 100644
--- a/src/com/android/customization/module/StatsLogUserEventLogger.java
+++ b/src/com/android/customization/module/logging/StatsLogUserEventLogger.java
@@ -13,7 +13,7 @@
* See the License for the specific language governing permissions and
* limitations under the License.
*/
-package com.android.customization.module;
+package com.android.customization.module.logging;
import static com.android.systemui.shared.system.SysUiStatsLog.STYLE_UICHANGED__ACTION__APP_LAUNCHED;
import static com.android.systemui.shared.system.SysUiStatsLog.STYLE_UICHANGED__LAUNCHED_PREFERENCE__LAUNCHED_CROP_AND_SET_ACTION;
@@ -42,9 +42,10 @@ import androidx.annotation.Nullable;
import com.android.customization.model.color.ColorOption;
import com.android.customization.model.grid.GridOption;
-import com.android.wallpaper.module.NoOpUserEventLogger;
+import com.android.customization.module.SysUiStatsLogger;
import com.android.wallpaper.module.WallpaperPreferences;
import com.android.wallpaper.module.WallpaperStatusChecker;
+import com.android.wallpaper.module.logging.NoOpUserEventLogger;
/**
* StatsLog-backed implementation of {@link ThemesUserEventLogger}.
diff --git a/src/com/android/customization/module/ThemesUserEventLogger.java b/src/com/android/customization/module/logging/ThemesUserEventLogger.java
index 2deb2e51..cc16d094 100644
--- a/src/com/android/customization/module/ThemesUserEventLogger.java
+++ b/src/com/android/customization/module/logging/ThemesUserEventLogger.java
@@ -13,11 +13,11 @@
* See the License for the specific language governing permissions and
* limitations under the License.
*/
-package com.android.customization.module;
+package com.android.customization.module.logging;
import com.android.customization.model.color.ColorOption;
import com.android.customization.model.grid.GridOption;
-import com.android.wallpaper.module.UserEventLogger;
+import com.android.wallpaper.module.logging.UserEventLogger;
/**
* Extension of {@link UserEventLogger} that adds ThemePicker specific events.
@@ -32,8 +32,10 @@ public interface ThemesUserEventLogger extends UserEventLogger {
*/
void logColorApplied(int action, ColorOption colorOption);
+ /** */
void logGridSelected(GridOption grid);
+ /** */
void logGridApplied(GridOption grid);
}
diff --git a/tests/common/src/com/android/customization/testing/TestCustomizationInjector.kt b/tests/common/src/com/android/customization/testing/TestCustomizationInjector.kt
index cd587d6d..2717b8a4 100644
--- a/tests/common/src/com/android/customization/testing/TestCustomizationInjector.kt
+++ b/tests/common/src/com/android/customization/testing/TestCustomizationInjector.kt
@@ -5,7 +5,7 @@ import android.content.res.Resources
import androidx.activity.ComponentActivity
import com.android.customization.module.CustomizationInjector
import com.android.customization.module.CustomizationPreferences
-import com.android.customization.module.ThemesUserEventLogger
+import com.android.customization.module.logging.ThemesUserEventLogger
import com.android.customization.picker.clock.domain.interactor.ClockPickerInteractor
import com.android.customization.picker.clock.ui.view.ClockViewFactory
import com.android.customization.picker.clock.ui.viewmodel.ClockCarouselViewModel
@@ -17,7 +17,7 @@ import com.android.customization.picker.color.ui.viewmodel.ColorPickerViewModel
import com.android.customization.picker.quickaffordance.domain.interactor.KeyguardQuickAffordancePickerInteractor
import com.android.systemui.shared.clocks.ClockRegistry
import com.android.wallpaper.model.WallpaperColorsRepository
-import com.android.wallpaper.module.UserEventLogger
+import com.android.wallpaper.module.logging.UserEventLogger
import com.android.wallpaper.testing.TestInjector
import javax.inject.Inject
import javax.inject.Singleton
diff --git a/tests/common/src/com/android/customization/testing/TestThemesUserEventLogger.java b/tests/common/src/com/android/customization/testing/TestThemesUserEventLogger.java
index 26fd22fd..01bb91d9 100644
--- a/tests/common/src/com/android/customization/testing/TestThemesUserEventLogger.java
+++ b/tests/common/src/com/android/customization/testing/TestThemesUserEventLogger.java
@@ -17,7 +17,7 @@ package com.android.customization.testing;
import com.android.customization.model.color.ColorOption;
import com.android.customization.model.grid.GridOption;
-import com.android.customization.module.ThemesUserEventLogger;
+import com.android.customization.module.logging.ThemesUserEventLogger;
import com.android.wallpaper.testing.TestUserEventLogger;
/**
diff --git a/tests/robotests/src/com/android/customization/model/grid/GridOptionsManagerTest.java b/tests/robotests/src/com/android/customization/model/grid/GridOptionsManagerTest.java
index 04ac0241..52c64898 100644
--- a/tests/robotests/src/com/android/customization/model/grid/GridOptionsManagerTest.java
+++ b/tests/robotests/src/com/android/customization/model/grid/GridOptionsManagerTest.java
@@ -24,7 +24,7 @@ import static org.mockito.Mockito.when;
import androidx.annotation.Nullable;
import com.android.customization.model.CustomizationManager.Callback;
-import com.android.customization.module.ThemesUserEventLogger;
+import com.android.customization.module.logging.ThemesUserEventLogger;
import org.junit.Before;
import org.junit.Ignore;