Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(246)

Unified Diff: media/base/test_helpers.h

Issue 416333011: Allow setContentDecryptionModule() to get called when setting is done. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: More changes Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/base/test_helpers.h
diff --git a/media/base/test_helpers.h b/media/base/test_helpers.h
index 8dc38958f5d2a0605b435d9f599e20bb17fd8ab5..b5c14b8be0063f5205a35758c742f20d85d5cd83 100644
--- a/media/base/test_helpers.h
+++ b/media/base/test_helpers.h
@@ -125,6 +125,20 @@ scoped_refptr<DecoderBuffer> CreateFakeVideoBufferForTest(
bool VerifyFakeVideoBufferForTest(const scoped_refptr<DecoderBuffer>& buffer,
const VideoDecoderConfig& config);
+// Used to verify that the number of calls to A() match the number of calls to
+// B() and that the calls happen in pairs (e.g. B() called after every A()
+// call). There may be any number of pairs (including 0).
+class MatchingCallbackVerifier {
+ public:
+ MatchingCallbackVerifier();
+ ~MatchingCallbackVerifier();
+ void A();
ddorwin 2014/08/08 21:15:26 nit: I wonder if this should be "RecordACalled" or
jrummell 2014/08/08 23:39:59 Done.
+ void B();
+
+ private:
+ bool expecting_B_;
ddorwin 2014/08/08 21:15:26 nit: 'b'
jrummell 2014/08/08 23:39:59 Done.
+};
+
} // namespace media
#endif // MEDIA_BASE_TEST_HELPERS_H_

Powered by Google App Engine
This is Rietveld 408576698