Commit 50212b8a authored by Patrick Monette's avatar Patrick Monette Committed by Commit Bot

[PM] Improve comment that explain tear down of the PerformanceManager

The comment was reworded to reduce repetitions and now references
worker nodes.

Bug: 993029
Change-Id: Id854cbfa86fc8367e98a5057cd002f9c034f67e0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2094712
Auto-Submit: Patrick Monette <pmonette@chromium.org>
Reviewed-by: default avatarSigurður Ásgeirsson <siggi@chromium.org>
Commit-Queue: Patrick Monette <pmonette@chromium.org>
Cr-Commit-Position: refs/heads/master@{#749361}
parent 515a7bd7
...@@ -157,11 +157,10 @@ void ChromeBrowserMainExtraPartsPerformanceManager::PostMainMessageLoopRun() { ...@@ -157,11 +157,10 @@ void ChromeBrowserMainExtraPartsPerformanceManager::PostMainMessageLoopRun() {
page_load_tracker_decorator_helper_.reset(); page_load_tracker_decorator_helper_.reset();
page_live_state_data_helper_.reset(); page_live_state_data_helper_.reset();
// There may still be WebContents and RenderProcessHosts with attached user // There may still be worker hosts, WebContents and RenderProcessHosts with
// data, retaining PageNodes, FrameNodes and ProcessNodes. Tear down the // attached user data, retaining WorkerNodes, PageNodes, FrameNodes and
// registry to release these nodes. There is no convenient later call-out to // ProcessNodes. Tear down the registry to release these nodes. After this,
// destroy the performance manager after all WebContents and // there is no convenient call-out to destroy the performance manager.
// RenderProcessHosts have been destroyed.
registry_->TearDown(); registry_->TearDown();
registry_.reset(); registry_.reset();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment