Commit d8b1f9ef by Oliver Woodman

Add API to get the version of native decoders

Add API to get the version of underlying libvpx and libopus
decoders. Also update the demo app to show the version in the UI.
parent 02d42753
...@@ -73,7 +73,8 @@ public class VideoPlayer extends Activity implements OnClickListener, ...@@ -73,7 +73,8 @@ public class VideoPlayer extends Activity implements OnClickListener,
private SurfaceView surfaceView; private SurfaceView surfaceView;
private VpxVideoSurfaceView vpxVideoSurfaceView; private VpxVideoSurfaceView vpxVideoSurfaceView;
private TextView debugInfoView; private TextView debugInfoView;
private TextView playerStateView; private String debugInfo;
private String playerState;
@Override @Override
public void onCreate(Bundle savedInstanceState) { public void onCreate(Bundle savedInstanceState) {
...@@ -106,7 +107,10 @@ public class VideoPlayer extends Activity implements OnClickListener, ...@@ -106,7 +107,10 @@ public class VideoPlayer extends Activity implements OnClickListener,
surfaceView = (SurfaceView) findViewById(R.id.surface_view); surfaceView = (SurfaceView) findViewById(R.id.surface_view);
vpxVideoSurfaceView = (VpxVideoSurfaceView) findViewById(R.id.vpx_surface_view); vpxVideoSurfaceView = (VpxVideoSurfaceView) findViewById(R.id.vpx_surface_view);
debugInfoView = (TextView) findViewById(R.id.debug_info); debugInfoView = (TextView) findViewById(R.id.debug_info);
playerStateView = (TextView) findViewById(R.id.player_state); debugInfo = "";
playerState = "";
filename = "";
updateDebugInfoTextView();
// Set the buttons' onclick listeners. // Set the buttons' onclick listeners.
((Button) findViewById(R.id.choose_file)).setOnClickListener(this); ((Button) findViewById(R.id.choose_file)).setOnClickListener(this);
...@@ -115,7 +119,6 @@ public class VideoPlayer extends Activity implements OnClickListener, ...@@ -115,7 +119,6 @@ public class VideoPlayer extends Activity implements OnClickListener,
// In case of DASH, start playback right away. // In case of DASH, start playback right away.
if (isDash) { if (isDash) {
findViewById(R.id.buttons).setVisibility(View.GONE); findViewById(R.id.buttons).setVisibility(View.GONE);
((TextView) findViewById(R.id.filename)).setVisibility(View.GONE);
startDashPlayback(); startDashPlayback();
} }
} }
...@@ -146,8 +149,7 @@ public class VideoPlayer extends Activity implements OnClickListener, ...@@ -146,8 +149,7 @@ public class VideoPlayer extends Activity implements OnClickListener,
case FILE_PICKER_REQUEST: case FILE_PICKER_REQUEST:
if (resultCode == Activity.RESULT_OK) { if (resultCode == Activity.RESULT_OK) {
filename = data.getStringExtra(FilePickerActivity.FILENAME_EXTRA_ID); filename = data.getStringExtra(FilePickerActivity.FILENAME_EXTRA_ID);
((TextView) findViewById(R.id.filename)).setText( updateDebugInfoTextView();
getString(R.string.current_path, filename));
} }
break; break;
} }
...@@ -186,7 +188,8 @@ public class VideoPlayer extends Activity implements OnClickListener, ...@@ -186,7 +188,8 @@ public class VideoPlayer extends Activity implements OnClickListener,
} }
private void startDashPlayback() { private void startDashPlayback() {
playerStateView.setText("Initializing"); playerState = "Initializing";
updateDebugInfoTextView();
final String userAgent = "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like" final String userAgent = "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like"
+ " Gecko) Chrome/38.0.2125.104 Safari/537.36"; + " Gecko) Chrome/38.0.2125.104 Safari/537.36";
DashRendererBuilder rendererBuilder = new DashRendererBuilder(manifestUrl, userAgent, this); DashRendererBuilder rendererBuilder = new DashRendererBuilder(manifestUrl, userAgent, this);
...@@ -213,7 +216,8 @@ public class VideoPlayer extends Activity implements OnClickListener, ...@@ -213,7 +216,8 @@ public class VideoPlayer extends Activity implements OnClickListener,
@Override @Override
public void onVideoSizeChanged(int width, int height) { public void onVideoSizeChanged(int width, int height) {
videoFrame.setAspectRatio(height == 0 ? 1 : (width * 1.0f) / height); videoFrame.setAspectRatio(height == 0 ? 1 : (width * 1.0f) / height);
debugInfoView.setText("Video: " + width + " x " + height); debugInfo = "Video: " + width + " x " + height;
updateDebugInfoTextView();
} }
@Override @Override
...@@ -223,12 +227,12 @@ public class VideoPlayer extends Activity implements OnClickListener, ...@@ -223,12 +227,12 @@ public class VideoPlayer extends Activity implements OnClickListener,
@Override @Override
public void onDecoderError(VpxDecoderException e) { public void onDecoderError(VpxDecoderException e) {
debugInfoView.setText("Libvpx decode failure. Giving up."); debugInfo = "Libvpx decode failure. Giving up.";
updateDebugInfoTextView();
} }
@Override @Override
public void onPlayerStateChanged(boolean playWhenReady, int state) { public void onPlayerStateChanged(boolean playWhenReady, int state) {
String playerState = "";
switch (player.getPlaybackState()) { switch (player.getPlaybackState()) {
case ExoPlayer.STATE_BUFFERING: case ExoPlayer.STATE_BUFFERING:
playerState = "buffering"; playerState = "buffering";
...@@ -246,12 +250,13 @@ public class VideoPlayer extends Activity implements OnClickListener, ...@@ -246,12 +250,13 @@ public class VideoPlayer extends Activity implements OnClickListener,
playerState = "ready"; playerState = "ready";
break; break;
} }
playerStateView.setText("Player State: " + playerState); updateDebugInfoTextView();
} }
@Override @Override
public void onPlayerError(ExoPlaybackException exception) { public void onPlayerError(ExoPlaybackException exception) {
debugInfoView.setText("Exoplayer Playback error. Giving up."); debugInfo = "Exoplayer Playback error. Giving up.";
updateDebugInfoTextView();
// TODO: show a retry button here. // TODO: show a retry button here.
} }
...@@ -282,4 +287,20 @@ public class VideoPlayer extends Activity implements OnClickListener, ...@@ -282,4 +287,20 @@ public class VideoPlayer extends Activity implements OnClickListener,
} }
} }
private void updateDebugInfoTextView() {
StringBuilder debugInfoText = new StringBuilder();
debugInfoText.append(
getString(R.string.libvpx_version, LibvpxVideoTrackRenderer.getLibvpxVersion()));
debugInfoText.append(" ");
debugInfoText.append(
getString(R.string.libopus_version, LibopusAudioTrackRenderer.getLibopusVersion()));
debugInfoText.append("\n");
debugInfoText.append(getString(R.string.current_path, filename));
debugInfoText.append(" ");
debugInfoText.append(debugInfo);
debugInfoText.append(" ");
debugInfoText.append(playerState);
debugInfoView.setText(debugInfoText.toString());
}
} }
...@@ -64,22 +64,6 @@ ...@@ -64,22 +64,6 @@
</LinearLayout> </LinearLayout>
<LinearLayout
android:layout_width="match_parent"
android:layout_height="wrap_content">
<TextView
android:id="@+id/filename"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:paddingRight="10dp"/>
<TextView
android:id="@+id/player_state"
android:paddingRight="10dp"
android:layout_width="wrap_content"
android:layout_height="wrap_content"/>
<TextView <TextView
android:id="@+id/debug_info" android:id="@+id/debug_info"
android:layout_width="wrap_content" android:layout_width="wrap_content"
...@@ -87,6 +71,4 @@ ...@@ -87,6 +71,4 @@
</LinearLayout> </LinearLayout>
</LinearLayout>
</FrameLayout> </FrameLayout>
...@@ -22,5 +22,11 @@ ...@@ -22,5 +22,11 @@
<string name="current_path"> <string name="current_path">
Path: <xliff:g id="path" example="/sdcard/test.webm">%1$s</xliff:g> Path: <xliff:g id="path" example="/sdcard/test.webm">%1$s</xliff:g>
</string> </string>
<string name="libvpx_version">
Libvpx: <xliff:g id="path">%1$s</xliff:g>
</string>
<string name="libopus_version">
Libopus: <xliff:g id="path">%1$s</xliff:g>
</string>
</resources> </resources>
...@@ -119,6 +119,15 @@ public final class LibopusAudioTrackRenderer extends SampleSourceTrackRenderer ...@@ -119,6 +119,15 @@ public final class LibopusAudioTrackRenderer extends SampleSourceTrackRenderer
formatHolder = new MediaFormatHolder(); formatHolder = new MediaFormatHolder();
} }
/**
* Get the version of underlying libopus library.
*
* @return version of the underlying libopus library.
*/
public static String getLibopusVersion() {
return OpusDecoder.getLibopusVersion();
}
@Override @Override
protected MediaClock getMediaClock() { protected MediaClock getMediaClock() {
return this; return this;
......
...@@ -81,6 +81,11 @@ import java.nio.ByteBuffer; ...@@ -81,6 +81,11 @@ import java.nio.ByteBuffer;
opusReset(nativeDecoderContext); opusReset(nativeDecoderContext);
} }
/**
* Returns the version string of the underlying libopus decoder.
*/
public static native String getLibopusVersion();
private native long opusInit(int sampleRate, int channelCount, int numStreams, int numCoupled, private native long opusInit(int sampleRate, int channelCount, int numStreams, int numCoupled,
int gain, byte[] streamMap); int gain, byte[] streamMap);
private native int opusDecode(long decoder, ByteBuffer inputBuffer, int inputSize, private native int opusDecode(long decoder, ByteBuffer inputBuffer, int inputSize,
......
...@@ -91,6 +91,10 @@ FUNC(void, opusReset, jlong jDecoder) { ...@@ -91,6 +91,10 @@ FUNC(void, opusReset, jlong jDecoder) {
opus_multistream_decoder_ctl(decoder, OPUS_RESET_STATE); opus_multistream_decoder_ctl(decoder, OPUS_RESET_STATE);
} }
FUNC(jstring, getLibopusVersion) {
return env->NewStringUTF(opus_get_version_string());
}
FUNC(jstring, opusGetErrorMessage, jint errorCode) { FUNC(jstring, opusGetErrorMessage, jint errorCode) {
return env->NewStringUTF(opus_strerror(errorCode)); return env->NewStringUTF(opus_strerror(errorCode));
} }
...@@ -150,6 +150,15 @@ public final class LibvpxVideoTrackRenderer extends SampleSourceTrackRenderer { ...@@ -150,6 +150,15 @@ public final class LibvpxVideoTrackRenderer extends SampleSourceTrackRenderer {
formatHolder = new MediaFormatHolder(); formatHolder = new MediaFormatHolder();
} }
/**
* Get the version of underlying libvpx library.
*
* @return version of the underlying libvpx library.
*/
public static String getLibvpxVersion() {
return VpxDecoder.getLibvpxVersion();
}
@Override @Override
protected boolean handlesTrack(MediaFormat mediaFormat) { protected boolean handlesTrack(MediaFormat mediaFormat) {
return MimeTypes.VIDEO_VP9.equalsIgnoreCase(mediaFormat.mimeType); return MimeTypes.VIDEO_VP9.equalsIgnoreCase(mediaFormat.mimeType);
......
...@@ -69,6 +69,11 @@ import java.nio.ByteBuffer; ...@@ -69,6 +69,11 @@ import java.nio.ByteBuffer;
vpxClose(vpxDecContext); vpxClose(vpxDecContext);
} }
/**
* Returns the version string of the underlying libvpx decoder.
*/
public static native String getLibvpxVersion();
private native long vpxInit(); private native long vpxInit();
private native long vpxClose(long context); private native long vpxClose(long context);
private native long vpxDecode(long context, ByteBuffer encoded, int length); private native long vpxDecode(long context, ByteBuffer encoded, int length);
......
...@@ -143,6 +143,10 @@ FUNC(jint, vpxGetFrame, jlong jContext, jobject jOutputBuffer, jboolean isRGB) { ...@@ -143,6 +143,10 @@ FUNC(jint, vpxGetFrame, jlong jContext, jobject jOutputBuffer, jboolean isRGB) {
return 0; return 0;
} }
FUNC(jstring, getLibvpxVersion) {
return env->NewStringUTF(vpx_codec_version_str());
}
FUNC(jstring, vpxGetErrorMessage, jlong jContext) { FUNC(jstring, vpxGetErrorMessage, jlong jContext) {
vpx_codec_ctx_t* const context = reinterpret_cast<vpx_codec_ctx_t*>(jContext); vpx_codec_ctx_t* const context = reinterpret_cast<vpx_codec_ctx_t*>(jContext);
return env->NewStringUTF(vpx_codec_error(context)); return env->NewStringUTF(vpx_codec_error(context));
......
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