Do not generate copyable slices for Settings Search

Copyable slices is to provide user a quickly sub-text copying way on
Settings Search. Since we plan to remove all sub-text from Settings
Search, the copying feature is no longer available.

Therefore, we don't need to generate copyable slices in Settings
anymore.

Impact search result:
Build number, Phone number, Hardware version, Serial number and IMEI

Bug: 143118037
Test: visual and robotests
Change-Id: I81474aed994678c42d73cc59e169573880de1378
This commit is contained in:
Yanting Yang
2020-04-01 20:00:23 +08:00
parent 79a38a733f
commit b6c30a2c31
10 changed files with 0 additions and 121 deletions

View File

@@ -97,21 +97,6 @@ public class BuildNumberPreferenceController extends BasePreferenceController im
return AVAILABLE; return AVAILABLE;
} }
@Override
public boolean isSliceable() {
return true;
}
@Override
public boolean isPublicSlice() {
return true;
}
@Override
public boolean isCopyableSlice() {
return true;
}
@Override @Override
public boolean useDynamicSliceSummary() { public boolean useDynamicSliceSummary() {
return true; return true;

View File

@@ -90,21 +90,6 @@ public class PhoneNumberPreferenceController extends BasePreferenceController {
} }
} }
@Override
public boolean isSliceable() {
return true;
}
@Override
public boolean isPublicSlice() {
return true;
}
@Override
public boolean isCopyableSlice() {
return true;
}
@Override @Override
public boolean useDynamicSliceSummary() { public boolean useDynamicSliceSummary() {
return true; return true;

View File

@@ -40,21 +40,6 @@ public class HardwareRevisionPreferenceController extends BasePreferenceControll
return true; return true;
} }
@Override
public boolean isSliceable() {
return true;
}
@Override
public boolean isPublicSlice() {
return true;
}
@Override
public boolean isCopyableSlice() {
return true;
}
@Override @Override
public void copy() { public void copy() {
Sliceable.setCopyContent(mContext, getSummary(), Sliceable.setCopyContent(mContext, getSummary(),

View File

@@ -35,21 +35,6 @@ public class SerialNumberPreferenceController extends BasePreferenceController {
? AVAILABLE : UNSUPPORTED_ON_DEVICE; ? AVAILABLE : UNSUPPORTED_ON_DEVICE;
} }
@Override
public boolean isSliceable() {
return true;
}
@Override
public boolean isPublicSlice() {
return true;
}
@Override
public boolean isCopyableSlice() {
return true;
}
@Override @Override
public boolean useDynamicSliceSummary() { public boolean useDynamicSliceSummary() {
return true; return true;

View File

@@ -118,21 +118,6 @@ public class ImeiInfoPreferenceController extends BasePreferenceController {
&& !Utils.isWifiOnly(mContext) ? AVAILABLE : UNSUPPORTED_ON_DEVICE; && !Utils.isWifiOnly(mContext) ? AVAILABLE : UNSUPPORTED_ON_DEVICE;
} }
@Override
public boolean isSliceable() {
return true;
}
@Override
public boolean isPublicSlice() {
return true;
}
@Override
public boolean isCopyableSlice() {
return true;
}
@Override @Override
public boolean useDynamicSliceSummary() { public boolean useDynamicSliceSummary() {
return true; return true;

View File

@@ -205,9 +205,4 @@ public class BuildNumberPreferenceControllerTest {
final CharSequence data = clipboard.getPrimaryClip().getItemAt(0).getText(); final CharSequence data = clipboard.getPrimaryClip().getItemAt(0).getText();
assertThat(data.toString()).isEqualTo(mController.getSummary()); assertThat(data.toString()).isEqualTo(mController.getSummary());
} }
@Test
public void isPublicSlice_returnTrue() {
assertThat(mController.isPublicSlice()).isTrue();
}
} }

View File

@@ -158,11 +158,6 @@ public class PhoneNumberPreferenceControllerTest {
assertThat(primaryNumber).isEqualTo(mContext.getString(R.string.device_info_default)); assertThat(primaryNumber).isEqualTo(mContext.getString(R.string.device_info_default));
} }
@Test
public void isSliceable_shouldBeTrue() {
assertThat(mController.isSliceable()).isTrue();
}
@Test @Test
public void copy_shouldCopyPhoneNumberToClipboard() { public void copy_shouldCopyPhoneNumberToClipboard() {
final List<SubscriptionInfo> list = new ArrayList<>(); final List<SubscriptionInfo> list = new ArrayList<>();
@@ -178,9 +173,4 @@ public class PhoneNumberPreferenceControllerTest {
final CharSequence data = clipboard.getPrimaryClip().getItemAt(0).getText(); final CharSequence data = clipboard.getPrimaryClip().getItemAt(0).getText();
assertThat(phoneNumber.contentEquals(data)).isTrue(); assertThat(phoneNumber.contentEquals(data)).isTrue();
} }
@Test
public void isPublicSlice_returnTrue() {
assertThat(mController.isPublicSlice()).isTrue();
}
} }

View File

@@ -45,21 +45,6 @@ public class HardwareRevisionPreferenceControllerTest {
"hardware_info_device_revision"); "hardware_info_device_revision");
} }
@Test
public void isSliceable_shouldBeSliceable() {
assertThat(mController.isSliceable()).isTrue();
}
@Test
public void isPublicSlice_returnTrue() {
assertThat(mController.isPublicSlice()).isTrue();
}
@Test
public void isCopyableSlice_shouldBeCopyableSlice() {
assertThat(mController.isCopyableSlice()).isTrue();
}
@Test @Test
public void copy_shouldCopyHardwareRevisionToClipboard() { public void copy_shouldCopyHardwareRevisionToClipboard() {
final String fakeHardwareVer = "FakeVer1.0"; final String fakeHardwareVer = "FakeVer1.0";

View File

@@ -42,17 +42,6 @@ public class SerialNumberPreferenceControllerTest {
mController = new SerialNumberPreferenceController(mContext, "test"); mController = new SerialNumberPreferenceController(mContext, "test");
} }
@Test
public void isPublicSlice_returnTrue() {
assertThat(mController.isPublicSlice()).isTrue();
}
@Test
public void isCopyableSlice() {
assertThat(mController.isSliceable()).isTrue();
assertThat(mController.isCopyableSlice()).isTrue();
}
@Test @Test
public void copy_shouldPutSerialNumberToClipBoard() { public void copy_shouldPutSerialNumberToClipBoard() {
mController.copy(); mController.copy();

View File

@@ -186,9 +186,4 @@ public class ImeiInfoPreferenceControllerTest {
final CharSequence data = clipboard.getPrimaryClip().getItemAt(0).getText(); final CharSequence data = clipboard.getPrimaryClip().getItemAt(0).getText();
assertThat(data.toString()).isEqualTo(meid); assertThat(data.toString()).isEqualTo(meid);
} }
@Test
public void isPublicSlice_returnTrue() {
assertThat(mController.isPublicSlice()).isTrue();
}
} }