Commit ebc9bdcc authored by Chris Hamilton's avatar Chris Hamilton Committed by Commit Bot

Disable freeze/discard links for visible tabs.

This is non-sensical and is a way for a user to crash the browser.

BUG=

Cq-Include-Trybots: luci.chromium.try:closure_compilation
Change-Id: I913e0197f3b8d8bda60fa28729d4e5e0bfafbf4b
Reviewed-on: https://chromium-review.googlesource.com/1140790Reviewed-by: default avatarSébastien Marchand <sebmarchand@chromium.org>
Commit-Queue: Chris Hamilton <chrisha@chromium.org>
Cr-Commit-Position: refs/heads/master@{#575819}
parent 61951be8
...@@ -368,14 +368,17 @@ cr.define('discards', function() { ...@@ -368,14 +368,17 @@ cr.define('discards', function() {
let discardLink = row.querySelector('.discard-link'); let discardLink = row.querySelector('.discard-link');
let discardUrgentLink = row.querySelector('.discard-urgent-link'); let discardUrgentLink = row.querySelector('.discard-urgent-link');
// Determine which action links should be enabled/disabled. // Determine which action links should be enabled/disabled. By default
// everything is disabled and links are selectively enabled depending on the
// tab state.
let loadEnabled = false; let loadEnabled = false;
let freezeEnabled = false; let freezeEnabled = false;
let discardEnabled = false; let discardEnabled = false;
let discardUrgentEnabled = false; let discardUrgentEnabled = false;
if (info.loadingState == mojom.LifecycleUnitLoadingState.UNLOADED) { if (info.loadingState == mojom.LifecycleUnitLoadingState.UNLOADED) {
loadEnabled = true; loadEnabled = true;
} else { } else if (info.visibility == 0) {
// Only tabs that aren't visible can be frozen or discarded for now.
freezeEnabled = true; freezeEnabled = true;
discardEnabled = true; discardEnabled = true;
discardUrgentEnabled = true; discardUrgentEnabled = true;
......
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