commit 7798f6f6a694fd4e22d3f4c8f4316cf9190b6119
author Tomasz Wasilczyk <twasilczyk@google.com> Fri Dec 06 14:23:03 2019 -0800
committer Tomasz Wasilczyk <twasilczyk@google.com> Fri Dec 06 14:26:27 2019 -0800
tree 4de5efdbb237ce6a21bbb53d8fb2157bc2281941
parent 80d1a1054a0c921af1f29b1293dc2852b752b1d1

Don't crash Car Service without display brightness

If VHAL doesn't implement display brightness control, SysUI will fail to start.

Bug: 143779011
Test: enable Jeep VHAL, SysUI doesn't crash anymore
Change-Id: I1953f853d8cfa22fddcf72e31f1fa96efcdfd936

service/src/com/android/car/hal/PowerHalService.java

diff --git a/service/src/com/android/car/hal/PowerHalService.java b/service/src/com/android/car/hal/PowerHalService.java
index 82d2b59..b215a76 100644
--- a/service/src/com/android/car/hal/PowerHalService.java
+++ b/service/src/com/android/car/hal/PowerHalService.java
@@ -278,7 +278,7 @@
         try {
             mHal.set(VehicleProperty.DISPLAY_BRIGHTNESS, 0).to(brightness);
             Log.i(CarLog.TAG_POWER, "send display brightness = " + brightness);
-        } catch (PropertyTimeoutException e) {
+        } catch (PropertyTimeoutException | IllegalArgumentException e) {
             Log.e(CarLog.TAG_POWER, "cannot set DISPLAY_BRIGHTNESS", e);
         }
     }