Commit 6108c053 by tonihei Committed by microkatz

Rename getVideoSurfaceSize to getSurfaceSize

This better matches the callback name (onSurfaceSizeChanged) and
probably cause less confusion with getVideoSize.

PiperOrigin-RevId: 488669786
(cherry picked from commit 8438daca)
parent db2ab52f
...@@ -728,7 +728,7 @@ public final class CastPlayer extends BasePlayer { ...@@ -728,7 +728,7 @@ public final class CastPlayer extends BasePlayer {
/** This method is not supported and returns {@link Size#UNKNOWN}. */ /** This method is not supported and returns {@link Size#UNKNOWN}. */
@Override @Override
public Size getVideoSurfaceSize() { public Size getSurfaceSize() {
return Size.UNKNOWN; return Size.UNKNOWN;
} }
......
...@@ -762,10 +762,10 @@ public class ForwardingPlayer implements Player { ...@@ -762,10 +762,10 @@ public class ForwardingPlayer implements Player {
return player.getVideoSize(); return player.getVideoSize();
} }
/** Calls {@link Player#getVideoSurfaceSize()} on the delegate and returns the result. */ /** Calls {@link Player#getSurfaceSize()} on the delegate and returns the result. */
@Override @Override
public Size getVideoSurfaceSize() { public Size getSurfaceSize() {
return player.getVideoSurfaceSize(); return player.getSurfaceSize();
} }
/** Calls {@link Player#clearVideoSurface()} on the delegate. */ /** Calls {@link Player#clearVideoSurface()} on the delegate. */
......
...@@ -2477,7 +2477,7 @@ public interface Player { ...@@ -2477,7 +2477,7 @@ public interface Player {
* *
* @see Listener#onSurfaceSizeChanged(int, int) * @see Listener#onSurfaceSizeChanged(int, int)
*/ */
Size getVideoSurfaceSize(); Size getSurfaceSize();
/** Returns the current {@link CueGroup}. */ /** Returns the current {@link CueGroup}. */
CueGroup getCurrentCues(); CueGroup getCurrentCues();
......
...@@ -592,7 +592,7 @@ public abstract class SimpleBasePlayer extends BasePlayer { ...@@ -592,7 +592,7 @@ public abstract class SimpleBasePlayer extends BasePlayer {
} }
@Override @Override
public final Size getVideoSurfaceSize() { public final Size getSurfaceSize() {
// TODO: implement. // TODO: implement.
throw new IllegalStateException(); throw new IllegalStateException();
} }
......
...@@ -1220,7 +1220,7 @@ import java.util.concurrent.TimeoutException; ...@@ -1220,7 +1220,7 @@ import java.util.concurrent.TimeoutException;
} }
@Override @Override
public Size getVideoSurfaceSize() { public Size getSurfaceSize() {
verifyApplicationThread(); verifyApplicationThread();
return surfaceSize; return surfaceSize;
} }
......
...@@ -519,9 +519,9 @@ public class SimpleExoPlayer extends BasePlayer ...@@ -519,9 +519,9 @@ public class SimpleExoPlayer extends BasePlayer
} }
@Override @Override
public Size getVideoSurfaceSize() { public Size getSurfaceSize() {
blockUntilConstructorFinished(); blockUntilConstructorFinished();
return player.getVideoSurfaceSize(); return player.getSurfaceSize();
} }
@Override @Override
......
...@@ -351,7 +351,7 @@ public class StubPlayer extends BasePlayer { ...@@ -351,7 +351,7 @@ public class StubPlayer extends BasePlayer {
} }
@Override @Override
public Size getVideoSurfaceSize() { public Size getSurfaceSize() {
throw new UnsupportedOperationException(); throw new UnsupportedOperationException();
} }
......
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 sign in to comment