Commit 91bbc288 authored by Andrei-Laurențiu Olteanu's avatar Andrei-Laurențiu Olteanu Committed by Commit Bot

[Telemetry SWX] Remove urandom routine

Remove urandom routine from getAvailableRoutines.

Bug: b:168283428
Change-Id: I01b57aad92f9bab755a741a9551177e3aed8071a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2404791Reviewed-by: default avatarOleh Lamzin <lamzin@google.com>
Reviewed-by: default avatarMahmoud Gawad <mgawad@google.com>
Reviewed-by: default avatarTom Sepez <tsepez@chromium.org>
Commit-Queue: Laurențiu Olteanu <lolteanu@google.com>
Cr-Commit-Position: refs/heads/master@{#807001}
parent 76b2a0fc
...@@ -109,8 +109,6 @@ base::Optional<health::mojom::DiagnosticRoutineEnum> Convert( ...@@ -109,8 +109,6 @@ base::Optional<health::mojom::DiagnosticRoutineEnum> Convert(
return health::mojom::DiagnosticRoutineEnum::kBatteryCapacity; return health::mojom::DiagnosticRoutineEnum::kBatteryCapacity;
case cros_healthd::mojom::DiagnosticRoutineEnum::kBatteryHealth: case cros_healthd::mojom::DiagnosticRoutineEnum::kBatteryHealth:
return health::mojom::DiagnosticRoutineEnum::kBatteryHealth; return health::mojom::DiagnosticRoutineEnum::kBatteryHealth;
case cros_healthd::mojom::DiagnosticRoutineEnum::kUrandom:
return health::mojom::DiagnosticRoutineEnum::kUrandom;
case cros_healthd::mojom::DiagnosticRoutineEnum::kSmartctlCheck: case cros_healthd::mojom::DiagnosticRoutineEnum::kSmartctlCheck:
return health::mojom::DiagnosticRoutineEnum::kSmartctlCheck; return health::mojom::DiagnosticRoutineEnum::kSmartctlCheck;
case cros_healthd::mojom::DiagnosticRoutineEnum::kAcPower: case cros_healthd::mojom::DiagnosticRoutineEnum::kAcPower:
......
...@@ -190,17 +190,16 @@ interface DiagnosticsService { ...@@ -190,17 +190,16 @@ interface DiagnosticsService {
enum DiagnosticRoutineEnum { enum DiagnosticRoutineEnum {
kBatteryCapacity = 0, kBatteryCapacity = 0,
kBatteryHealth = 1, kBatteryHealth = 1,
kUrandom = 2, kSmartctlCheck = 2,
kSmartctlCheck = 3, kAcPower = 3,
kAcPower = 4, kCpuCache = 4,
kCpuCache = 5, kCpuStress = 5,
kCpuStress = 6, kFloatingPointAccuracy = 6,
kFloatingPointAccuracy = 7, kNvmeWearLevel = 7,
kNvmeWearLevel = 8, kNvmeSelfTest = 8,
kNvmeSelfTest = 9, kDiskRead = 9,
kDiskRead = 10, kPrimeSearch = 10,
kPrimeSearch = 11, kBatteryDischarge = 11,
kBatteryDischarge = 12,
}; };
// Enumeration of each of the possible statuses for a diagnostics routine. // Enumeration of each of the possible statuses for a diagnostics routine.
......
...@@ -52,7 +52,6 @@ class DiagnosticsProxy { ...@@ -52,7 +52,6 @@ class DiagnosticsProxy {
this.enumToRoutineName_ = new Map([ this.enumToRoutineName_ = new Map([
[routineEnum.kBatteryCapacity, 'battery-capacity'], [routineEnum.kBatteryCapacity, 'battery-capacity'],
[routineEnum.kBatteryHealth, 'battery-health'], [routineEnum.kBatteryHealth, 'battery-health'],
[routineEnum.kUrandom, 'urandom'],
[routineEnum.kSmartctlCheck, 'smartctl-check'], [routineEnum.kSmartctlCheck, 'smartctl-check'],
[routineEnum.kAcPower, 'ac-power'], [routineEnum.kAcPower, 'ac-power'],
[routineEnum.kCpuCache, 'cpu-cache'], [routineEnum.kCpuCache, 'cpu-cache'],
......
...@@ -69,7 +69,6 @@ void TelemetryExtensionUiBrowserTest::SetUpOnMainThread() { ...@@ -69,7 +69,6 @@ void TelemetryExtensionUiBrowserTest::SetUpOnMainThread() {
std::vector<cros_diagnostics::DiagnosticRoutineEnum> input{ std::vector<cros_diagnostics::DiagnosticRoutineEnum> input{
cros_diagnostics::DiagnosticRoutineEnum::kBatteryCapacity, cros_diagnostics::DiagnosticRoutineEnum::kBatteryCapacity,
cros_diagnostics::DiagnosticRoutineEnum::kBatteryHealth, cros_diagnostics::DiagnosticRoutineEnum::kBatteryHealth,
cros_diagnostics::DiagnosticRoutineEnum::kUrandom,
cros_diagnostics::DiagnosticRoutineEnum::kSmartctlCheck, cros_diagnostics::DiagnosticRoutineEnum::kSmartctlCheck,
cros_diagnostics::DiagnosticRoutineEnum::kAcPower, cros_diagnostics::DiagnosticRoutineEnum::kAcPower,
cros_diagnostics::DiagnosticRoutineEnum::kCpuCache, cros_diagnostics::DiagnosticRoutineEnum::kCpuCache,
......
...@@ -73,7 +73,6 @@ UNTRUSTED_TEST('UntrustedRequestAvailableRoutines', async () => { ...@@ -73,7 +73,6 @@ UNTRUSTED_TEST('UntrustedRequestAvailableRoutines', async () => {
assertDeepEquals(response, [ assertDeepEquals(response, [
'battery-capacity', 'battery-capacity',
'battery-health', 'battery-health',
'urandom',
'smartctl-check', 'smartctl-check',
'ac-power', 'ac-power',
'cpu-cache', 'cpu-cache',
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment