Revert "Reland "[PM] Connect dedicated workers to their client frames in the graph.""

This reverts commit 5047802a.

Reason for revert: https://crbug.com/1066450

Original change's description:
> Reland "[PM] Connect dedicated workers to their client frames in the graph."
>
> This is a reland of 8e70db4d
>
> Original change's description:
> > [PM] Connect dedicated workers to their client frames in the graph.
> >
> > Bug: 993029
> > Change-Id: Iae20fa0ede4421782fc5fbf2851d21c83cfbf4c8
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095452
> > Commit-Queue: Patrick Monette <pmonette@chromium.org>
> > Reviewed-by: Sigurður Ásgeirsson <siggi@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#753319}
>
> Bug: 993029
> Change-Id: I859f004b938164a02e43b74e6f9225d25e90d083
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2122476
> Reviewed-by: Sigurður Ásgeirsson <siggi@chromium.org>
> Commit-Queue: Patrick Monette <pmonette@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#754620}

TBR=pmonette@chromium.org,siggi@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 993029, 1066450
Change-Id: I738ef7877b1b841eb9aea5af0a3427959e5857cb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2134199
Commit-Queue: Patrick Monette <pmonette@chromium.org>
Reviewed-by: Sigurður Ásgeirsson <siggi@chromium.org>
Reviewed-by: Patrick Monette <pmonette@chromium.org>
Cr-Commit-Position: refs/heads/master@{#755965}

components/performance_manager/worker_watcher.cc

@@ -132,11 +132,12 @@ void WorkerWatcher::OnWorkerStarted(
       browser_context_id_, WorkerNode::WorkerType::kDedicated,
       process_node_source_->GetProcessNode(worker_process_id),
       base::UnguessableToken::Create());
-  AddClientFrame(worker_node.get(), ancestor_render_frame_host_id);
   bool inserted = dedicated_worker_nodes_
                       .emplace(dedicated_worker_id, std::move(worker_node))
                       .second;
   DCHECK(inserted);
+
+  // TODO(pmonette): Connect |worker_node| to its client frame.
 }
 
 void WorkerWatcher::OnBeforeWorkerTerminated(
@@ -144,15 +145,15 @@ void WorkerWatcher::OnBeforeWorkerTerminated(
     content::GlobalFrameRoutingId ancestor_render_frame_host_id) {
   auto it = dedicated_worker_nodes_.find(dedicated_worker_id);
   DCHECK(it != dedicated_worker_nodes_.end());
-  auto worker_node = std::move(it->second);
-  dedicated_worker_nodes_.erase(it);
 
-  RemoveClientFrame(worker_node.get(), ancestor_render_frame_host_id);
+  auto worker_node = std::move(it->second);
+  // TODO(pmonette): Disconnect |worker_node| from its client frame.
 #if DCHECK_IS_ON()
   DCHECK(!base::Contains(clients_to_remove_, worker_node.get()));
 #endif  // DCHECK_IS_ON()
-
   PerformanceManagerImpl::DeleteNode(std::move(worker_node));
+
+  dedicated_worker_nodes_.erase(it);
 }
 
 void WorkerWatcher::OnFinalResponseURLDetermined(

components/performance_manager/worker_watcher_unittest.cc

@@ -727,7 +727,9 @@ TEST_F(WorkerWatcherTest, SimpleDedicatedWorker) {
         EXPECT_EQ(worker_node->worker_type(),
                   WorkerNode::WorkerType::kDedicated);
         EXPECT_EQ(worker_node->process_node(), process_node);
-        EXPECT_TRUE(IsWorkerClient(worker_node, client_frame_node));
+        // TODO(pmonette): Change the following to EXPECT_TRUE when the
+        //                 dedicated worker node gets hooked up correctly.
+        EXPECT_FALSE(IsWorkerClient(worker_node, client_frame_node));
       }));
 
   // Disconnect and clean up the worker.
@@ -976,7 +978,9 @@ TEST_F(WorkerWatcherTest, FrameDestroyed) {
         EXPECT_TRUE(graph->NodeInGraph(dedicated_worker_node));
         EXPECT_TRUE(graph->NodeInGraph(shared_worker_node));
         EXPECT_TRUE(graph->NodeInGraph(service_worker_node));
-        EXPECT_TRUE(IsWorkerClient(dedicated_worker_node, client_frame_node));
+        // TODO(pmonette): Change the following to EXPECT_TRUE when the
+        //                 dedicated worker node gets hooked up correctly.
+        EXPECT_FALSE(IsWorkerClient(dedicated_worker_node, client_frame_node));
         EXPECT_TRUE(IsWorkerClient(shared_worker_node, client_frame_node));
         // TODO(pmonette): Change the following to EXPECT_TRUE when the
         //                 service worker node gets hooked up correctly.