summaryrefslogtreecommitdiff
path: root/android
diff options
context:
space:
mode:
authorMichael Weghorn <m.weghorn@posteo.de>2023-11-08 15:10:19 +0100
committerMichael Weghorn <m.weghorn@posteo.de>2023-11-09 07:54:01 +0100
commit32423989263e17b8fdd1b8952180e025c50ed1bd (patch)
treef6cfd00bc194e3315ffabfb3a6a63e9d5be4f20d /android
parent9032551306bfac623984bed0031cbd6b8bfe54fb (diff)
android: Drop unused GLController#resumeCompositorIfValid etc.
The comment claiming that this function is invoked by JNI was true when the method was added upstream in Fennec/Gecko in commit [1] commit ae9420729da323d02a3f2f02d5a4b3043a619007 Author: Kartikaya Gupta <kgupta@mozilla.com> Date: Fri Jul 13 10:19:46 2012 -0400 Bug 769269 - Update the graphics code to be able to deal with blowing away the layer controller, client, and view and creating them afresh. r=ajuma , but it's not true for the copy here in Android Viewer, so drop the unused method and `RenderControllerThread#compositionResumeRequested` that would only have been called from there and wouldn't have done anything anyway. [1] https://github.com/mozilla/gecko-dev/commit/ae9420729da323d02a3f2f02d5a4b3043a619007 Change-Id: I54dd33a2ce30ebdc375a7c1fefacca6e99d2f571 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159159 Tested-by: Jenkins Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
Diffstat (limited to 'android')
-rw-r--r--android/source/src/java/org/mozilla/gecko/gfx/GLController.java7
-rw-r--r--android/source/src/java/org/mozilla/gecko/gfx/LayerView.java1
-rw-r--r--android/source/src/java/org/mozilla/gecko/gfx/RenderControllerThread.java5
3 files changed, 0 insertions, 13 deletions
diff --git a/android/source/src/java/org/mozilla/gecko/gfx/GLController.java b/android/source/src/java/org/mozilla/gecko/gfx/GLController.java
index 533980979683..76af0e78fc4d 100644
--- a/android/source/src/java/org/mozilla/gecko/gfx/GLController.java
+++ b/android/source/src/java/org/mozilla/gecko/gfx/GLController.java
@@ -117,13 +117,6 @@ public class GLController {
return true;
}
- // This function is invoked by JNI
- public synchronized void resumeCompositorIfValid() {
- if (mSurfaceValid) {
- mView.getListener().compositionResumeRequested(mWidth, mHeight);
- }
- }
-
// Wait until we are allowed to use EGL functions on the Surface backing
// this window. This function is invoked by JNI
public synchronized void waitForValidSurface() {
diff --git a/android/source/src/java/org/mozilla/gecko/gfx/LayerView.java b/android/source/src/java/org/mozilla/gecko/gfx/LayerView.java
index c3f639ddf7d1..955bcde316f9 100644
--- a/android/source/src/java/org/mozilla/gecko/gfx/LayerView.java
+++ b/android/source/src/java/org/mozilla/gecko/gfx/LayerView.java
@@ -304,7 +304,6 @@ public class LayerView extends FrameLayout {
void compositorCreated();
void renderRequested();
void compositionPauseRequested();
- void compositionResumeRequested(int width, int height);
void surfaceChanged(int width, int height);
}
diff --git a/android/source/src/java/org/mozilla/gecko/gfx/RenderControllerThread.java b/android/source/src/java/org/mozilla/gecko/gfx/RenderControllerThread.java
index 06f82f158366..a5b2ed79cbd3 100644
--- a/android/source/src/java/org/mozilla/gecko/gfx/RenderControllerThread.java
+++ b/android/source/src/java/org/mozilla/gecko/gfx/RenderControllerThread.java
@@ -82,11 +82,6 @@ public class RenderControllerThread extends Thread implements LayerView.Listener
}
@Override
- public void compositionResumeRequested(int width, int height) {
-
- }
-
- @Override
public void surfaceChanged(int width, int height) {
this.width = width;
this.height = height;