Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4140)

Unified Diff: chrome/browser/memory/tab_manager.cc

Issue 2399293002: Add MemoryCoordinatorDelegate (Closed)
Patch Set: Add a separate delegate class Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/memory/tab_manager.h ('k') | chrome/browser/memory/tab_manager_browsertest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/memory/tab_manager.cc
diff --git a/chrome/browser/memory/tab_manager.cc b/chrome/browser/memory/tab_manager.cc
index 582b2e667099f6fa0085f6721c4b9c69a37709c8..8c53fe7f154a316855deff74eb78b644af589fc2 100644
--- a/chrome/browser/memory/tab_manager.cc
+++ b/chrome/browser/memory/tab_manager.cc
@@ -418,6 +418,23 @@ void TabManager::SetTabAutoDiscardableState(content::WebContents* contents,
GetWebContentsData(contents)->SetAutoDiscardableState(state);
}
+bool TabManager::CanSuspendBackgroundedRenderer(int render_process_id) {
+ // A renderer can be suspended if it's not playing media.
+ auto tab_stats = GetUnsortedTabStats();
+ for (auto& tab : tab_stats) {
+ if (tab.child_process_host_id != render_process_id)
+ continue;
+ TabStripModel* model;
+ int index = FindTabStripModelById(tab.tab_contents_id, &model);
+ if (index == -1)
+ return false;
+ WebContents* web_contents = model->GetWebContentsAt(index);
+ if (IsMediaTab(web_contents))
+ return false;
+ }
+ return true;
+}
+
// static
bool TabManager::CompareTabStats(const TabStats& first,
const TabStats& second) {
« no previous file with comments | « chrome/browser/memory/tab_manager.h ('k') | chrome/browser/memory/tab_manager_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698