Add an API test to exersize the open error case. This is a regression test for

chrome/browser/extensions/api/cast_channel/cast_channel_apitest.cc

Index: chrome/browser/extensions/api/cast_channel/cast_channel_apitest.cc
===================================================================
--- chrome/browser/extensions/api/cast_channel/cast_channel_apitest.cc	(revision 250590)
+++ chrome/browser/extensions/api/cast_channel/cast_channel_apitest.cc	(working copy)
@@ -16,14 +16,17 @@
 #include "net/base/completion_callback.h"
 #include "net/base/net_errors.h"
 #include "testing/gmock/include/gmock/gmock.h"
+#include "testing/gmock_mutant.h"
 
 namespace cast_channel =  extensions::api::cast_channel;
 using cast_channel::CastSocket;
+using cast_channel::ChannelError;
 using cast_channel::ChannelInfo;
 using cast_channel::MessageInfo;
 
+using ::testing::_;
 using ::testing::A;
-using ::testing::_;
+using ::testing::DoAll;
 using ::testing::Invoke;
 using ::testing::Return;
 
@@ -46,6 +49,10 @@
   ::std::tr1::get<k>(args).Run(result);
 }
 
+ACTION_P2(InvokeDelegateOnError, api_test, api) {
+  api_test->CallOnError(api);
+}
+
 class MockCastSocket : public CastSocket {
  public:
   explicit MockCastSocket(CastSocket::Delegate* delegate,
@@ -103,21 +110,25 @@
     channel_info->error_state = cast_channel::CHANNEL_ERROR_NONE;
   }
 
+  void CallOnError(extensions::CastChannelAPI* api) {
+    api->OnError(mock_cast_socket_,
+                 cast_channel::CHANNEL_ERROR_CONNECT_ERROR);
+  }
+
  protected:
   void CallOnMessage(const std::string& message) {
     content::BrowserThread::PostTask(
         content::BrowserThread::IO,
         FROM_HERE,
         base::Bind(&CastChannelAPITest::DoCallOnMessage, this,
-                   GetApi(), mock_cast_socket_, message));
+                   mock_cast_socket_, message));
   }
 
-  void DoCallOnMessage(extensions::CastChannelAPI* api,
-                       MockCastSocket* cast_socket,
+  void DoCallOnMessage(MockCastSocket* cast_socket,
                        const std::string& message) {
     MessageInfo message_info;
     FillMessageInfo(&message_info, message);
-    api->OnMessage(cast_socket, message_info);
+    GetApi()->OnMessage(cast_socket, message_info);
   }
 
   MockCastSocket* mock_cast_socket_;
@@ -182,3 +193,30 @@
   CallOnMessage("some-message");
   EXPECT_TRUE(catcher.GetNextResult()) << catcher.message();
 }
+
+// TODO(munjal): Win Dbg has a workaround that makes RunExtensionSubtest
+// always return true without actually running the test. Remove when fixed.
+#if defined(OS_WIN) && !defined(NDEBUG)
+#define MAYBE_TestOpenError DISABLED_TestOpenError
+#else
+#define MAYBE_TestOpenError TestOpenError
+#endif
+// Test the case when socket open results in an error.
+IN_PROC_BROWSER_TEST_F(CastChannelAPITest, MAYBE_TestOpenError) {
+  SetUpMockCastSocket();
+
+  EXPECT_CALL(*mock_cast_socket_, Connect(_))
+        .WillOnce(DoAll(

[Wez, 2014/03/04 21:29:53]

nit: Indentation

[Munjal (Google), 2014/05/21 22:41:06]

It is indented 4 spaces from .WillOnce. That is not good?

[Wez, 2014/05/21 22:57:34]

Indentation of .WillOnce is six from EXPECT_CALL, rather than four.

+            InvokeDelegateOnError(this, GetApi()),
+            InvokeCompletionCallback<0>(net::ERR_FAILED)));
+  EXPECT_CALL(*mock_cast_socket_, FillChannelInfo(_))
+      .WillOnce(Invoke(FillChannelInfoForClosedState))
+      .WillOnce(Invoke(FillChannelInfoForClosedState));
+  EXPECT_CALL(*mock_cast_socket_, Close(_));
+
+  EXPECT_TRUE(RunExtensionSubtest("cast_channel/api",
+                                  "test_open_error.html"));
+
+  ResultCatcher catcher;
+  EXPECT_TRUE(catcher.GetNextResult()) << catcher.message();
+}