[Chromoting] Fix unit test warnings introduced in r146455.

remoting/protocol/jingle_session_unittest.cc

Index: remoting/protocol/jingle_session_unittest.cc
diff --git a/remoting/protocol/jingle_session_unittest.cc b/remoting/protocol/jingle_session_unittest.cc
index 95a1de50a6e0b23a7a9f945bcc776ad60965e7b9..cab23af9c2ff2828e1c74a40209526aa1459dedf 100644
--- a/remoting/protocol/jingle_session_unittest.cc
+++ b/remoting/protocol/jingle_session_unittest.cc
@@ -196,6 +196,9 @@ class JingleSessionTest : public testing::Test {
             .Times(AtMost(1));
       }
     }
+    EXPECT_CALL(host_session_event_handler_,
+                OnSessionRouteChange(kChannelName, _))
+        .Times(AtMost(1));

[Sergey Ulanov, 2012/07/23 21:44:26]

It might be flaky with AtMost(1). It should be AtLeast(1) instead.

[simonmorris, 2012/07/23 22:00:06]

That makes the Connect* tests fail. I've removed this expectation for those
tests.

 
     {
       InSequence dummy;
@@ -217,6 +220,9 @@ class JingleSessionTest : public testing::Test {
             .Times(AtMost(1));
       }
     }
+    EXPECT_CALL(client_session_event_handler_,
+                OnSessionRouteChange(kChannelName, _))
+        .Times(AtMost(1));
 
     scoped_ptr<Authenticator> authenticator(new FakeAuthenticator(
         FakeAuthenticator::CLIENT, auth_round_trips, auth_action, true));