Skip to content

Commit

Permalink
GUI: Move Surface.aidl here and implement for native
Browse files Browse the repository at this point in the history
- Move android/view/Surface.aidl parcelable definition from frameworks/base
- Implement matching native behavior to android.view.Surface parceling,
  to enable use of android.view.Surface in generated native AIDL interfaces

Bug: 25091611
Change-Id: I87ffbb9a19c27a99ce0749d0ca1f17aaa6b3fe74
  • Loading branch information
Eino-Ville Talvala committed Mar 2, 2016
1 parent 86e8a79 commit 8861291
Show file tree
Hide file tree
Showing 3 changed files with 112 additions and 0 deletions.
20 changes: 20 additions & 0 deletions aidl/gui/android/view/Surface.aidl
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
/* //device/java/android/android/view/Surface.aidl
**
** Copyright 2007, The Android Open Source Project
**
** Licensed under the Apache License, Version 2.0 (the "License");
** you may not use this file except in compliance with the License.
** You may obtain a copy of the License at
**
** http://www.apache.org/licenses/LICENSE-2.0
**
** Unless required by applicable law or agreed to in writing, software
** distributed under the License is distributed on an "AS IS" BASIS,
** WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
** See the License for the specific language governing permissions and
** limitations under the License.
*/

package android.view;

parcelable Surface cpp_header "gui/Surface.h";
39 changes: 39 additions & 0 deletions include/gui/Surface.h
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,8 @@
#include <ui/ANativeObjectBase.h>
#include <ui/Region.h>

#include <binder/Parcelable.h>

#include <utils/RefBase.h>
#include <utils/threads.h>
#include <utils/KeyedVector.h>
Expand Down Expand Up @@ -348,6 +350,43 @@ class Surface
bool mSharedBufferHasBeenQueued;
};

namespace view {

/**
* A simple holder for an IGraphicBufferProducer, to match the managed-side
* android.view.Surface parcelable behavior.
*
* This implements android/view/Surface.aidl
*
* TODO: Convert IGraphicBufferProducer into AIDL so that it can be directly
* used in managed Binder calls.
*/
class Surface : public Parcelable {
public:

String16 name;
sp<IGraphicBufferProducer> graphicBufferProducer;

virtual status_t writeToParcel(Parcel* parcel) const override;
virtual status_t readFromParcel(const Parcel* parcel) override;

// nameAlreadyWritten set to true by Surface.java, because it splits
// Parceling itself between managed and native code, so it only wants a part
// of the full parceling to happen on its native side.
status_t writeToParcel(Parcel* parcel, bool nameAlreadyWritten) const;

// nameAlreadyRead set to true by Surface.java, because it splits
// Parceling itself between managed and native code, so it only wants a part
// of the full parceling to happen on its native side.
status_t readFromParcel(const Parcel* parcel, bool nameAlreadyRead);

private:

static String16 readMaybeEmptyString16(const Parcel* parcel);
};

} // namespace view

}; // namespace android

#endif // ANDROID_GUI_SURFACE_H
53 changes: 53 additions & 0 deletions libs/gui/Surface.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1259,4 +1259,57 @@ status_t Surface::unlockAndPost()
return err;
}

namespace view {

status_t Surface::writeToParcel(Parcel* parcel) const {
return writeToParcel(parcel, false);
}

status_t Surface::writeToParcel(Parcel* parcel, bool nameAlreadyWritten) const {
if (parcel == nullptr) return BAD_VALUE;

status_t res = OK;

if (!nameAlreadyWritten) res = parcel->writeString16(name);

if (res == OK) {
res = parcel->writeStrongBinder(
IGraphicBufferProducer::asBinder(graphicBufferProducer));
}
return res;
}

status_t Surface::readFromParcel(const Parcel* parcel) {
return readFromParcel(parcel, false);
}

status_t Surface::readFromParcel(const Parcel* parcel, bool nameAlreadyRead) {
if (parcel == nullptr) return BAD_VALUE;

if (!nameAlreadyRead) {
name = readMaybeEmptyString16(parcel);
}

sp<IBinder> binder;

status_t res = parcel->readStrongBinder(&binder);
if (res != OK) return res;

graphicBufferProducer = interface_cast<IGraphicBufferProducer>(binder);

return OK;
}

String16 Surface::readMaybeEmptyString16(const Parcel* parcel) {
size_t len;
const char16_t* str = parcel->readString16Inplace(&len);
if (str != nullptr) {
return String16(str, len);
} else {
return String16();
}
}

} // namespace view

}; // namespace android

0 comments on commit 8861291

Please sign in to comment.