aboutsummaryrefslogtreecommitdiffhomepage
path: root/tests/MessageBusTest.cpp
diff options
context:
space:
mode:
authorGravatar Brian Salomon <bsalomon@google.com>2018-07-11 15:58:57 -0400
committerGravatar Skia Commit-Bot <skia-commit-bot@chromium.org>2018-07-11 20:36:38 +0000
commit238069b3f58d3b20f710f77a64632470a3bf256a (patch)
treecb4545c08c079aada830c52968c74c4e6af85bcd /tests/MessageBusTest.cpp
parent97c6a0ee41bb1af9cc81695590e2bb97ec85fecf (diff)
Reland "Make GPU cache invalidation SkMessageBus messages go to one GrContext."
This is a reland of f4c5bb9aba485aa47c27b15905d81992b7cf4707 Original change's description: > Make GPU cache invalidation SkMessageBus messages go to one GrContext. > > Makes it so the template param to SkMessageBus must implement: > bool shouldSend(uint32_t inboxID) const > > Updates all GPU backend message types to only go to the GrContext that > is adding a cache entry. > > Bug: skia: > Change-Id: I3e8a4eb90654b7b8ac57cac9fb508c0ef1d51058 > Reviewed-on: https://skia-review.googlesource.com/140220 > Reviewed-by: Robert Phillips <robertphillips@google.com> > Reviewed-by: Jim Van Verth <jvanverth@google.com> Bug: skia: Change-Id: I8402bfe3ed0170c99936d47050458817030b473b Reviewed-on: https://skia-review.googlesource.com/140801 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
Diffstat (limited to 'tests/MessageBusTest.cpp')
-rw-r--r--tests/MessageBusTest.cpp43
1 files changed, 43 insertions, 0 deletions
diff --git a/tests/MessageBusTest.cpp b/tests/MessageBusTest.cpp
index 163addf8f0..145176ba1b 100644
--- a/tests/MessageBusTest.cpp
+++ b/tests/MessageBusTest.cpp
@@ -8,10 +8,15 @@
#include "SkMessageBus.h"
#include "Test.h"
+namespace {
struct TestMessage {
+ bool shouldSend(uint32_t inboxID) const { return true; }
+ TestMessage(int i, float f) : x(i), y(f) {}
+
int x;
float y;
};
+}
DECLARE_SKMESSAGEBUS_MESSAGE(TestMessage)
DEF_TEST(MessageBus, r) {
@@ -50,4 +55,42 @@ DEF_TEST(MessageBus, r) {
REPORTER_ASSERT(r, 1 == messages[2].x);
}
+namespace {
+struct AddressedMessage {
+ uint32_t fInboxID;
+
+ bool shouldSend(uint32_t inboxID) const {
+ SkASSERT(inboxID);
+ if (!fInboxID) {
+ return true;
+ }
+ return inboxID == fInboxID;
+ }
+};
+}
+DECLARE_SKMESSAGEBUS_MESSAGE(AddressedMessage)
+
+DEF_TEST(MessageBus_shouldSend, r) {
+ SkMessageBus<AddressedMessage>::Inbox inbox1(1), inbox2(2);
+
+ SkMessageBus<AddressedMessage>::Post({0}); // Should go to both
+ SkMessageBus<AddressedMessage>::Post({1}); // Should go to inbox1
+ SkMessageBus<AddressedMessage>::Post({2}); // Should go to inbox2
+ SkMessageBus<AddressedMessage>::Post({3}); // Should go nowhere
+
+ SkTArray<AddressedMessage> messages;
+ inbox1.poll(&messages);
+ REPORTER_ASSERT(r, messages.count() == 2);
+ if (messages.count() == 2) {
+ REPORTER_ASSERT(r, messages[0].fInboxID == 0);
+ REPORTER_ASSERT(r, messages[1].fInboxID == 1);
+ }
+ inbox2.poll(&messages);
+ REPORTER_ASSERT(r, messages.count() == 2);
+ if (messages.count() == 2) {
+ REPORTER_ASSERT(r, messages[0].fInboxID == 0);
+ REPORTER_ASSERT(r, messages[1].fInboxID == 2);
+ }
+}
+
// Multithreaded tests tbd.