Remove ChattyServer ipc_tests, which are slow & unnecessary.

These tests were added to guard against chatty synchronous IPC causing
Windows' PostMessage() limit being reached, and causing messages to be
dropped.

Since Chrome IPC now runs over Mojo, and our message pumps do not call
PostMessage() on each task, and these tests are slow and timeout on
Android & Fuchsia ARM64, just remove the tests.

Bug: 734791, 339980
Change-Id: I26c105f3685995e24011bbc90ed9a391833ec4e2
Reviewed-on: https://chromium-review.googlesource.com/c/1424162
Reviewed-by: Ken Rockot <rockot@google.com>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625053}
diff --git a/ipc/ipc_sync_channel_unittest.cc b/ipc/ipc_sync_channel_unittest.cc
index f5134a51..a1bd7a5 100644
--- a/ipc/ipc_sync_channel_unittest.cc
+++ b/ipc/ipc_sync_channel_unittest.cc
@@ -945,60 +945,6 @@
 
 //------------------------------------------------------------------------------
 
-class ChattyClient : public Worker {
- public:
-  explicit ChattyClient(mojo::ScopedMessagePipeHandle channel_handle)
-      : Worker(Channel::MODE_CLIENT,
-               "chatty_client",
-               std::move(channel_handle)) {}
-
-  void OnAnswer(int* answer) override {
-    // The PostMessage limit is 10k.  Send 20% more than that.
-    const int kMessageLimit = 10000;
-    const int kMessagesToSend = kMessageLimit * 120 / 100;
-    for (int i = 0; i < kMessagesToSend; ++i) {
-      if (!SendDouble(false, true))
-        break;
-    }
-    *answer = 42;
-    Done();
-  }
-};
-
-void ChattyServer(bool pump_during_send) {
-  std::vector<Worker*> workers;
-  mojo::MessagePipe pipe;
-  workers.push_back(
-      new UnblockServer(pump_during_send, false, std::move(pipe.handle0)));
-  workers.push_back(new ChattyClient(std::move(pipe.handle1)));
-  RunTest(workers);
-}
-
-#if defined(OS_ANDROID)
-// Times out.
-#define MAYBE_ChattyServer DISABLED_ChattyServer
-#else
-#define MAYBE_ChattyServer ChattyServer
-#endif
-// Tests http://b/1093251 - that sending lots of sync messages while
-// the receiver is waiting for a sync reply does not overflow the PostMessage
-// queue.
-TEST_F(IPCSyncChannelTest, MAYBE_ChattyServer) {
-  ChattyServer(false);
-}
-
-#if defined(OS_ANDROID)
-// Times out.
-#define MAYBE_ChattyServerPumpDuringSend DISABLED_ChattyServerPumpDuringSend
-#else
-#define MAYBE_ChattyServerPumpDuringSend ChattyServerPumpDuringSend
-#endif
-TEST_F(IPCSyncChannelTest, MAYBE_ChattyServerPumpDuringSend) {
-  ChattyServer(true);
-}
-
-//------------------------------------------------------------------------------
-
 void NestedCallback(Worker* server) {
   // Sleep a bit so that we wake up after the reply has been received.
   base::PlatformThread::Sleep(base::TimeDelta::FromMilliseconds(250));
diff --git a/testing/buildbot/chromium.fyi.json b/testing/buildbot/chromium.fyi.json
index 17b99a1..5e4f452 100644
--- a/testing/buildbot/chromium.fyi.json
+++ b/testing/buildbot/chromium.fyi.json
@@ -3986,9 +3986,6 @@
         "test": "http_service_tests"
       },
       {
-        "args": [
-          "--test-launcher-filter-file=../../testing/buildbot/filters/fuchsia.ipc_tests.filter"
-        ],
         "swarming": {
           "can_use_on_swarming_builders": true
         },
@@ -4192,9 +4189,6 @@
         "test": "http_service_tests"
       },
       {
-        "args": [
-          "--test-launcher-filter-file=../../testing/buildbot/filters/fuchsia.ipc_tests.filter"
-        ],
         "swarming": {
           "can_use_on_swarming_builders": true,
           "dimension_sets": [
@@ -4453,9 +4447,6 @@
         "test": "http_service_tests"
       },
       {
-        "args": [
-          "--test-launcher-filter-file=../../testing/buildbot/filters/fuchsia.ipc_tests.filter"
-        ],
         "swarming": {
           "can_use_on_swarming_builders": true,
           "dimension_sets": [
diff --git a/testing/buildbot/chromium.linux.json b/testing/buildbot/chromium.linux.json
index a6b5388..0b67a8d 100644
--- a/testing/buildbot/chromium.linux.json
+++ b/testing/buildbot/chromium.linux.json
@@ -780,7 +780,6 @@
       },
       {
         "args": [
-          "--test-launcher-filter-file=../../testing/buildbot/filters/fuchsia.ipc_tests.filter",
           "--qemu-require-kvm"
         ],
         "swarming": {
diff --git a/testing/buildbot/filters/BUILD.gn b/testing/buildbot/filters/BUILD.gn
index 7d03993..d75dcf9 100644
--- a/testing/buildbot/filters/BUILD.gn
+++ b/testing/buildbot/filters/BUILD.gn
@@ -63,9 +63,10 @@
 
 source_set("fuchsia_filters") {
   data = [
+    "//testing/buildbot/filters/fuchsia.base_perftests.filter",
     "//testing/buildbot/filters/fuchsia.content_unittests.filter",
-    "//testing/buildbot/filters/fuchsia.ipc_tests.filter",
     "//testing/buildbot/filters/fuchsia.mojo_unittests.filter",
+    "//testing/buildbot/filters/fuchsia.net_perftests.filter",
     "//testing/buildbot/filters/fuchsia.net_unittests.filter",
     "//testing/buildbot/filters/fuchsia.ui_base_unittests.filter",
   ]
diff --git a/testing/buildbot/filters/fuchsia.ipc_tests.filter b/testing/buildbot/filters/fuchsia.ipc_tests.filter
deleted file mode 100644
index abcb4c76..0000000
--- a/testing/buildbot/filters/fuchsia.ipc_tests.filter
+++ /dev/null
@@ -1,4 +0,0 @@
-# TODO(fuchsia): Being ported, see https://crbug.com/734791
-
-# These tests are too slow under Fuchsia in QEMU w/out KVM.
--IPCSyncChannelTest.ChattyServer*
diff --git a/testing/buildbot/test_suites.pyl b/testing/buildbot/test_suites.pyl
index 0ec7df9..409cd30 100644
--- a/testing/buildbot/test_suites.pyl
+++ b/testing/buildbot/test_suites.pyl
@@ -2732,11 +2732,7 @@
       'fidlgen_js_unittests': {},
       'gpu_unittests': {},
       'http_service_tests': {},
-      'ipc_tests': {
-        'args': [
-          '--test-launcher-filter-file=../../testing/buildbot/filters/fuchsia.ipc_tests.filter',
-        ],
-      },
+      'ipc_tests': {},
       'media_unittests': {},
       'mojo_unittests': {
         'args': [