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

Unified Diff: webrtc/api/test/mockpeerconnectionobservers.h

Issue 2622413005: Replace use of ASSERT in test code. (Closed)
Patch Set: Fixed another signed/unsigned comparison. Created 3 years, 11 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
« no previous file with comments | « webrtc/api/test/fakedatachannelprovider.h ('k') | webrtc/api/webrtcsession_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/api/test/mockpeerconnectionobservers.h
diff --git a/webrtc/api/test/mockpeerconnectionobservers.h b/webrtc/api/test/mockpeerconnectionobservers.h
index 23647f6de3b5cbf3d5c8e6fda845103c77a04441..1f000aff7950a89695ce63ca59276c7f52b8b05c 100644
--- a/webrtc/api/test/mockpeerconnectionobservers.h
+++ b/webrtc/api/test/mockpeerconnectionobservers.h
@@ -17,6 +17,7 @@
#include <string>
#include "webrtc/api/datachannelinterface.h"
+#include "webrtc/base/checks.h"
namespace webrtc {
@@ -109,7 +110,7 @@ class MockStatsObserver : public webrtc::StatsObserver {
virtual ~MockStatsObserver() {}
virtual void OnComplete(const StatsReports& reports) {
- ASSERT(!called_);
+ RTC_CHECK(!called_);
called_ = true;
stats_.Clear();
stats_.number_of_reports = reports.size();
@@ -143,37 +144,37 @@ class MockStatsObserver : public webrtc::StatsObserver {
double timestamp() const { return stats_.timestamp; }
int AudioOutputLevel() const {
- ASSERT(called_);
+ RTC_CHECK(called_);
return stats_.audio_output_level;
}
int AudioInputLevel() const {
- ASSERT(called_);
+ RTC_CHECK(called_);
return stats_.audio_input_level;
}
int BytesReceived() const {
- ASSERT(called_);
+ RTC_CHECK(called_);
return stats_.bytes_received;
}
int BytesSent() const {
- ASSERT(called_);
+ RTC_CHECK(called_);
return stats_.bytes_sent;
}
int AvailableReceiveBandwidth() const {
- ASSERT(called_);
+ RTC_CHECK(called_);
return stats_.available_receive_bandwidth;
}
std::string DtlsCipher() const {
- ASSERT(called_);
+ RTC_CHECK(called_);
return stats_.dtls_cipher;
}
std::string SrtpCipher() const {
- ASSERT(called_);
+ RTC_CHECK(called_);
return stats_.srtp_cipher;
}
« no previous file with comments | « webrtc/api/test/fakedatachannelprovider.h ('k') | webrtc/api/webrtcsession_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698