commit 14bec7aa4e0d1875c7f05945c18570a726b478e6
author Owen Lin <owenlin@chromium.org> Mon Apr 21 11:39:40 2014 +0800
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Mon Apr 21 12:51:01 2014 +0000
tree 078ea6d56789e12b376ea6074e34c1eb5d84218e
parent dc5c03cc6e4d209641e648e1f32faac14140f09f

audio_helper - Fix a NoneType error.

The get_mic_jack_status and get_hp_jack_status could be None.
Cast it to bool explicitly to solve the issue.

BUG=chromium:364968
TEST=None

Change-Id: Ib58c6e4000839fe9537b6c85b180017682e81e7a
Reviewed-on: https://chromium-review.googlesource.com/195716
Reviewed-by: Hsinyu Chao <hychao@chromium.org>
Commit-Queue: Owen Lin <owenlin@chromium.org>
Tested-by: Owen Lin <owenlin@chromium.org>
Reviewed-by: Rohit Makasana <rohitbm@chromium.org>

client/cros/audio/audio_helper.py

diff --git a/client/cros/audio/audio_helper.py b/client/cros/audio/audio_helper.py
index b6f890b..47bb809 100644
--- a/client/cros/audio/audio_helper.py
+++ b/client/cros/audio/audio_helper.py
@@ -194,12 +194,12 @@
     # Check Mic Jack
     mic_jack_status = get_mic_jack_status()
     logging.info('Mic jack status: %s', mic_jack_status)
-    dongle_status_ok &= mic_jack_status
+    dongle_status_ok &= bool(mic_jack_status)
 
     # Check Headphone Jack
     hp_jack_status = get_hp_jack_status()
     logging.info('Headphone jack status: %s', hp_jack_status)
-    dongle_status_ok &= hp_jack_status
+    dongle_status_ok &= bool(hp_jack_status)
 
     # Use latency check to test if audio can be captured through dongle.
     # We only want to know the basic function of dongle, so no need to