From f5d30a6bc69d7144bfdb99c09fa6ca4dccd67b0c Mon Sep 17 00:00:00 2001
From: Tyler Gunn <tgunn@google.com>
Date: Tue, 12 May 2015 10:47:48 -0700
Subject: Fix issue where video surfaces show when dowgrading to audio in
 landscape.

The issue is the "inflateVideoCallViews" call in onViewCreated.  We should
not call inflateVideoCallViews unless we know that the call is a video
call.  Removing the call to inflateVideoCallViews in onViewCreated solves
the problem.  Once the UI sets up, onPrimaryCallChanged in the
VideoCallPresenter is called to establish the primary call.  At this point
inflateVideoCallViews will get called anyways, so having it done in the
onViewCreated is redundant.

Bug: 20188274
Change-Id: Iface049e28192d58cf85347f6e88ca6e38a6a9aa
---
 InCallUI/src/com/android/incallui/VideoCallFragment.java | 7 -------
 1 file changed, 7 deletions(-)

(limited to 'InCallUI/src/com/android/incallui/VideoCallFragment.java')

diff --git a/InCallUI/src/com/android/incallui/VideoCallFragment.java b/InCallUI/src/com/android/incallui/VideoCallFragment.java
index 24e9e8d76..ce14e48f9 100644
--- a/InCallUI/src/com/android/incallui/VideoCallFragment.java
+++ b/InCallUI/src/com/android/incallui/VideoCallFragment.java
@@ -473,13 +473,6 @@ public class VideoCallFragment extends BaseFragment<VideoCallPresenter,
         Log.d(this, "onViewCreated: VideoSurfacesInUse=" + sVideoSurfacesInUse);
 
         mVideoViewsStub = (ViewStub) view.findViewById(R.id.videoCallViewsStub);
-
-        // If the surfaces are already in use, we have just changed orientation or otherwise
-        // re-created the fragment.  In this case we need to inflate the video call views and
-        // restore the surfaces.
-        if (sVideoSurfacesInUse) {
-            inflateVideoCallViews();
-        }
     }
 
     @Override
-- 
cgit v1.2.3