Chromium Code Reviews| Index: webrtc/media/engine/webrtcvoiceengine_unittest.cc |
| diff --git a/webrtc/media/engine/webrtcvoiceengine_unittest.cc b/webrtc/media/engine/webrtcvoiceengine_unittest.cc |
| index 612d5c2e6b0fb27eef815b91f8c7272c767877d2..40ed66ad9c63f2f3a6b7354a50f19153ed439cae 100644 |
| --- a/webrtc/media/engine/webrtcvoiceengine_unittest.cc |
| +++ b/webrtc/media/engine/webrtcvoiceengine_unittest.cc |
| @@ -253,6 +253,9 @@ class WebRtcVoiceEngineTestFake : public testing::Test { |
| // Check we fail if the ssrc is invalid. |
| EXPECT_FALSE(channel_->InsertDtmf(-1, 1, 111)); |
| + // Check that we fail if we specify a too short duration |
|
Taylor Brandstetter
2017/02/16 18:18:22
nit: Period at end of comment, "duration that's to
|
| + EXPECT_FALSE(channel_->InsertDtmf(ssrc, 2, 39)); |
|
Taylor Brandstetter
2017/02/16 18:18:22
There should also be a test somewhere that "Insert
|
| + |
| // Test send. |
| cricket::FakeAudioSendStream::TelephoneEvent telephone_event = |
| GetSendStream(kSsrc1).GetLatestTelephoneEvent(); |