Clean up old Mojo types for js_to_cpp::blink::ForTesting

This CL cleans up old Mojo types for js_to_cpp::blink::ForTesting
using PendingAssociatedRemote.
It also updates BitFlipResponse with pending_associated_remote
from js_to_cpp.mojom.

Bug: 955171
Change-Id: Ia17d27ce0776591346af404975e52615234a77c4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1942685
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
Cr-Commit-Position: refs/heads/master@{#720475}

third_party/blink/renderer/core/mojo/tests/js_to_cpp.mojom

@@ -43,9 +43,10 @@ interface CppSide {
   PingResponse();
   EchoResponse(EchoArgsList list);
 
-  // Having an associated interface pointer in the message makes sure the
-  // message header version 2 is tested.
-  BitFlipResponse(EchoArgsList arg, associated ForTesting? not_used);
+  // Having an associated remote in the message makes sure the message header
+  // version 2 is tested.
+  BitFlipResponse(EchoArgsList arg,
+                  pending_associated_remote<ForTesting>? not_used);
 
   BackPointerResponse(EchoArgsList arg);
 };

third_party/blink/renderer/core/mojo/tests/js_to_cpp_test.cc

@@ -3,6 +3,7 @@
 // found in the LICENSE file.
 
 #include "base/stl_util.h"
+#include "mojo/public/cpp/bindings/pending_associated_remote.h"
 #include "mojo/public/cpp/bindings/receiver.h"
 #include "mojo/public/cpp/system/wait.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ -242,7 +243,8 @@ class CppSideConnection : public js_to_cpp::blink::CppSide {
 
   void BitFlipResponse(
       js_to_cpp::blink::EchoArgsListPtr list,
-      js_to_cpp::blink::ForTestingAssociatedPtrInfo not_used) override {
+      mojo::PendingAssociatedRemote<js_to_cpp::blink::ForTesting> not_used)
+      override {
     mishandled_messages_ += 1;
   }
 
@@ -329,7 +331,8 @@ class BitFlipCppSideConnection : public CppSideConnection {
 
   void BitFlipResponse(
       js_to_cpp::blink::EchoArgsListPtr list,
-      js_to_cpp::blink::ForTestingAssociatedPtrInfo not_used) override {
+      mojo::PendingAssociatedRemote<js_to_cpp::blink::ForTesting> not_used)
+      override {
     CheckCorruptedEchoArgsList(list);
   }