Merge "Implement controller to disable transcoding cache" into sc-dev
This commit is contained in:
@@ -40,4 +40,9 @@
|
|||||||
android:key="transcode_notification"
|
android:key="transcode_notification"
|
||||||
android:title="@string/transcode_notification"
|
android:title="@string/transcode_notification"
|
||||||
settings:controller="com.android.settings.development.transcode.TranscodeNotificationPreferenceController" />
|
settings:controller="com.android.settings.development.transcode.TranscodeNotificationPreferenceController" />
|
||||||
|
|
||||||
|
<SwitchPreference
|
||||||
|
android:key="transcode_disable_cache"
|
||||||
|
android:title="@string/transcode_disable_cache"
|
||||||
|
settings:controller="com.android.settings.development.transcode.TranscodeDisableCachePreferenceController" />
|
||||||
</PreferenceScreen>
|
</PreferenceScreen>
|
||||||
|
@@ -0,0 +1,53 @@
|
|||||||
|
/*
|
||||||
|
* Copyright (C) 2021 The Android Open Source Project
|
||||||
|
*
|
||||||
|
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||||
|
* you may not use this file except in compliance with the License.
|
||||||
|
* You may obtain a copy of the License at
|
||||||
|
*
|
||||||
|
* http://www.apache.org/licenses/LICENSE-2.0
|
||||||
|
*
|
||||||
|
* Unless required by applicable law or agreed to in writing, software
|
||||||
|
* distributed under the License is distributed on an "AS IS" BASIS,
|
||||||
|
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||||
|
* See the License for the specific language governing permissions and
|
||||||
|
* limitations under the License.
|
||||||
|
*/
|
||||||
|
|
||||||
|
package com.android.settings.development.transcode;
|
||||||
|
|
||||||
|
import android.content.Context;
|
||||||
|
import android.os.SystemProperties;
|
||||||
|
|
||||||
|
import com.android.internal.annotations.VisibleForTesting;
|
||||||
|
import com.android.settings.core.TogglePreferenceController;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* The controller (in the Media transcoding settings) indicating the user's preference to disable
|
||||||
|
* the cache for transcoding.
|
||||||
|
*/
|
||||||
|
public class TranscodeDisableCachePreferenceController extends TogglePreferenceController {
|
||||||
|
@VisibleForTesting
|
||||||
|
static final String TRANSCODE_DISABLE_CACHE_SYS_PROP_KEY =
|
||||||
|
"persist.sys.fuse.disable_transcode_cache";
|
||||||
|
|
||||||
|
public TranscodeDisableCachePreferenceController(Context context, String preferenceKey) {
|
||||||
|
super(context, preferenceKey);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public boolean isChecked() {
|
||||||
|
return SystemProperties.getBoolean(TRANSCODE_DISABLE_CACHE_SYS_PROP_KEY, false);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public boolean setChecked(boolean isChecked) {
|
||||||
|
SystemProperties.set(TRANSCODE_DISABLE_CACHE_SYS_PROP_KEY, String.valueOf(isChecked));
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public int getAvailabilityStatus() {
|
||||||
|
return AVAILABLE;
|
||||||
|
}
|
||||||
|
}
|
@@ -70,7 +70,7 @@ public class TranscodeDefaultOptionPreferenceControllerTest {
|
|||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
public void getAvailabilityStatus_shouldReturn_isAvailable() {
|
public void getAvailabilityStatus_shouldReturnAVAILABLE() {
|
||||||
assertThat(mUnderTest.getAvailabilityStatus()).isEqualTo(
|
assertThat(mUnderTest.getAvailabilityStatus()).isEqualTo(
|
||||||
BasePreferenceController.AVAILABLE);
|
BasePreferenceController.AVAILABLE);
|
||||||
}
|
}
|
||||||
|
@@ -0,0 +1,77 @@
|
|||||||
|
/*
|
||||||
|
* Copyright (C) 2021 The Android Open Source Project
|
||||||
|
*
|
||||||
|
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||||
|
* you may not use this file except in compliance with the License.
|
||||||
|
* You may obtain a copy of the License at
|
||||||
|
*
|
||||||
|
* http://www.apache.org/licenses/LICENSE-2.0
|
||||||
|
*
|
||||||
|
* Unless required by applicable law or agreed to in writing, software
|
||||||
|
* distributed under the License is distributed on an "AS IS" BASIS,
|
||||||
|
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||||
|
* See the License for the specific language governing permissions and
|
||||||
|
* limitations under the License.
|
||||||
|
*/
|
||||||
|
|
||||||
|
package com.android.settings.development.transcode;
|
||||||
|
|
||||||
|
import static com.android.settings.development.transcode.TranscodeDisableCachePreferenceController.TRANSCODE_DISABLE_CACHE_SYS_PROP_KEY;
|
||||||
|
|
||||||
|
import static com.google.common.truth.Truth.assertThat;
|
||||||
|
|
||||||
|
import android.content.Context;
|
||||||
|
import android.os.SystemProperties;
|
||||||
|
|
||||||
|
import androidx.test.core.app.ApplicationProvider;
|
||||||
|
import androidx.test.ext.junit.runners.AndroidJUnit4;
|
||||||
|
|
||||||
|
import com.android.settings.core.BasePreferenceController;
|
||||||
|
|
||||||
|
import org.junit.Before;
|
||||||
|
import org.junit.Test;
|
||||||
|
import org.junit.runner.RunWith;
|
||||||
|
|
||||||
|
@RunWith(AndroidJUnit4.class)
|
||||||
|
public class TranscodeDisableCachePreferenceControllerTest {
|
||||||
|
|
||||||
|
private TranscodeUserControlPreferenceController mUnderTest;
|
||||||
|
|
||||||
|
@Before
|
||||||
|
public void setUp() {
|
||||||
|
Context context = ApplicationProvider.getApplicationContext();
|
||||||
|
mUnderTest = new TranscodeUserControlPreferenceController(context, "some_key");
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void isChecked_whenSysPropSet_shouldReturnTrue() {
|
||||||
|
SystemProperties.set(TRANSCODE_DISABLE_CACHE_SYS_PROP_KEY, "true");
|
||||||
|
assertThat(mUnderTest.isChecked()).isTrue();
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void isChecked_whenSysPropUnset_shouldReturnFalse() {
|
||||||
|
SystemProperties.set(TRANSCODE_DISABLE_CACHE_SYS_PROP_KEY, "false");
|
||||||
|
assertThat(mUnderTest.isChecked()).isFalse();
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void setChecked_withTrue_shouldSetSysProp() {
|
||||||
|
mUnderTest.setChecked(true);
|
||||||
|
assertThat(
|
||||||
|
SystemProperties.getBoolean(TRANSCODE_DISABLE_CACHE_SYS_PROP_KEY, false)).isTrue();
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void setChecked_withFalse_shouldUnsetSysProp() {
|
||||||
|
mUnderTest.setChecked(false);
|
||||||
|
assertThat(
|
||||||
|
SystemProperties.getBoolean(TRANSCODE_DISABLE_CACHE_SYS_PROP_KEY, true)).isFalse();
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void getAvailabilityStatus_shouldReturnAVAILABLE() {
|
||||||
|
assertThat(mUnderTest.getAvailabilityStatus()).isEqualTo(
|
||||||
|
BasePreferenceController.AVAILABLE);
|
||||||
|
}
|
||||||
|
}
|
@@ -69,7 +69,7 @@ public class TranscodeNotificationPreferenceControllerTest {
|
|||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
public void getAvailabilityStatus_shouldReturn_isAvailable() {
|
public void getAvailabilityStatus_shouldReturnAVAILABLE() {
|
||||||
assertThat(mUnderTest.getAvailabilityStatus()).isEqualTo(
|
assertThat(mUnderTest.getAvailabilityStatus()).isEqualTo(
|
||||||
BasePreferenceController.AVAILABLE);
|
BasePreferenceController.AVAILABLE);
|
||||||
}
|
}
|
||||||
|
@@ -70,7 +70,7 @@ public class TranscodeUserControlPreferenceControllerTest {
|
|||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
public void getAvailabilityStatus_shouldReturn_isAvailable() {
|
public void getAvailabilityStatus_shouldReturnAVAILABLE() {
|
||||||
assertThat(mUnderTest.getAvailabilityStatus()).isEqualTo(
|
assertThat(mUnderTest.getAvailabilityStatus()).isEqualTo(
|
||||||
BasePreferenceController.AVAILABLE);
|
BasePreferenceController.AVAILABLE);
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user