summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Jim Shargo <jshargo@google.com> 2024-07-29 23:21:08 +0000
committer Jim Shargo <jshargo@google.com> 2024-08-14 20:36:34 +0000
commit785d2e2a9e6ee1e876dbe53f2760382e03570e4d (patch)
tree628296d18a8affbc7e7d266b1a0989c32fa8534f
parent089099c30fe3ec284bb484b06a83cea438f9f1fc (diff)
view::Surface: Add toString()
This is used for the generation of AIDL files, I'm not sure why it hasn't broken before. For now, it just returns the set name of the view Surface. Bug: 340933754 Flag: EXEMPT only builds in later test, is otherwise unused functionality, required to build Test: new test in libgui_test Change-Id: I86ea29a8ea7dc8270fc471a0cfd04eb38e015045
-rw-r--r--libs/gui/include/gui/view/Surface.h3
-rw-r--r--libs/gui/tests/Surface_test.cpp9
-rw-r--r--libs/gui/view/Surface.cpp6
3 files changed, 17 insertions, 1 deletions
diff --git a/libs/gui/include/gui/view/Surface.h b/libs/gui/include/gui/view/Surface.h
index b7aba2b9dc..7ddac8139a 100644
--- a/libs/gui/include/gui/view/Surface.h
+++ b/libs/gui/include/gui/view/Surface.h
@@ -59,8 +59,9 @@ class Surface : public Parcelable {
// of the full parceling to happen on its native side.
status_t readFromParcel(const Parcel* parcel, bool nameAlreadyRead);
- private:
+ std::string toString() const;
+private:
static String16 readMaybeEmptyString16(const Parcel* parcel);
};
diff --git a/libs/gui/tests/Surface_test.cpp b/libs/gui/tests/Surface_test.cpp
index 8ab8783c4c..6356482fee 100644
--- a/libs/gui/tests/Surface_test.cpp
+++ b/libs/gui/tests/Surface_test.cpp
@@ -14,6 +14,7 @@
* limitations under the License.
*/
+#include "gui/view/Surface.h"
#include "Constants.h"
#include "MockConsumer.h"
@@ -2363,6 +2364,14 @@ TEST_F(SurfaceTest, QueueAcquireReleaseDequeue_CalledInStack_DoesNotDeadlock) {
EXPECT_EQ(OK, surface->disconnect(NATIVE_WINDOW_API_CPU));
}
+
+TEST_F(SurfaceTest, ViewSurface_toString) {
+ view::Surface surface{};
+ EXPECT_EQ("", surface.toString());
+
+ surface.name = String16("name");
+ EXPECT_EQ("name", surface.toString());
+}
#endif // COM_ANDROID_GRAPHICS_LIBGUI_FLAGS(WB_PLATFORM_API_IMPROVEMENTS)
} // namespace android
diff --git a/libs/gui/view/Surface.cpp b/libs/gui/view/Surface.cpp
index 7c15e7cf92..84c2a6ac71 100644
--- a/libs/gui/view/Surface.cpp
+++ b/libs/gui/view/Surface.cpp
@@ -121,5 +121,11 @@ String16 Surface::readMaybeEmptyString16(const Parcel* parcel) {
return str.value_or(String16());
}
+std::string Surface::toString() const {
+ std::stringstream out;
+ out << name;
+ return out.str();
+}
+
} // namespace view
} // namespace android