diff options
-rw-r--r-- | include/gui/BufferQueueConsumer.h | 2 | ||||
-rw-r--r-- | include/gui/BufferQueueCore.h | 2 | ||||
-rw-r--r-- | include/gui/ConsumerBase.h | 6 | ||||
-rw-r--r-- | include/gui/IGraphicBufferConsumer.h | 2 | ||||
-rw-r--r-- | libs/gui/BufferQueueConsumer.cpp | 4 | ||||
-rw-r--r-- | libs/gui/BufferQueueCore.cpp | 2 | ||||
-rw-r--r-- | libs/gui/ConsumerBase.cpp | 8 | ||||
-rw-r--r-- | libs/gui/IGraphicBufferConsumer.cpp | 4 | ||||
-rw-r--r-- | libs/gui/tests/BufferQueue_test.cpp | 2 | ||||
-rw-r--r-- | services/surfaceflinger/DisplayHardware/FramebufferSurface.cpp | 2 | ||||
-rw-r--r-- | services/surfaceflinger/Layer.cpp | 2 |
11 files changed, 18 insertions, 18 deletions
diff --git a/include/gui/BufferQueueConsumer.h b/include/gui/BufferQueueConsumer.h index 8ec0546c7e..e2bafec4a9 100644 --- a/include/gui/BufferQueueConsumer.h +++ b/include/gui/BufferQueueConsumer.h @@ -144,7 +144,7 @@ public: virtual status_t discardFreeBuffers() override; // dump our state in a String - virtual void dump(String8& result, const char* prefix) const; + virtual void dumpState(String8& result, const char* prefix) const; // Functions required for backwards compatibility. // These will be modified/renamed in IGraphicBufferConsumer and will be diff --git a/include/gui/BufferQueueCore.h b/include/gui/BufferQueueCore.h index 3792441038..d35b22a9b9 100644 --- a/include/gui/BufferQueueCore.h +++ b/include/gui/BufferQueueCore.h @@ -86,7 +86,7 @@ public: private: // Dump our state in a string - void dump(String8& result, const char* prefix) const; + void dumpState(String8& result, const char* prefix) const; // getMinUndequeuedBufferCountLocked returns the minimum number of buffers // that must remain in a state other than DEQUEUED. The async parameter diff --git a/include/gui/ConsumerBase.h b/include/gui/ConsumerBase.h index de61662fe0..9f8b638a3a 100644 --- a/include/gui/ConsumerBase.h +++ b/include/gui/ConsumerBase.h @@ -63,11 +63,11 @@ public: // log messages. void setName(const String8& name); - // dump writes the current state to a string. Child classes should add + // dumpState writes the current state to a string. Child classes should add // their state to the dump by overriding the dumpLocked method, which is // called by these methods after locking the mutex. - void dump(String8& result) const; - void dump(String8& result, const char* prefix) const; + void dumpState(String8& result) const; + void dumpState(String8& result, const char* prefix) const; // setFrameAvailableListener sets the listener object that will be notified // when a new frame becomes available. diff --git a/include/gui/IGraphicBufferConsumer.h b/include/gui/IGraphicBufferConsumer.h index 08ca39dd3e..dcddca46e8 100644 --- a/include/gui/IGraphicBufferConsumer.h +++ b/include/gui/IGraphicBufferConsumer.h @@ -278,7 +278,7 @@ public: virtual status_t discardFreeBuffers() = 0; // dump state into a string - virtual void dump(String8& result, const char* prefix) const = 0; + virtual void dumpState(String8& result, const char* prefix) const = 0; public: DECLARE_META_INTERFACE(GraphicBufferConsumer) diff --git a/libs/gui/BufferQueueConsumer.cpp b/libs/gui/BufferQueueConsumer.cpp index fc53dbdc95..7fbf312727 100644 --- a/libs/gui/BufferQueueConsumer.cpp +++ b/libs/gui/BufferQueueConsumer.cpp @@ -732,7 +732,7 @@ status_t BufferQueueConsumer::discardFreeBuffers() { return NO_ERROR; } -void BufferQueueConsumer::dump(String8& result, const char* prefix) const { +void BufferQueueConsumer::dumpState(String8& result, const char* prefix) const { const IPCThreadState* ipc = IPCThreadState::self(); const pid_t pid = ipc->getCallingPid(); const uid_t uid = ipc->getCallingUid(); @@ -744,7 +744,7 @@ void BufferQueueConsumer::dump(String8& result, const char* prefix) const { android_errorWriteWithInfoLog(0x534e4554, "27046057", static_cast<int32_t>(uid), NULL, 0); } else { - mCore->dump(result, prefix); + mCore->dumpState(result, prefix); } } diff --git a/libs/gui/BufferQueueCore.cpp b/libs/gui/BufferQueueCore.cpp index 336783092e..dccc4a93d5 100644 --- a/libs/gui/BufferQueueCore.cpp +++ b/libs/gui/BufferQueueCore.cpp @@ -130,7 +130,7 @@ BufferQueueCore::BufferQueueCore(const sp<IGraphicBufferAlloc>& allocator) : BufferQueueCore::~BufferQueueCore() {} -void BufferQueueCore::dump(String8& result, const char* prefix) const { +void BufferQueueCore::dumpState(String8& result, const char* prefix) const { Mutex::Autolock lock(mMutex); String8 fifo; diff --git a/libs/gui/ConsumerBase.cpp b/libs/gui/ConsumerBase.cpp index a1bdf4a4a1..805a10d1b6 100644 --- a/libs/gui/ConsumerBase.cpp +++ b/libs/gui/ConsumerBase.cpp @@ -254,11 +254,11 @@ status_t ConsumerBase::discardFreeBuffers() { return mConsumer->discardFreeBuffers(); } -void ConsumerBase::dump(String8& result) const { - dump(result, ""); +void ConsumerBase::dumpState(String8& result) const { + dumpState(result, ""); } -void ConsumerBase::dump(String8& result, const char* prefix) const { +void ConsumerBase::dumpState(String8& result, const char* prefix) const { Mutex::Autolock _l(mMutex); dumpLocked(result, prefix); } @@ -267,7 +267,7 @@ void ConsumerBase::dumpLocked(String8& result, const char* prefix) const { result.appendFormat("%smAbandoned=%d\n", prefix, int(mAbandoned)); if (!mAbandoned) { - mConsumer->dump(result, prefix); + mConsumer->dumpState(result, prefix); } } diff --git a/libs/gui/IGraphicBufferConsumer.cpp b/libs/gui/IGraphicBufferConsumer.cpp index e39ff6e74e..240146455e 100644 --- a/libs/gui/IGraphicBufferConsumer.cpp +++ b/libs/gui/IGraphicBufferConsumer.cpp @@ -302,7 +302,7 @@ public: return result; } - virtual void dump(String8& result, const char* prefix) const { + virtual void dumpState(String8& result, const char* prefix) const { Parcel data, reply; data.writeInterfaceToken(IGraphicBufferConsumer::getInterfaceDescriptor()); data.writeString8(result); @@ -480,7 +480,7 @@ status_t BnGraphicBufferConsumer::onTransact( CHECK_INTERFACE(IGraphicBufferConsumer, data, reply); String8 result = data.readString8(); String8 prefix = data.readString8(); - static_cast<IGraphicBufferConsumer*>(this)->dump(result, prefix); + static_cast<IGraphicBufferConsumer*>(this)->dumpState(result, prefix); reply->writeString8(result); return NO_ERROR; } diff --git a/libs/gui/tests/BufferQueue_test.cpp b/libs/gui/tests/BufferQueue_test.cpp index 8a9eeeec21..65df7dc991 100644 --- a/libs/gui/tests/BufferQueue_test.cpp +++ b/libs/gui/tests/BufferQueue_test.cpp @@ -1044,7 +1044,7 @@ TEST_F(BufferQueueTest, TestDiscardFreeBuffers) { // Check no free buffers in dump String8 dumpString; - mConsumer->dump(dumpString, nullptr); + mConsumer->dumpState(dumpString, nullptr); // Parse the dump to ensure that all buffer slots that are FREE also // have a null GraphicBuffer diff --git a/services/surfaceflinger/DisplayHardware/FramebufferSurface.cpp b/services/surfaceflinger/DisplayHardware/FramebufferSurface.cpp index 7368d775ad..18c7945577 100644 --- a/services/surfaceflinger/DisplayHardware/FramebufferSurface.cpp +++ b/services/surfaceflinger/DisplayHardware/FramebufferSurface.cpp @@ -240,7 +240,7 @@ status_t FramebufferSurface::compositionComplete() #endif void FramebufferSurface::dumpAsString(String8& result) const { - ConsumerBase::dump(result); + ConsumerBase::dumpState(result); } void FramebufferSurface::dumpLocked(String8& result, const char* prefix) const diff --git a/services/surfaceflinger/Layer.cpp b/services/surfaceflinger/Layer.cpp index 6649d5a331..d13b6dbe19 100644 --- a/services/surfaceflinger/Layer.cpp +++ b/services/surfaceflinger/Layer.cpp @@ -2235,7 +2235,7 @@ void Layer::dump(String8& result, Colorizer& colorizer) const mQueuedFrames, mRefreshPending); if (mSurfaceFlingerConsumer != 0) { - mSurfaceFlingerConsumer->dump(result, " "); + mSurfaceFlingerConsumer->dumpState(result, " "); } } |