Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 16e781f1 authored by Colin Cross's avatar Colin Cross Committed by Android (Google) Code Review
Browse files

Merge "Simplify ThemePicker dependencies" into main

parents bc698c81 08db0645
Loading
Loading
Loading
Loading
+44 −51
Original line number Diff line number Diff line
@@ -19,22 +19,6 @@ package {
    default_applicable_licenses: ["Android-Apache-2.0"],
}

filegroup {
    name: "ThemePicker_srcs",
    srcs: [
        "src/**/*.java",
        "src/**/*.kt",
    ],
}

filegroup {
    name: "ThemePicker_src_overrides",
    srcs: [
        "src_override/**/*.java",
        "src_override/**/*.kt",
    ],
}

// Production-only files that should be excluded in tests.
filegroup {
    name: "ThemePicker_src_prod",
@@ -42,40 +26,39 @@ filegroup {
    srcs: ["src_override/com/android/customization/picker/CustomizationPickerApplication.java"],
}

filegroup {
    name: "ThemePicker_Manifest",
java_library {
    name: "ThemePickerOverridesLib",
    srcs: [
        "AndroidManifest.xml",
        "src_override/**/*.java",
        "src_override/**/*.kt",
    ],
    exclude_srcs: [":ThemePicker_src_prod"],
    libs: [
        "hilt_android",
        "ThemePickerLib",
    ],
}

genrule {
    name: "ThemePicker_res",
    tools: ["soong_zip"],
    srcs: [
        "res/**/*",
java_library {
    name: "ThemePickerApplicationLib",
    srcs: [":ThemePicker_src_prod"],
    libs: [
        "hilt_android",
        "ThemePickerLib",
        "ThemePickerOverridesLib",
    ],
    out: ["ThemePicker_res.zip"],
    cmd: "INPUTS=($(in)) && "
        + "RES_DIR=$$(dirname $$(dirname $${INPUTS[0]})) && "
        + "$(location soong_zip) -o $(out) -C $$RES_DIR -D $$RES_DIR"
}

genrule {
    name: "ThemePicker_res_overrides",
    tools: ["soong_zip"],

filegroup {
    name: "ThemePicker_Manifest",
    srcs: [
        "res_override/**/*",
        "AndroidManifest.xml",
    ],
    out: ["ThemePicker_res_overrides.zip"],
    cmd: "INPUTS=($(in)) && "
        + "RES_DIR=$$(dirname $$(dirname $${INPUTS[0]})) && "
        + "$(location soong_zip) -o $(out) -C $$RES_DIR -D $$RES_DIR"
}

// Common defaults that doesn't have ThemePicker specifics.
java_defaults {
    name: "ThemePicker_common_defaults",
android_library {
    name: "ThemePickerLib",

    static_libs: [
        "guava",
@@ -93,20 +76,35 @@ java_defaults {
        "hilt_android",
    ],

    jni_libs: [
        "librenderscript-toolkit",
    srcs: [
        "src/**/*.java",
        "src/**/*.kt",
    ],

    srcs: [
        ":ThemePicker_srcs",
    resource_dirs: [
        "res",
        "res_override",
    ],

    manifest: "AndroidManifest-empty.xml",
}

java_defaults {
    name: "ThemePicker_defaults",
    static_libs: [
        "ThemePickerLib",
        "ThemePickerOverridesLib",
        "hilt_android",
    ],

    jni_libs: [
        "librenderscript-toolkit",
    ],

    required: ["android.software.theme_picker.xml"],

    use_embedded_native_libs: true,

    resource_zips: [":ThemePicker_res", ":ThemePicker_res_overrides"],

    optimize: {
        enabled: false,
    },
@@ -117,12 +115,6 @@ java_defaults {
    system_ext_specific: true,
}

java_defaults {
    name: "ThemePicker_defaults",
    defaults: ["ThemePicker_common_defaults"],
    srcs: [":ThemePicker_src_overrides"],
}

prebuilt_etc {
    name: "android.software.theme_picker.xml",
    system_ext_specific: true,
@@ -141,4 +133,5 @@ android_app {
    manifest: "AndroidManifest.xml",
    additional_manifests: [":WallpaperPicker2_Manifest"],
    overrides: ["WallpaperPicker", "WallpaperPicker2"],
    static_libs: ["ThemePickerApplicationLib"],
}
+5 −0
Original line number Diff line number Diff line
<?xml version="1.0" encoding="utf-8"?>
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
          xmlns:tools="http://schemas.android.com/tools"
          package="com.android.wallpaper">
</manifest>
+1 −1
Original line number Diff line number Diff line
@@ -89,7 +89,7 @@ import kotlinx.coroutines.CoroutineScope
@Singleton
open class ThemePickerInjector
@Inject
internal constructor(
constructor(
    @MainDispatcher private val mainScope: CoroutineScope,
    @MainDispatcher private val mainDispatcher: CoroutineDispatcher,
    @BackgroundDispatcher private val bgScope: CoroutineScope,
+1 −1
Original line number Diff line number Diff line
@@ -23,12 +23,12 @@ package {
android_library {
    name: "ThemePickerTestLib",

    defaults: ["ThemePicker_common_defaults"],
    srcs: [
        "src/**/*.java",
        "src/**/*.kt",
    ],
    static_libs: [
        "ThemePickerLib",
        "WallpaperPicker2TestLib",
        "androidx.annotation_annotation",
        "kotlinx_coroutines_test",
+1 −0
Original line number Diff line number Diff line
@@ -25,6 +25,7 @@ android_robolectric_test {
        "kotlinx_coroutines_test",
        "truth",
        "SystemUICustomizationTestUtils",
        "ThemePickerApplicationLib",
    ],

    libs: [