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

Unified Diff: chromeos/components/tether/disconnect_tethering_operation_unittest.cc

Issue 2915713002: Tether MessageTransferOperation: Only wait for a response from a host for a certain amount of time … (Closed)
Patch Set: khorimoto@ comments. Created 3 years, 7 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: chromeos/components/tether/disconnect_tethering_operation_unittest.cc
diff --git a/chromeos/components/tether/disconnect_tethering_operation_unittest.cc b/chromeos/components/tether/disconnect_tethering_operation_unittest.cc
index cfdac1e5cdf4e10c3da65c96a455b01eb343b358..49e3b17dd670eb7f48bca789792fe7f4f014f588 100644
--- a/chromeos/components/tether/disconnect_tethering_operation_unittest.cc
+++ b/chromeos/components/tether/disconnect_tethering_operation_unittest.cc
@@ -78,6 +78,10 @@ class DisconnectTetheringOperationTest : public testing::Test {
ASSERT_EQ(1u, sent_messages.size());
EXPECT_EQ(test_device_, sent_messages[0].remote_device);
EXPECT_EQ(disconnect_tethering_request_string_, sent_messages[0].message);
+
+ // There should be no response timers set for DisconnectTetheringOperation.
+ EXPECT_FALSE(
+ operation_->GetResponseTimerForDevice(sent_messages[0].remote_device));
}
void SimulateConnectionTimeout() {

Powered by Google App Engine
This is Rietveld 408576698