Remove HostFrameSinkManagerLocalTest

With OOP-D on all platforms, HostFrameSinkManager will never be in the
same process as FrameSinkManagerImpl, so HostFrameSinkManagerLocalTest
no longer makes much sense. Merge it with HostFrameSinkManagerRemoteTest
into HostFrameSinkManagerTest, and refactor the tests to work on remote.

CommunicateFrameToken and HitTestAggregatorQuery require access to
CompositorFrameSinkSupport, but they test things out of scope of this
class, so they can be removed.

Bug: 1033683
Change-Id: I32e1150d7f7858c24f7733b8ad5e62ac242a6fac
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1972895Reviewed-by: kylechar <kylechar@chromium.org>
Commit-Queue: Sean Gilhuly <sgilhuly@chromium.org>
Cr-Commit-Position: refs/heads/master@{#732595}

components/viz/host/host_frame_sink_manager.h

@@ -206,8 +206,8 @@ class VIZ_HOST_EXPORT HostFrameSinkManager
   void EvictCachedBackBuffer(uint32_t cache_id);
 
  private:
+  friend class HostFrameSinkManagerTest;
   friend class HostFrameSinkManagerTestApi;
-  friend class HostFrameSinkManagerTestBase;
 
   struct FrameSinkData {
     FrameSinkData();