Remove ResultCatcher since test doesn't output any result, just succeeds or fails.

BUG=393969

Review URL: https://codereview.chromium.org/480173004

Cr-Commit-Position: refs/heads/master@{#291309}
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291309 0039d316-1c4b-4281-b951-d872f2087c98

chrome/test/data/extensions/api_test/cast_channel/api/test_open_error.js

@@ -9,7 +9,6 @@ var onClose = function(channel) {
   chrome.test.assertLastError('Unknown error.');
   assertClosedChannelWithError(channel, 'connect_error');
   chrome.test.succeed();
-  chrome.test.notifyPass();
 }
 
 var onError = function(channel, error) {
@@ -30,7 +29,6 @@ var onOpen = function(channel) {
 
 var maybeClose = function(channel) {
   if (errorEvent && openCallback) {
-    console.log("closing " + JSON.toString(channel));
     chrome.cast.channel.close(channel, onClose);
   }
 };

extensions/browser/api/cast_channel/cast_channel_apitest.cc

@@ -269,8 +269,7 @@ IN_PROC_BROWSER_TEST_F(CastChannelAPITest, MAYBE_TestGetLogs) {
 
 // TODO(munjal): Win Dbg has a workaround that makes RunExtensionSubtest
 // always return true without actually running the test. Remove when fixed.
-// Flaky on mac: crbug.com/393969
-#if (defined(OS_WIN) && !defined(NDEBUG)) || defined(OS_MACOSX)
+#if defined(OS_WIN) && !defined(NDEBUG)
 #define MAYBE_TestOpenError DISABLED_TestOpenError
 #else
 #define MAYBE_TestOpenError TestOpenError
@@ -291,9 +290,6 @@ IN_PROC_BROWSER_TEST_F(CastChannelAPITest, MAYBE_TestOpenError) {
 
   EXPECT_TRUE(RunExtensionSubtest("cast_channel/api",
                                   "test_open_error.html"));
-
-  ResultCatcher catcher;
-  EXPECT_TRUE(catcher.GetNextResult()) << catcher.message();
 }
 
 IN_PROC_BROWSER_TEST_F(CastChannelAPITest, TestOpenInvalidConnectInfo) {