Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 3d04b769 authored by Jae Seo's avatar Jae Seo
Browse files

TIF: API Review: android.media.tv 

Addressed the feedback from the API Review:

android.media.tv.TvView.TimeShiftPositionCallback
Please add more documentation to explain why we are using the wall-clock
time here.

android.media.tv.TvInputService
onTimeShiftSetPlaybackRate should be renamed to
onTimeShiftSetPlaybackParams and pass in the PlaybackParams object
(formerly known as PlaybackSettings) instead of the rate and mode
separately.

android.media.tv.TvTrackInfo
getDescription() should return a CharSequence not a String.

android.media.tv.TvTrackInfo.Builder
setDescription() should take a CharSequence as input.

android.media.tv.TvView
Change timeShiftSetPlaybackRate() to setTimeShiftPlaybackParams() and
use the PlaybackParams object (formerly known as PlaybackSettings)
instead of the rate and mode separately.

Bug: 21045120
Change-Id: I768665ae9c385fcb034bfb346c21b4c9f0891172
parent 642737c8

api/current.txt

+4 −4
Original line number Diff line number Diff line
@@ -17852,7 +17852,7 @@ package android.media.tv { 
    method public void onTimeShiftPause();

    method public void onTimeShiftResume();

    method public void onTimeShiftSeekTo(long);

    method public void onTimeShiftSetPlaybackRate(float, int);

    method public void onTimeShiftSetPlaybackParams(android.media.PlaybackParams);

    method public boolean onTouchEvent(android.view.MotionEvent);

    method public boolean onTrackballEvent(android.view.MotionEvent);

    method public abstract boolean onTune(android.net.Uri);
@@ -17864,7 +17864,7 @@ package android.media.tv { 
    method public int describeContents();

    method public final int getAudioChannelCount();

    method public final int getAudioSampleRate();

    method public final java.lang.String getDescription();

    method public final java.lang.CharSequence getDescription();

    method public final android.os.Bundle getExtra();

    method public final java.lang.String getId();

    method public final java.lang.String getLanguage();
@@ -17885,7 +17885,7 @@ package android.media.tv { 
    method public android.media.tv.TvTrackInfo build();

    method public final android.media.tv.TvTrackInfo.Builder setAudioChannelCount(int);

    method public final android.media.tv.TvTrackInfo.Builder setAudioSampleRate(int);

    method public final android.media.tv.TvTrackInfo.Builder setDescription(java.lang.String);

    method public final android.media.tv.TvTrackInfo.Builder setDescription(java.lang.CharSequence);

    method public final android.media.tv.TvTrackInfo.Builder setExtra(android.os.Bundle);

    method public final android.media.tv.TvTrackInfo.Builder setLanguage(java.lang.String);

    method public final android.media.tv.TvTrackInfo.Builder setVideoFrameRate(float);
@@ -17913,7 +17913,7 @@ package android.media.tv { 
    method public void timeShiftPause();

    method public void timeShiftResume();

    method public void timeShiftSeekTo(long);

    method public void timeShiftSetPlaybackRate(float, int);

    method public void timeShiftSetPlaybackParams(android.media.PlaybackParams);

    method public void tune(java.lang.String, android.net.Uri);

  }

























api/system-current.txt



+4
−4




























Original line number
Diff line number
Diff line








@@ -19298,7 +19298,7 @@ package android.media.tv {













    method public void onTimeShiftPause();















    method public void onTimeShiftResume();















    method public void onTimeShiftSeekTo(long);













    method public void onTimeShiftSetPlaybackRate(float, int);













    method public void onTimeShiftSetPlaybackParams(android.media.PlaybackParams);















    method public boolean onTouchEvent(android.view.MotionEvent);















    method public boolean onTrackballEvent(android.view.MotionEvent);















    method public abstract boolean onTune(android.net.Uri);










@@ -19334,7 +19334,7 @@ package android.media.tv {













    method public int describeContents();















    method public final int getAudioChannelCount();















    method public final int getAudioSampleRate();













    method public final java.lang.String getDescription();













    method public final java.lang.CharSequence getDescription();















    method public final android.os.Bundle getExtra();















    method public final java.lang.String getId();















    method public final java.lang.String getLanguage();









@@ -19355,7 +19355,7 @@ package android.media.tv {













    method public android.media.tv.TvTrackInfo build();















    method public final android.media.tv.TvTrackInfo.Builder setAudioChannelCount(int);















    method public final android.media.tv.TvTrackInfo.Builder setAudioSampleRate(int);













    method public final android.media.tv.TvTrackInfo.Builder setDescription(java.lang.String);













    method public final android.media.tv.TvTrackInfo.Builder setDescription(java.lang.CharSequence);















    method public final android.media.tv.TvTrackInfo.Builder setExtra(android.os.Bundle);















    method public final android.media.tv.TvTrackInfo.Builder setLanguage(java.lang.String);















    method public final android.media.tv.TvTrackInfo.Builder setVideoFrameRate(float);










@@ -19388,7 +19388,7 @@ package android.media.tv {













    method public void timeShiftPause();















    method public void timeShiftResume();















    method public void timeShiftSeekTo(long);













    method public void timeShiftSetPlaybackRate(float, int);













    method public void timeShiftSetPlaybackParams(android.media.PlaybackParams);















    method public void tune(java.lang.String, android.net.Uri);















    method public void tune(java.lang.String, android.net.Uri, android.os.Bundle);















    method public void unblockContent(android.media.tv.TvContentRating);

























media/java/android/media/tv/ITvInputSessionWrapper.java



+5
−1




























Original line number
Diff line number
Diff line








@@ -18,6 +18,7 @@ package android.media.tv;





























import android.content.Context;















import android.graphics.Rect;













import android.media.PlaybackParams;















import android.net.Uri;















import android.os.Bundle;















import android.os.IBinder;










@@ -172,7 +173,10 @@ public class ITvInputSessionWrapper extends ITvInputSession.Stub implements Hand













                break;















            }















            case DO_TIME_SHIFT_SET_PLAYBACK_RATE: {













                mTvInputSessionImpl.timeShiftSetPlaybackRate((Float) msg.obj, msg.arg1);













                PlaybackParams params = new PlaybackParams()













                        .setSpeed((Float) msg.obj)













                        .setAudioFallbackMode(msg.arg1);













                mTvInputSessionImpl.timeShiftSetPlaybackParams(params);















                break;















            }















            case DO_TIME_SHIFT_ENABLE_POSITION_TRACKING: {

































media/java/android/media/tv/TvInputService.java



+17
−20




























Original line number
Diff line number
Diff line








@@ -26,6 +26,8 @@ import android.content.Intent;













import android.graphics.PixelFormat;















import android.graphics.Rect;















import android.hardware.hdmi.HdmiDeviceInfo;













import android.media.PlaybackParams;













import android.media.tv.TvInputService.HardwareSession;















import android.net.Uri;















import android.os.AsyncTask;















import android.os.Bundle;










@@ -586,7 +588,7 @@ public abstract class TvInputService extends Service {













         * set playback rate and audio mode. The implementation should override















         * {@link #onTimeShiftPause}, {@link #onTimeShiftResume}, {@link #onTimeShiftSeekTo},















         * {@link #onTimeShiftGetStartPosition}, {@link #onTimeShiftGetCurrentPosition} and













         * {@link #onTimeShiftSetPlaybackRate}.













         * {@link #onTimeShiftSetPlaybackParams}.















         *















         * @param status The current time shift status. Should be one of the followings.















         * <ul>










@@ -853,7 +855,7 @@ public abstract class TvInputService extends Service {













         *















         * @see #onTimeShiftResume















         * @see #onTimeShiftSeekTo













         * @see #onTimeShiftSetPlaybackRate













         * @see #onTimeShiftSetPlaybackParams















         * @see #onTimeShiftGetStartPosition















         * @see #onTimeShiftGetCurrentPosition















         */









@@ -865,7 +867,7 @@ public abstract class TvInputService extends Service {













         *















         * @see #onTimeShiftPause















         * @see #onTimeShiftSeekTo













         * @see #onTimeShiftSetPlaybackRate













         * @see #onTimeShiftSetPlaybackParams















         * @see #onTimeShiftGetStartPosition















         * @see #onTimeShiftGetCurrentPosition















         */









@@ -881,7 +883,7 @@ public abstract class TvInputService extends Service {













         * @param timeMs The time position to seek to, in milliseconds since the epoch.















         * @see #onTimeShiftResume















         * @see #onTimeShiftPause













         * @see #onTimeShiftSetPlaybackRate













         * @see #onTimeShiftSetPlaybackParams















         * @see #onTimeShiftGetStartPosition















         * @see #onTimeShiftGetCurrentPosition















         */









@@ -889,25 +891,20 @@ public abstract class TvInputService extends Service {













        }































        /**













         * Called when the application sets playback rate and audio mode.













         * Called when the application sets playback parameters containing the speed and audio mode.















         *













         * <p>Once a playback rate is set, the implementation should honor the value until a new













         * tune request. Pause/resume/seek request does not reset the playback rate previously set.













         * <p>Once the playback parameters are set, the implementation should honor the current













         * settings until the next tune request. Pause/resume/seek request does not reset the













         * parameters previously set.















         *













         * @param rate The ratio between desired playback rate and normal one.













         * @param audioMode Audio playback mode. Must be one of the supported audio modes:













         * <ul>













         * <li> {@link android.media.MediaPlayer#PLAYBACK_RATE_AUDIO_MODE_DEFAULT}













         * <li> {@link android.media.MediaPlayer#PLAYBACK_RATE_AUDIO_MODE_STRETCH}













         * <li> {@link android.media.MediaPlayer#PLAYBACK_RATE_AUDIO_MODE_RESAMPLE}













         * </ul>













         * @param params The playback params.















         * @see #onTimeShiftResume















         * @see #onTimeShiftPause















         * @see #onTimeShiftSeekTo















         * @see #onTimeShiftGetStartPosition















         * @see #onTimeShiftGetCurrentPosition















         */













        public void onTimeShiftSetPlaybackRate(float rate, int audioMode) {













        public void onTimeShiftSetPlaybackParams(PlaybackParams params) {















        }































        /**









@@ -924,7 +921,7 @@ public abstract class TvInputService extends Service {













         * @see #onTimeShiftResume















         * @see #onTimeShiftPause















         * @see #onTimeShiftSeekTo













         * @see #onTimeShiftSetPlaybackRate













         * @see #onTimeShiftSetPlaybackParams















         * @see #onTimeShiftGetCurrentPosition















         */















        public long onTimeShiftGetStartPosition() {









@@ -939,7 +936,7 @@ public abstract class TvInputService extends Service {













         * @see #onTimeShiftResume















         * @see #onTimeShiftPause















         * @see #onTimeShiftSeekTo













         * @see #onTimeShiftSetPlaybackRate













         * @see #onTimeShiftSetPlaybackParams















         * @see #onTimeShiftGetStartPosition















         */















        public long onTimeShiftGetCurrentPosition() {










@@ -1273,10 +1270,10 @@ public abstract class TvInputService extends Service {













        }































        /**













         * Calls {@link #onTimeShiftSetPlaybackRate}.













         * Calls {@link #onTimeShiftSetPlaybackParams}.















         */













        void timeShiftSetPlaybackRate(float rate, int audioMode) {













            onTimeShiftSetPlaybackRate(rate, audioMode);













        void timeShiftSetPlaybackParams(PlaybackParams params) {













            onTimeShiftSetPlaybackParams(params);















        }































        /**

































media/java/android/media/tv/TvTrackInfo.java



+6
−6




























Original line number
Diff line number
Diff line








@@ -45,7 +45,7 @@ public final class TvTrackInfo implements Parcelable {













    private final int mType;















    private final String mId;















    private final String mLanguage;













    private final String mDescription;













    private final CharSequence mDescription;















    private final int mAudioChannelCount;















    private final int mAudioSampleRate;















    private final int mVideoWidth;









@@ -54,7 +54,7 @@ public final class TvTrackInfo implements Parcelable {













    private final float mVideoPixelAspectRatio;















    private final Bundle mExtra;





























    private TvTrackInfo(int type, String id, String language, String description,













    private TvTrackInfo(int type, String id, String language, CharSequence description,















            int audioChannelCount, int audioSampleRate, int videoWidth, int videoHeight,















            float videoFrameRate, float videoPixelAspectRatio, Bundle extra) {















        mType = type;










@@ -110,7 +110,7 @@ public final class TvTrackInfo implements Parcelable {













    /**















     * Returns a user readable description for the current track.















     */













    public final String getDescription() {













    public final CharSequence getDescription() {















        return mDescription;















    }


























@@ -201,7 +201,7 @@ public final class TvTrackInfo implements Parcelable {













        dest.writeInt(mType);















        dest.writeString(mId);















        dest.writeString(mLanguage);













        dest.writeString(mDescription);













        dest.writeString(mDescription != null ? mDescription.toString() : null);















        dest.writeInt(mAudioChannelCount);















        dest.writeInt(mAudioSampleRate);















        dest.writeInt(mVideoWidth);










@@ -231,7 +231,7 @@ public final class TvTrackInfo implements Parcelable {













        private final String mId;















        private final int mType;















        private String mLanguage;













        private String mDescription;













        private CharSequence mDescription;















        private int mAudioChannelCount;















        private int mAudioSampleRate;















        private int mVideoWidth;










@@ -274,7 +274,7 @@ public final class TvTrackInfo implements Parcelable {













         *















         * @param description The user readable description.















         */













        public final Builder setDescription(String description) {













        public final Builder setDescription(CharSequence description) {















            mDescription = description;















            return this;















        }