Commit ecd023d3 authored by Bo Liu's avatar Bo Liu Committed by Commit Bot

android: Use DisplayAndroid in media

Bug: 1042063
Change-Id: I8b61a7ccdbf50703115a0e6b971a02623771756d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2019603Reviewed-by: default avatarYuri Wiitala <miu@chromium.org>
Reviewed-by: default avatarDale Curtis <dalecurtis@chromium.org>
Commit-Queue: Bo <boliu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#736028}
parent 0145256b
...@@ -19,6 +19,7 @@ include_rules = [ ...@@ -19,6 +19,7 @@ include_rules = [
"+third_party/libyuv", "+third_party/libyuv",
"+third_party/opus", "+third_party/opus",
"+third_party/skia", "+third_party/skia",
"+ui/android",
"+ui/display", "+ui/display",
"+ui/events", "+ui/events",
"+ui/gfx", "+ui/gfx",
......
...@@ -37,6 +37,7 @@ android_library("screen_capture_java") { ...@@ -37,6 +37,7 @@ android_library("screen_capture_java") {
deps = [ deps = [
"//base:base_java", "//base:base_java",
"//base:jni_java", "//base:jni_java",
"//ui/android:ui_java",
] ]
sources = [ "java/src/org/chromium/media/ScreenCapture.java" ] sources = [ "java/src/org/chromium/media/ScreenCapture.java" ]
annotation_processor_deps = [ "//base/android/jni_generator:jni_processor" ] annotation_processor_deps = [ "//base/android/jni_generator:jni_processor" ]
......
...@@ -25,7 +25,6 @@ import android.os.HandlerThread; ...@@ -25,7 +25,6 @@ import android.os.HandlerThread;
import android.util.DisplayMetrics; import android.util.DisplayMetrics;
import android.view.Display; import android.view.Display;
import android.view.Surface; import android.view.Surface;
import android.view.WindowManager;
import androidx.annotation.IntDef; import androidx.annotation.IntDef;
...@@ -35,6 +34,7 @@ import org.chromium.base.Log; ...@@ -35,6 +34,7 @@ import org.chromium.base.Log;
import org.chromium.base.annotations.CalledByNative; import org.chromium.base.annotations.CalledByNative;
import org.chromium.base.annotations.JNINamespace; import org.chromium.base.annotations.JNINamespace;
import org.chromium.base.annotations.NativeMethods; import org.chromium.base.annotations.NativeMethods;
import org.chromium.ui.display.DisplayAndroidManager;
import java.lang.annotation.Retention; import java.lang.annotation.Retention;
import java.lang.annotation.RetentionPolicy; import java.lang.annotation.RetentionPolicy;
...@@ -238,10 +238,8 @@ public class ScreenCapture extends Fragment { ...@@ -238,10 +238,8 @@ public class ScreenCapture extends Fragment {
return false; return false;
} }
WindowManager windowManager = mDisplay = DisplayAndroidManager.getDefaultDisplayForContext(
(WindowManager) ContextUtils.getApplicationContext().getSystemService( ContextUtils.getApplicationContext());
Context.WINDOW_SERVICE);
mDisplay = windowManager.getDefaultDisplay();
DisplayMetrics metrics = new DisplayMetrics(); DisplayMetrics metrics = new DisplayMetrics();
mDisplay.getMetrics(metrics); mDisplay.getMetrics(metrics);
......
...@@ -53,6 +53,7 @@ android_library("capture_java") { ...@@ -53,6 +53,7 @@ android_library("capture_java") {
"//base:base_java", "//base:base_java",
"//base:jni_java", "//base:jni_java",
"//third_party/android_deps:androidx_annotation_annotation_java", "//third_party/android_deps:androidx_annotation_annotation_java",
"//ui/android:ui_java",
] ]
annotation_processor_deps = [ "//base/android/jni_generator:jni_processor" ] annotation_processor_deps = [ "//base/android/jni_generator:jni_processor" ]
......
...@@ -4,15 +4,13 @@ ...@@ -4,15 +4,13 @@
package org.chromium.media; package org.chromium.media;
import android.content.Context;
import android.graphics.ImageFormat; import android.graphics.ImageFormat;
import android.view.Surface;
import android.view.WindowManager;
import org.chromium.base.ContextUtils; import org.chromium.base.ContextUtils;
import org.chromium.base.annotations.CalledByNative; import org.chromium.base.annotations.CalledByNative;
import org.chromium.base.annotations.JNINamespace; import org.chromium.base.annotations.JNINamespace;
import org.chromium.base.annotations.NativeMethods; import org.chromium.base.annotations.NativeMethods;
import org.chromium.ui.display.DisplayAndroid;
import java.nio.ByteBuffer; import java.nio.ByteBuffer;
import java.util.ArrayList; import java.util.ArrayList;
...@@ -157,25 +155,9 @@ public abstract class VideoCapture { ...@@ -157,25 +155,9 @@ public abstract class VideoCapture {
} }
protected final int getDeviceRotation() { protected final int getDeviceRotation() {
final int orientation; DisplayAndroid display =
WindowManager wm = (WindowManager) ContextUtils.getApplicationContext().getSystemService( DisplayAndroid.getNonMultiDisplay(ContextUtils.getApplicationContext());
Context.WINDOW_SERVICE); return display.getRotationDegrees();
switch (wm.getDefaultDisplay().getRotation()) {
case Surface.ROTATION_90:
orientation = 90;
break;
case Surface.ROTATION_180:
orientation = 180;
break;
case Surface.ROTATION_270:
orientation = 270;
break;
case Surface.ROTATION_0:
default:
orientation = 0;
break;
}
return orientation;
} }
// {@link VideoCaptureJni.get().onPhotoTaken()} needs to be called back if there's any // {@link VideoCaptureJni.get().onPhotoTaken()} needs to be called back if there's any
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment