summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAnkit Gupta <agupta@cyngn.com>2016-06-14 13:49:47 -0700
committerAdrian DC <radian.dc@gmail.com>2016-12-04 00:34:39 +0100
commit33bf0323921de0ec8352971479500b3fc3cfc96a (patch)
tree79bd018944ec7196dc71a6cb8d93d51d75b6e747
parentaf1025f2f9bd0cbf02453a51d844b70897617d8d (diff)
downloadandroid_packages_apps_Messaging-33bf0323921de0ec8352971479500b3fc3cfc96a.tar.gz
android_packages_apps_Messaging-33bf0323921de0ec8352971479500b3fc3cfc96a.tar.bz2
android_packages_apps_Messaging-33bf0323921de0ec8352971479500b3fc3cfc96a.zip
MediaPicker: Check for NPE
make sure view has been created before asking for its visibility. RM-290 Issue-id: FEIJ-1135 Change-Id: Id5c537b5446c4e8edb1a6748d931e245fa31fd64
-rw-r--r--src/com/android/messaging/ui/mediapicker/AudioMediaChooser.java4
1 files changed, 4 insertions, 0 deletions
diff --git a/src/com/android/messaging/ui/mediapicker/AudioMediaChooser.java b/src/com/android/messaging/ui/mediapicker/AudioMediaChooser.java
index 5d79293..3553f10 100644
--- a/src/com/android/messaging/ui/mediapicker/AudioMediaChooser.java
+++ b/src/com/android/messaging/ui/mediapicker/AudioMediaChooser.java
@@ -123,6 +123,10 @@ class AudioMediaChooser extends MediaChooser implements
final int requestCode, final String permissions[], final int[] grantResults) {
if (requestCode == MediaPicker.RECORD_AUDIO_PERMISSION_REQUEST_CODE) {
final boolean permissionGranted = grantResults[0] == PackageManager.PERMISSION_GRANTED;
+ // onRequestPermissionsResult can sometimes get called before createView().
+ if (mEnabledView == null) {
+ return;
+ }
mEnabledView.setVisibility(permissionGranted ? View.VISIBLE : View.GONE);
mMissingPermissionView.setVisibility(permissionGranted ? View.GONE : View.VISIBLE);
}