Merge "Remove surface legacy APIs and code." into ics-mr1
This commit is contained in:
committed by
Android (Google) Code Review
commit
26f70db99f
@ -360,7 +360,7 @@ int main(int argc, char **argv) {
|
||||
service->create(getpid(), client, 0);
|
||||
|
||||
if (player != NULL && player->setDataSource(source) == NO_ERROR) {
|
||||
player->setVideoSurface(surface);
|
||||
player->setVideoSurfaceTexture(surface->getSurfaceTexture());
|
||||
player->start();
|
||||
|
||||
client->waitForEOS();
|
||||
|
@ -40,7 +40,6 @@ public:
|
||||
const KeyedVector<String8, String8>* headers) = 0;
|
||||
virtual status_t setDataSource(int fd, int64_t offset, int64_t length) = 0;
|
||||
virtual status_t setDataSource(const sp<IStreamSource>& source) = 0;
|
||||
virtual status_t setVideoSurface(const sp<Surface>& surface) = 0;
|
||||
virtual status_t setVideoSurfaceTexture(
|
||||
const sp<ISurfaceTexture>& surfaceTexture) = 0;
|
||||
virtual status_t prepareAsync() = 0;
|
||||
|
@ -117,9 +117,6 @@ public:
|
||||
return INVALID_OPERATION;
|
||||
}
|
||||
|
||||
// pass the buffered Surface to the media player service
|
||||
virtual status_t setVideoSurface(const sp<Surface>& surface) = 0;
|
||||
|
||||
// pass the buffered ISurfaceTexture to the media player service
|
||||
virtual status_t setVideoSurfaceTexture(
|
||||
const sp<ISurfaceTexture>& surfaceTexture) = 0;
|
||||
|
@ -170,7 +170,6 @@ public:
|
||||
|
||||
status_t setDataSource(int fd, int64_t offset, int64_t length);
|
||||
status_t setDataSource(const sp<IStreamSource> &source);
|
||||
status_t setVideoSurface(const sp<Surface>& surface);
|
||||
status_t setVideoSurfaceTexture(
|
||||
const sp<ISurfaceTexture>& surfaceTexture);
|
||||
status_t setListener(const sp<MediaPlayerListener>& listener);
|
||||
|
@ -35,7 +35,6 @@ enum {
|
||||
SET_DATA_SOURCE_URL,
|
||||
SET_DATA_SOURCE_FD,
|
||||
SET_DATA_SOURCE_STREAM,
|
||||
SET_VIDEO_SURFACE,
|
||||
PREPARE_ASYNC,
|
||||
START,
|
||||
STOP,
|
||||
@ -112,16 +111,6 @@ public:
|
||||
return reply.readInt32();
|
||||
}
|
||||
|
||||
// pass the buffered Surface to the media player service
|
||||
status_t setVideoSurface(const sp<Surface>& surface)
|
||||
{
|
||||
Parcel data, reply;
|
||||
data.writeInterfaceToken(IMediaPlayer::getInterfaceDescriptor());
|
||||
Surface::writeToParcel(surface, &data);
|
||||
remote()->transact(SET_VIDEO_SURFACE, data, &reply);
|
||||
return reply.readInt32();
|
||||
}
|
||||
|
||||
// pass the buffered ISurfaceTexture to the media player service
|
||||
status_t setVideoSurfaceTexture(const sp<ISurfaceTexture>& surfaceTexture)
|
||||
{
|
||||
@ -345,12 +334,6 @@ status_t BnMediaPlayer::onTransact(
|
||||
reply->writeInt32(setDataSource(source));
|
||||
return NO_ERROR;
|
||||
}
|
||||
case SET_VIDEO_SURFACE: {
|
||||
CHECK_INTERFACE(IMediaPlayer, data, reply);
|
||||
sp<Surface> surface = Surface::readFromParcel(data);
|
||||
reply->writeInt32(setVideoSurface(surface));
|
||||
return NO_ERROR;
|
||||
} break;
|
||||
case SET_VIDEO_SURFACETEXTURE: {
|
||||
CHECK_INTERFACE(IMediaPlayer, data, reply);
|
||||
sp<ISurfaceTexture> surfaceTexture =
|
||||
|
@ -219,14 +219,6 @@ status_t MediaPlayer::getMetadata(bool update_only, bool apply_filter, Parcel *m
|
||||
return mPlayer->getMetadata(update_only, apply_filter, metadata);
|
||||
}
|
||||
|
||||
status_t MediaPlayer::setVideoSurface(const sp<Surface>& surface)
|
||||
{
|
||||
LOGV("setVideoSurface");
|
||||
Mutex::Autolock _l(mLock);
|
||||
if (mPlayer == 0) return NO_INIT;
|
||||
return mPlayer->setVideoSurface(surface);
|
||||
}
|
||||
|
||||
status_t MediaPlayer::setVideoSurfaceTexture(
|
||||
const sp<ISurfaceTexture>& surfaceTexture)
|
||||
{
|
||||
|
@ -788,14 +788,6 @@ status_t MediaPlayerService::Client::setDataSource(
|
||||
return mStatus;
|
||||
}
|
||||
|
||||
status_t MediaPlayerService::Client::setVideoSurface(const sp<Surface>& surface)
|
||||
{
|
||||
LOGV("[%d] setVideoSurface(%p)", mConnId, surface.get());
|
||||
sp<MediaPlayerBase> p = getPlayer();
|
||||
if (p == 0) return UNKNOWN_ERROR;
|
||||
return p->setVideoSurface(surface);
|
||||
}
|
||||
|
||||
void MediaPlayerService::Client::disconnectNativeWindow() {
|
||||
if (mConnectedWindow != NULL) {
|
||||
status_t err = native_window_api_disconnect(mConnectedWindow.get(),
|
||||
|
@ -248,7 +248,6 @@ private:
|
||||
|
||||
// IMediaPlayer interface
|
||||
virtual void disconnect();
|
||||
virtual status_t setVideoSurface(const sp<Surface>& surface);
|
||||
virtual status_t setVideoSurfaceTexture(
|
||||
const sp<ISurfaceTexture>& surfaceTexture);
|
||||
virtual status_t prepareAsync();
|
||||
|
@ -35,7 +35,6 @@ public:
|
||||
const char* path, const KeyedVector<String8, String8> *headers);
|
||||
|
||||
virtual status_t setDataSource(int fd, int64_t offset, int64_t length);
|
||||
virtual status_t setVideoSurface(const sp<Surface>& surface) { return UNKNOWN_ERROR; }
|
||||
virtual status_t setVideoSurfaceTexture(
|
||||
const sp<ISurfaceTexture>& surfaceTexture)
|
||||
{ return UNKNOWN_ERROR; }
|
||||
|
@ -69,12 +69,6 @@ status_t StagefrightPlayer::setDataSource(const sp<IStreamSource> &source) {
|
||||
return mPlayer->setDataSource(source);
|
||||
}
|
||||
|
||||
status_t StagefrightPlayer::setVideoSurface(const sp<Surface> &surface) {
|
||||
LOGV("setVideoSurface");
|
||||
|
||||
return mPlayer->setSurface(surface);
|
||||
}
|
||||
|
||||
status_t StagefrightPlayer::setVideoSurfaceTexture(
|
||||
const sp<ISurfaceTexture> &surfaceTexture) {
|
||||
LOGV("setVideoSurfaceTexture");
|
||||
|
@ -40,7 +40,6 @@ public:
|
||||
|
||||
virtual status_t setDataSource(const sp<IStreamSource> &source);
|
||||
|
||||
virtual status_t setVideoSurface(const sp<Surface> &surface);
|
||||
virtual status_t setVideoSurfaceTexture(
|
||||
const sp<ISurfaceTexture> &surfaceTexture);
|
||||
virtual status_t prepare();
|
||||
|
@ -75,9 +75,6 @@ class TestPlayerStub : public MediaPlayerInterface {
|
||||
|
||||
|
||||
// All the methods below wrap the mPlayer instance.
|
||||
virtual status_t setVideoSurface(const android::sp<android::Surface>& s) {
|
||||
return mPlayer->setVideoSurface(s);
|
||||
}
|
||||
virtual status_t setVideoSurfaceTexture(
|
||||
const android::sp<android::ISurfaceTexture>& st) {
|
||||
return mPlayer->setVideoSurfaceTexture(st);
|
||||
|
@ -99,12 +99,6 @@ void NuPlayer::setDataSource(
|
||||
msg->post();
|
||||
}
|
||||
|
||||
void NuPlayer::setVideoSurface(const sp<Surface> &surface) {
|
||||
sp<AMessage> msg = new AMessage(kWhatSetVideoNativeWindow, id());
|
||||
msg->setObject("native-window", new NativeWindowWrapper(surface));
|
||||
msg->post();
|
||||
}
|
||||
|
||||
void NuPlayer::setVideoSurfaceTexture(const sp<ISurfaceTexture> &surfaceTexture) {
|
||||
sp<AMessage> msg = new AMessage(kWhatSetVideoNativeWindow, id());
|
||||
sp<SurfaceTextureClient> surfaceTextureClient(surfaceTexture != NULL ?
|
||||
|
@ -42,7 +42,6 @@ struct NuPlayer : public AHandler {
|
||||
void setDataSource(
|
||||
const char *url, const KeyedVector<String8, String8> *headers);
|
||||
|
||||
void setVideoSurface(const sp<Surface> &surface);
|
||||
void setVideoSurfaceTexture(const sp<ISurfaceTexture> &surfaceTexture);
|
||||
void setAudioSink(const sp<MediaPlayerBase::AudioSink> &sink);
|
||||
void start();
|
||||
|
@ -89,12 +89,6 @@ status_t NuPlayerDriver::setDataSource(const sp<IStreamSource> &source) {
|
||||
return OK;
|
||||
}
|
||||
|
||||
status_t NuPlayerDriver::setVideoSurface(const sp<Surface> &surface) {
|
||||
mPlayer->setVideoSurface(surface);
|
||||
|
||||
return OK;
|
||||
}
|
||||
|
||||
status_t NuPlayerDriver::setVideoSurfaceTexture(
|
||||
const sp<ISurfaceTexture> &surfaceTexture) {
|
||||
mPlayer->setVideoSurfaceTexture(surfaceTexture);
|
||||
|
@ -37,7 +37,6 @@ struct NuPlayerDriver : public MediaPlayerInterface {
|
||||
|
||||
virtual status_t setDataSource(const sp<IStreamSource> &source);
|
||||
|
||||
virtual status_t setVideoSurface(const sp<Surface> &surface);
|
||||
virtual status_t setVideoSurfaceTexture(
|
||||
const sp<ISurfaceTexture> &surfaceTexture);
|
||||
virtual status_t prepare();
|
||||
|
@ -1139,18 +1139,9 @@ bool AwesomePlayer::isPlaying() const {
|
||||
return (mFlags & PLAYING) || (mFlags & CACHE_UNDERRUN);
|
||||
}
|
||||
|
||||
status_t AwesomePlayer::setSurface(const sp<Surface> &surface) {
|
||||
Mutex::Autolock autoLock(mLock);
|
||||
|
||||
mSurface = surface;
|
||||
return setNativeWindow_l(surface);
|
||||
}
|
||||
|
||||
status_t AwesomePlayer::setSurfaceTexture(const sp<ISurfaceTexture> &surfaceTexture) {
|
||||
Mutex::Autolock autoLock(mLock);
|
||||
|
||||
mSurface.clear();
|
||||
|
||||
status_t err;
|
||||
if (surfaceTexture != NULL) {
|
||||
err = setNativeWindow_l(new SurfaceTextureClient(surfaceTexture));
|
||||
|
@ -81,7 +81,6 @@ struct AwesomePlayer {
|
||||
|
||||
bool isPlaying() const;
|
||||
|
||||
status_t setSurface(const sp<Surface> &surface);
|
||||
status_t setSurfaceTexture(const sp<ISurfaceTexture> &surfaceTexture);
|
||||
void setAudioSink(const sp<MediaPlayerBase::AudioSink> &audioSink);
|
||||
status_t setLooping(bool shouldLoop);
|
||||
@ -154,7 +153,6 @@ private:
|
||||
bool mUIDValid;
|
||||
uid_t mUID;
|
||||
|
||||
sp<Surface> mSurface;
|
||||
sp<ANativeWindow> mNativeWindow;
|
||||
sp<MediaPlayerBase::AudioSink> mAudioSink;
|
||||
|
||||
|
@ -68,7 +68,6 @@ class Player: public MediaPlayerBase
|
||||
}
|
||||
|
||||
virtual status_t setDataSource(int fd, int64_t offset, int64_t length) {return OK;}
|
||||
virtual status_t setVideoSurface(const sp<Surface>& surface) {return OK;}
|
||||
virtual status_t setVideoSurfaceTexture(
|
||||
const sp<ISurfaceTexture>& surfaceTexture) {return OK;}
|
||||
virtual status_t prepare() {return OK;}
|
||||
|
Reference in New Issue
Block a user