Chromium Code Reviews| Index: jingle/notifier/base/xmpp_connection_unittest.cc |
| diff --git a/jingle/notifier/base/xmpp_connection_unittest.cc b/jingle/notifier/base/xmpp_connection_unittest.cc |
| index 682c8d10d843ada4222768ca8a27450687d97feb..2e620920d9ae473d4d7970c772944678f066413c 100644 |
| --- a/jingle/notifier/base/xmpp_connection_unittest.cc |
| +++ b/jingle/notifier/base/xmpp_connection_unittest.cc |
| @@ -103,7 +103,7 @@ TEST_F(XmppConnectionTest, CreateDestroy) { |
| &mock_xmpp_connection_delegate_, NULL); |
| } |
| -#if !defined(_MSC_VER) || _MSC_VER < 1700 // http://crbug.com/158570 |
| +#if !defined(_MSC_VER) // http://crbug.com/158570 |
|
scottmg
2014/12/13 20:11:02
These are just buggy. :( I guess add 158570 to the
Mostyn Bramley-Moore
2014/12/13 20:22:45
Done.
|
| TEST_F(XmppConnectionTest, ImmediateFailure) { |
| // ChromeAsyncSocket::Connect() will always return false since we're |
| // not setting a valid host, but this gets bubbled up as ERROR_NONE |
| @@ -210,7 +210,7 @@ TEST_F(XmppConnectionTest, MultipleConnect) { |
| }, "more than once"); |
| } |
| -#if !defined(_MSC_VER) || _MSC_VER < 1700 // http://crbug.com/158570 |
| +#if !defined(_MSC_VER) // http://crbug.com/158570 |
| TEST_F(XmppConnectionTest, ConnectThenError) { |
| base::WeakPtr<rtc::Task> weak_ptr; |
| EXPECT_CALL(mock_xmpp_connection_delegate_, OnConnect(_)). |