diff options
| author | Tyler Gunn <tgunn@google.com> | 2015-05-12 10:47:48 -0700 |
|---|---|---|
| committer | Tyler Gunn <tgunn@google.com> | 2015-05-13 10:51:20 -0700 |
| commit | f5d30a6bc69d7144bfdb99c09fa6ca4dccd67b0c (patch) | |
| tree | f8d1384d11ba157b4c4c07cb0ca003b7f2f457c2 | |
| parent | 9b0faa6cc03a8e05d6ee6ba63f2abb5d4ea00a75 (diff) | |
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
| -rw-r--r-- | InCallUI/src/com/android/incallui/VideoCallFragment.java | 7 |
1 files changed, 0 insertions, 7 deletions
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 |