Update VideoStreamManager to avoid querying for capabilities in HMI NONE

bilal-alsharifi · bilal-alsharifi · commit 32ba00131305 · 2020-03-26T18:39:00.000-04:00
diff --git a/android/sdl_android/src/main/java/com/smartdevicelink/managers/video/VideoStreamManager.java b/android/sdl_android/src/main/java/com/smartdevicelink/managers/video/VideoStreamManager.java
@@ -151,6 +151,9 @@ public void onNotified(RPCNotification notification) {
 				}
 				HMILevel prevHMILevel = hmiLevel;
 				hmiLevel = onHMIStatus.getHmiLevel();
+				if (!hmiLevel.equals(HMILevel.HMI_NONE) && VideoStreamManager.this.parameters == null) {
+					getVideoStreamingParams();
+				}
 				if(hmiLevel.equals(HMILevel.HMI_FULL)){
 					checkState();
 				}
@@ -196,7 +199,6 @@ public VideoStreamManager(ISdl internalInterface){
 	@Override
 	public void start(CompletionListener listener) {
 		isTransportAvailable = internalInterface.isTransportForServiceAvailable(SessionType.NAV);
-		getVideoStreamingParams();
 		checkState();
 		super.start(listener);
 	}

Original file line number	Diff line number	Diff line change
`@@ -151,6 +151,9 @@ public void onNotified(RPCNotification notification) {`
`151`	`151`	`}`
`152`	`152`	`HMILevel prevHMILevel = hmiLevel;`
`153`	`153`	`hmiLevel = onHMIStatus.getHmiLevel();`
	`154`	`+ if (!hmiLevel.equals(HMILevel.HMI_NONE) && VideoStreamManager.this.parameters == null) {`
	`155`	`+ getVideoStreamingParams();`
	`156`	`+ }`
`154`	`157`	`if(hmiLevel.equals(HMILevel.HMI_FULL)){`
`155`	`158`	`checkState();`
`156`	`159`	`}`
`@@ -196,7 +199,6 @@ public VideoStreamManager(ISdl internalInterface){`
`196`	`199`	`@Override`
`197`	`200`	`public void start(CompletionListener listener) {`
`198`	`201`	`isTransportAvailable = internalInterface.isTransportForServiceAvailable(SessionType.NAV);`
`199`		`- getVideoStreamingParams();`
`200`	`202`	`checkState();`
`201`	`203`	`super.start(listener);`
`202`	`204`	`}`