From 2adaf04fab35cf47c824d74d901b54094e01ccd3 Mon Sep 17 00:00:00 2001 From: Andy McFadden Date: Tue, 18 Dec 2012 09:49:45 -0800 Subject: Rename ISurfaceTexture and SurfaceTexture The C++ class names don't match what the classes do, so rename ISurfaceTexture to IGraphicBufferProducer, and SurfaceTexture to GLConsumer. Bug 7736700 Change-Id: Ia03e468888025b5cae3c0ee1995434515dbea387 --- libs/gui/Surface.cpp | 30 +++++++++++++++--------------- 1 file changed, 15 insertions(+), 15 deletions(-) (limited to 'libs/gui/Surface.cpp') diff --git a/libs/gui/Surface.cpp b/libs/gui/Surface.cpp index 174506107d..51d37b3cf7 100644 --- a/libs/gui/Surface.cpp +++ b/libs/gui/Surface.cpp @@ -179,7 +179,7 @@ status_t SurfaceControl::writeSurfaceToParcel( identity = control->mIdentity; } parcel->writeStrongBinder(sur!=0 ? sur->asBinder() : NULL); - parcel->writeStrongBinder(NULL); // NULL ISurfaceTexture in this case. + parcel->writeStrongBinder(NULL); // NULL IGraphicBufferProducer in this case. parcel->writeInt32(identity); return NO_ERROR; } @@ -205,7 +205,7 @@ Surface::Surface(const sp& surface) mSurface(surface->mSurface), mIdentity(surface->mIdentity) { - sp st; + sp st; if (mSurface != NULL) { st = mSurface->getSurfaceTexture(); } @@ -217,9 +217,9 @@ Surface::Surface(const Parcel& parcel, const sp& ref) { mSurface = interface_cast(ref); sp st_binder(parcel.readStrongBinder()); - sp st; + sp st; if (st_binder != NULL) { - st = interface_cast(st_binder); + st = interface_cast(st_binder); } else if (mSurface != NULL) { st = mSurface->getSurfaceTexture(); } @@ -228,7 +228,7 @@ Surface::Surface(const Parcel& parcel, const sp& ref) init(st); } -Surface::Surface(const sp& st) +Surface::Surface(const sp& st) : SurfaceTextureClient(), mSurface(NULL), mIdentity(0) @@ -240,7 +240,7 @@ status_t Surface::writeToParcel( const sp& surface, Parcel* parcel) { sp sur; - sp st; + sp st; uint32_t identity = 0; if (Surface::isValid(surface)) { sur = surface->mSurface; @@ -249,8 +249,8 @@ status_t Surface::writeToParcel( } else if (surface != 0 && (surface->mSurface != NULL || surface->getISurfaceTexture() != NULL)) { - ALOGE("Parceling invalid surface with non-NULL ISurface/ISurfaceTexture as NULL: " - "mSurface = %p, surfaceTexture = %p, mIdentity = %d, ", + ALOGE("Parceling invalid surface with non-NULL ISurface/IGraphicBufferProducer " + "as NULL: mSurface = %p, bufferProducer = %p, mIdentity = %d, ", surface->mSurface.get(), surface->getISurfaceTexture().get(), surface->mIdentity); } @@ -275,7 +275,7 @@ sp Surface::readFromParcel(const Parcel& data) { } else { // The Surface was found in the cache, but we still should clear any // remaining data from the parcel. - data.readStrongBinder(); // ISurfaceTexture + data.readStrongBinder(); // IGraphicBufferProducer data.readInt32(); // identity } if (surface->mSurface == NULL && surface->getISurfaceTexture() == NULL) { @@ -296,12 +296,12 @@ void Surface::cleanCachedSurfacesLocked() { } } -void Surface::init(const sp& surfaceTexture) +void Surface::init(const sp& bufferProducer) { - if (mSurface != NULL || surfaceTexture != NULL) { - ALOGE_IF(surfaceTexture==0, "got a NULL ISurfaceTexture from ISurface"); - if (surfaceTexture != NULL) { - setISurfaceTexture(surfaceTexture); + if (mSurface != NULL || bufferProducer != NULL) { + ALOGE_IF(bufferProducer==0, "got a NULL IGraphicBufferProducer from ISurface"); + if (bufferProducer != NULL) { + setISurfaceTexture(bufferProducer); setUsage(GraphicBuffer::USAGE_HW_RENDER); } @@ -328,7 +328,7 @@ bool Surface::isValid() { return getISurfaceTexture() != NULL; } -sp Surface::getSurfaceTexture() { +sp Surface::getSurfaceTexture() { return getISurfaceTexture(); } -- cgit v1.2.3-59-g8ed1b