Bug 1363885 - 2. Remove ViERenderer; r?jesup draft
authorJim Chen <nchen@mozilla.com>
Fri, 12 May 2017 15:55:43 -0400
changeset 577086 f61548ce8acc3d4f746e08cd3fcd06ba2baa437f
parent 577085 e0646613b06c820003000978d0ed14f9917747e7
child 628415 78ea47c035cdb1cae98b17bf3ad045d5dc43a310
push id58597
push userbmo:nchen@mozilla.com
push dateFri, 12 May 2017 20:03:10 +0000
reviewersjesup
bugs1363885, 100644
milestone55.0a1
Bug 1363885 - 2. Remove ViERenderer; r?jesup ViERenderer is not used anywhere and it adds dependencies on enableOrientationListener and disableOrientationListener and GeckoInterface. --- .../src/org/webrtc/videoengine/ViERenderer.java | 64 ---------------------- mobile/android/base/moz.build | 1 - 2 files changed, 65 deletions(-) delete mode 100644 media/webrtc/trunk/webrtc/modules/video_render/android/java/src/org/webrtc/videoengine/ViERenderer.java MozReview-Commit-ID: GOauJeani54
media/webrtc/trunk/webrtc/modules/video_render/android/java/src/org/webrtc/videoengine/ViERenderer.java
mobile/android/base/moz.build
deleted file mode 100644
--- a/media/webrtc/trunk/webrtc/modules/video_render/android/java/src/org/webrtc/videoengine/ViERenderer.java
+++ /dev/null
@@ -1,64 +0,0 @@
-/*
- *  Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
- *
- *  Use of this source code is governed by a BSD-style license
- *  that can be found in the LICENSE file in the root of the source
- *  tree. An additional intellectual property rights grant can be found
- *  in the file PATENTS.  All contributing project authors may
- *  be found in the AUTHORS file in the root of the source tree.
- */
-
-package org.webrtc.videoengine;
-
-import android.content.Context;
-import android.util.Log;
-import android.view.Surface;
-import android.view.SurfaceHolder;
-import android.view.SurfaceView;
-import android.view.View;
-
-import org.mozilla.gecko.GeckoApp;
-import org.mozilla.gecko.GeckoAppShell;
-import org.mozilla.gecko.util.ThreadUtils;
-
-public class ViERenderer {
-    private final static String TAG = "WEBRTC-ViEREnderer";
-
-    // Call this function before ViECapture::StartCapture.
-    // The created view needs to be added to a visible layout
-    // after a camera has been allocated
-    // (with the call ViECapture::AllocateCaptureDevice).
-    // IE.
-    // CreateLocalRenderer
-    // ViECapture::AllocateCaptureDevice
-    // LinearLayout.addview
-    // ViECapture::StartCapture
-    public static void CreateLocalRenderer() {
-        ThreadUtils.getUiHandler().post(new Runnable() {
-            @Override
-            public void run() {
-                try {
-                    GeckoAppShell.getGeckoInterface().enableOrientationListener();
-                } catch (Exception e) {
-                    Log.e(TAG, "enableOrientationListener exception: "
-                          + e.getLocalizedMessage());
-                }
-            }
-        });
-    }
-
-    public static void DestroyLocalRenderer() {
-        ThreadUtils.getUiHandler().post(new Runnable() {
-            @Override
-            public void run() {
-                try {
-                    GeckoAppShell.getGeckoInterface().disableOrientationListener();
-                } catch (Exception e) {
-                    Log.e(TAG,
-                          "disableOrientationListener exception: " +
-                          e.getLocalizedMessage());
-                }
-            }
-        });
-    }
-}
--- a/mobile/android/base/moz.build
+++ b/mobile/android/base/moz.build
@@ -318,17 +318,16 @@ if CONFIG['MOZ_WEBRTC']:
     wrjar.sources += [
         base_root + 'Logging.java',
     ]
     wrjar.sources += [
         video_capture_root + 'CaptureCapabilityAndroid.java',
         video_capture_root + 'VideoCaptureAndroid.java',
         video_capture_root + 'VideoCaptureDeviceInfoAndroid.java',
         video_render_root + 'ViEAndroidGLES20.java',
-        video_render_root + 'ViERenderer.java',
         video_render_root + 'ViESurfaceRenderer.java',
     ]
     wrjar.sources += [
         audio_root + 'BuildInfo.java',
         audio_root + 'WebRtcAudioEffects.java',
         audio_root + 'WebRtcAudioManager.java',
         audio_root + 'WebRtcAudioRecord.java',
         audio_root + 'WebRtcAudioTrack.java',