Make some notification related slices public.
Only PreferenceControllers with isPublicSlice() set to true are exposed to other apps. Others will be Settings only. Bug: 141088937 Test: robotests Change-Id: I6e7b9d45084ea9944143e606bfac56b2cae3e8a3
This commit is contained in:
@@ -41,6 +41,11 @@ public class AlarmVolumePreferenceController extends VolumeSeekBarPreferenceCont
|
|||||||
return TextUtils.equals(getPreferenceKey(), "alarm_volume");
|
return TextUtils.equals(getPreferenceKey(), "alarm_volume");
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public boolean isPublicSlice() {
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public boolean useDynamicSliceSummary() {
|
public boolean useDynamicSliceSummary() {
|
||||||
return true;
|
return true;
|
||||||
|
@@ -43,6 +43,11 @@ public class CallVolumePreferenceController extends VolumeSeekBarPreferenceContr
|
|||||||
return TextUtils.equals(getPreferenceKey(), "call_volume");
|
return TextUtils.equals(getPreferenceKey(), "call_volume");
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public boolean isPublicSlice() {
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public boolean useDynamicSliceSummary() {
|
public boolean useDynamicSliceSummary() {
|
||||||
return true;
|
return true;
|
||||||
|
@@ -42,6 +42,11 @@ public class MediaVolumePreferenceController extends VolumeSeekBarPreferenceCont
|
|||||||
return TextUtils.equals(getPreferenceKey(), KEY_MEDIA_VOLUME);
|
return TextUtils.equals(getPreferenceKey(), KEY_MEDIA_VOLUME);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public boolean isPublicSlice() {
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public boolean useDynamicSliceSummary() {
|
public boolean useDynamicSliceSummary() {
|
||||||
return true;
|
return true;
|
||||||
|
@@ -44,6 +44,11 @@ public class NotificationVolumePreferenceController extends
|
|||||||
return TextUtils.equals(getPreferenceKey(), KEY_NOTIFICATION_VOLUME);
|
return TextUtils.equals(getPreferenceKey(), KEY_NOTIFICATION_VOLUME);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public boolean isPublicSlice() {
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public String getPreferenceKey() {
|
public String getPreferenceKey() {
|
||||||
return KEY_NOTIFICATION_VOLUME;
|
return KEY_NOTIFICATION_VOLUME;
|
||||||
|
@@ -196,6 +196,11 @@ public class RemoteVolumePreferenceController extends VolumeSeekBarPreferenceCon
|
|||||||
return TextUtils.equals(getPreferenceKey(), KEY_REMOTE_VOLUME);
|
return TextUtils.equals(getPreferenceKey(), KEY_REMOTE_VOLUME);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public boolean isPublicSlice() {
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public boolean useDynamicSliceSummary() {
|
public boolean useDynamicSliceSummary() {
|
||||||
return true;
|
return true;
|
||||||
|
@@ -95,6 +95,11 @@ public class RingVolumePreferenceController extends VolumeSeekBarPreferenceContr
|
|||||||
return TextUtils.equals(getPreferenceKey(), KEY_RING_VOLUME);
|
return TextUtils.equals(getPreferenceKey(), KEY_RING_VOLUME);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public boolean isPublicSlice() {
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public boolean useDynamicSliceSummary() {
|
public boolean useDynamicSliceSummary() {
|
||||||
return true;
|
return true;
|
||||||
|
@@ -77,6 +77,11 @@ public class VibrateWhenRingPreferenceController extends TogglePreferenceControl
|
|||||||
return TextUtils.equals(getPreferenceKey(), "vibrate_when_ringing");
|
return TextUtils.equals(getPreferenceKey(), "vibrate_when_ringing");
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public boolean isPublicSlice() {
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public void displayPreference(PreferenceScreen screen) {
|
public void displayPreference(PreferenceScreen screen) {
|
||||||
super.displayPreference(screen);
|
super.displayPreference(screen);
|
||||||
|
@@ -78,4 +78,9 @@ public class AlarmVolumePreferenceControllerTest {
|
|||||||
public void isSliceableCorrectKey_returnsTrue() {
|
public void isSliceableCorrectKey_returnsTrue() {
|
||||||
assertThat(mController.isSliceable()).isTrue();
|
assertThat(mController.isSliceable()).isTrue();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void isPublicSlice_returnTrue() {
|
||||||
|
assertThat(mController.isPublicSlice()).isTrue();
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
@@ -102,4 +102,9 @@ public class CallVolumePreferenceControllerTest {
|
|||||||
new CallVolumePreferenceController(mContext, "bad_key");
|
new CallVolumePreferenceController(mContext, "bad_key");
|
||||||
assertThat(controller.isSliceable()).isFalse();
|
assertThat(controller.isSliceable()).isFalse();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void isPublicSlice_returnTrue() {
|
||||||
|
assertThat(mController.isPublicSlice()).isTrue();
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
@@ -63,4 +63,9 @@ public class MediaVolumePreferenceControllerTest {
|
|||||||
mContext);
|
mContext);
|
||||||
assertThat(controller.isSliceable()).isTrue();
|
assertThat(controller.isSliceable()).isTrue();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void isPublicSlice_returnTrue() {
|
||||||
|
assertThat(mController.isPublicSlice()).isTrue();
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
@@ -102,4 +102,9 @@ public class NotificationVolumePreferenceControllerTest {
|
|||||||
new NotificationVolumePreferenceController(mContext);
|
new NotificationVolumePreferenceController(mContext);
|
||||||
assertThat(controller.isSliceable()).isTrue();
|
assertThat(controller.isSliceable()).isTrue();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void isPublicSlice_returnTrue() {
|
||||||
|
assertThat(mController.isPublicSlice()).isTrue();
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
@@ -218,4 +218,9 @@ public class RemoteVolumePreferenceControllerTest {
|
|||||||
|
|
||||||
assertThat(mController.mPreference.getProgress()).isEqualTo(CURRENT_POS);
|
assertThat(mController.mPreference.getProgress()).isEqualTo(CURRENT_POS);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void isPublicSlice_returnTrue() {
|
||||||
|
assertThat(mController.isPublicSlice()).isTrue();
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
@@ -104,4 +104,9 @@ public class RingVolumePreferenceControllerTest {
|
|||||||
new RingVolumePreferenceController(mContext);
|
new RingVolumePreferenceController(mContext);
|
||||||
assertThat(controller.isSliceable()).isTrue();
|
assertThat(controller.isSliceable()).isTrue();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void isPublicSlice_returnTrue() {
|
||||||
|
assertThat(mController.isPublicSlice()).isTrue();
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
@@ -44,9 +44,9 @@ import org.junit.Test;
|
|||||||
import org.junit.runner.RunWith;
|
import org.junit.runner.RunWith;
|
||||||
import org.mockito.Mock;
|
import org.mockito.Mock;
|
||||||
import org.mockito.MockitoAnnotations;
|
import org.mockito.MockitoAnnotations;
|
||||||
import org.robolectric.annotation.Config;
|
|
||||||
import org.robolectric.RobolectricTestRunner;
|
import org.robolectric.RobolectricTestRunner;
|
||||||
import org.robolectric.RuntimeEnvironment;
|
import org.robolectric.RuntimeEnvironment;
|
||||||
|
import org.robolectric.annotation.Config;
|
||||||
import org.robolectric.shadow.api.Shadow;
|
import org.robolectric.shadow.api.Shadow;
|
||||||
import org.robolectric.shadows.ShadowContentResolver;
|
import org.robolectric.shadows.ShadowContentResolver;
|
||||||
|
|
||||||
@@ -225,4 +225,8 @@ public class VibrateWhenRingPreferenceControllerTest {
|
|||||||
assertThat(controller.isSliceable()).isFalse();
|
assertThat(controller.isSliceable()).isFalse();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void isPublicSlice_returnTrue() {
|
||||||
|
assertThat(mController.isPublicSlice()).isTrue();
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user