From b95cee4816092054dca8786b840e51ce36e2cca9 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')

diff --git a/InCallUI/src/com/android/incallui/VideoCallFragment.java b/InCallUI/src/com/android/incallui/VideoCallFragment.java
index 808655968..47c19e08e 100644
--- a/InCallUI/src/com/android/incallui/VideoCallFragment.java
+++ b/InCallUI/src/com/android/incallui/VideoCallFragment.java
@@ -477,13 +477,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