diff options
author | Jeff Brown <jeffbrown@google.com> | 2012-09-06 10:15:30 -0700 |
---|---|---|
committer | Android (Google) Code Review <android-gerrit@google.com> | 2012-09-06 10:15:31 -0700 |
commit | 49c74da132fe42dd4d2066131f8f0e2df301c1dd (patch) | |
tree | da319566144a2dfcc05d9551e1f889fa97c2f0bb | |
parent | e148910dfb3f69cdf1a078066724cb32b18e7b8a (diff) | |
parent | 455d02eca342d8159637af317a5d83219d267812 (diff) |
Merge "Update remote display API." into jb-mr1-dev
-rw-r--r-- | include/media/IRemoteDisplay.h | 6 | ||||
-rw-r--r-- | include/media/IRemoteDisplayClient.h | 7 | ||||
-rw-r--r-- | media/libmedia/IRemoteDisplay.cpp | 10 | ||||
-rw-r--r-- | media/libmediaplayerservice/MediaPlayerService.cpp | 4 | ||||
-rw-r--r-- | media/libmediaplayerservice/RemoteDisplay.cpp | 2 | ||||
-rw-r--r-- | media/libmediaplayerservice/RemoteDisplay.h | 2 |
6 files changed, 15 insertions, 16 deletions
diff --git a/include/media/IRemoteDisplay.h b/include/media/IRemoteDisplay.h index f39286e0..a61704ee 100644 --- a/include/media/IRemoteDisplay.h +++ b/include/media/IRemoteDisplay.h @@ -39,10 +39,8 @@ class IRemoteDisplay : public IInterface public: DECLARE_META_INTERFACE(RemoteDisplay); - // Disconnects the remote display. - // The remote display should respond back to the IRemoteDisplayClient with an - // onDisplayDisconnected() event when the disconnection is complete. - virtual status_t disconnect() = 0; + // Disconnects the remote display and stops listening for new connections. + virtual status_t dispose() = 0; }; diff --git a/include/media/IRemoteDisplayClient.h b/include/media/IRemoteDisplayClient.h index 38a0c9a3..553ad36c 100644 --- a/include/media/IRemoteDisplayClient.h +++ b/include/media/IRemoteDisplayClient.h @@ -40,9 +40,9 @@ public: enum { // Error: An unknown / generic error occurred. - kErrorUnknown = 0, + kDisplayErrorUnknown = 1, // Error: The connection was dropped unexpectedly. - kErrorConnectionDropped = 1, + kDisplayErrorConnectionDropped = 2, }; // Indicates that the remote display has been connected successfully. @@ -52,7 +52,8 @@ public: uint32_t width, uint32_t height, uint32_t flags) = 0; // one-way // Indicates that the remote display has been disconnected normally. - // This method should only be called once the client has called 'disconnect()'. + // This method should only be called once the client has called 'dispose()' + // on the IRemoteDisplay. // It is currently an error for the display to disconnect for any other reason. virtual void onDisplayDisconnected() = 0; // one-way diff --git a/media/libmedia/IRemoteDisplay.cpp b/media/libmedia/IRemoteDisplay.cpp index 5d6ab34e..da25a15d 100644 --- a/media/libmedia/IRemoteDisplay.cpp +++ b/media/libmedia/IRemoteDisplay.cpp @@ -22,7 +22,7 @@ namespace android { enum { - DISCONNECT = IBinder::FIRST_CALL_TRANSACTION, + DISPOSE = IBinder::FIRST_CALL_TRANSACTION, }; class BpRemoteDisplay: public BpInterface<IRemoteDisplay> @@ -33,11 +33,11 @@ public: { } - status_t disconnect() + status_t dispose() { Parcel data, reply; data.writeInterfaceToken(IRemoteDisplay::getInterfaceDescriptor()); - remote()->transact(DISCONNECT, data, &reply); + remote()->transact(DISPOSE, data, &reply); return reply.readInt32(); } }; @@ -50,9 +50,9 @@ status_t BnRemoteDisplay::onTransact( uint32_t code, const Parcel& data, Parcel* reply, uint32_t flags) { switch (code) { - case DISCONNECT: { + case DISPOSE: { CHECK_INTERFACE(IRemoteDisplay, data, reply); - reply->writeInt32(disconnect()); + reply->writeInt32(dispose()); return NO_ERROR; } default: diff --git a/media/libmediaplayerservice/MediaPlayerService.cpp b/media/libmediaplayerservice/MediaPlayerService.cpp index 90055006..423d6ce1 100644 --- a/media/libmediaplayerservice/MediaPlayerService.cpp +++ b/media/libmediaplayerservice/MediaPlayerService.cpp @@ -283,7 +283,7 @@ sp<ICrypto> MediaPlayerService::makeCrypto() { sp<IRemoteDisplay> MediaPlayerService::listenForRemoteDisplay( const sp<IRemoteDisplayClient>& client, const String8& iface) { - return new RemoteDisplay(client, iface.string());; + return new RemoteDisplay(client, iface.string()); } status_t MediaPlayerService::enableRemoteDisplay(const char *iface) { @@ -299,7 +299,7 @@ status_t MediaPlayerService::enableRemoteDisplay(const char *iface) { } if (mRemoteDisplay != NULL) { - mRemoteDisplay->disconnect(); + mRemoteDisplay->dispose(); mRemoteDisplay.clear(); } diff --git a/media/libmediaplayerservice/RemoteDisplay.cpp b/media/libmediaplayerservice/RemoteDisplay.cpp index 1cc605e9..5542bb5d 100644 --- a/media/libmediaplayerservice/RemoteDisplay.cpp +++ b/media/libmediaplayerservice/RemoteDisplay.cpp @@ -39,7 +39,7 @@ RemoteDisplay::RemoteDisplay( RemoteDisplay::~RemoteDisplay() { } -status_t RemoteDisplay::disconnect() { +status_t RemoteDisplay::dispose() { mSource->stop(); mLooper->stop(); diff --git a/media/libmediaplayerservice/RemoteDisplay.h b/media/libmediaplayerservice/RemoteDisplay.h index 63c52863..0d87250c 100644 --- a/media/libmediaplayerservice/RemoteDisplay.h +++ b/media/libmediaplayerservice/RemoteDisplay.h @@ -33,7 +33,7 @@ struct WifiDisplaySource; struct RemoteDisplay : public BnRemoteDisplay { RemoteDisplay(const sp<IRemoteDisplayClient> &client, const char *iface); - virtual status_t disconnect(); + virtual status_t dispose(); protected: virtual ~RemoteDisplay(); |