Bug 40701: Add in pane security warning when downloading a file (34a3d408) · Commits · The Tor Project / Applications / Tor Browser

browser/components/downloads/content/downloads.css

+26 −0

Original line number	Diff line number	Diff line
		@@ -95,6 +95,32 @@
		padding: 0.62em;
		}

		#downloadsPanel-mainView {
		/* Fix the layout to ensure the #downloadsWarningDescription is given enough
		* vertical space. For tor-browser#40701.
		* TODO: May no longer be necessary after esr 115 due to bugzilla bug 1816455.
		*/
		display: flex;
		flex-direction: column;
		}

		#downloadsWarning p {
		padding-inline: 8px;
		margin-block-start: 8px;
		margin-block-end: 0;
		line-height: 1.4;
		}

		#downloadsWarningHeaderTitle {
		font-weight: bold;
		}

		#downloadsWarningDescription {
		/* Make sure we wrap the text rather than request the default max-content
		* width from the parent XUL -moz-box. */
		width: 0;
		}

		#downloadsHistory,
		#downloadsFooterButtons {
		margin: 0;

browser/components/downloads/content/downloads.js

+97 −11

Original line number	Diff line number	Diff line
		@@ -31,6 +31,10 @@

		"use strict";

		const PREF_SHOW_DOWNLOAD_WARNING = "browser.download.showTorWarning";

		const TAILS_URI = "https://tails.boum.org/";

		var { XPCOMUtils } = ChromeUtils.importESModule(
		"resource://gre/modules/XPCOMUtils.sys.mjs"
		);
		@@ -77,6 +81,13 @@ var DownloadsPanel = {
		*/
		_state: 0,

		/**
		* State of tor's download warning. It should only be initialized once (text assigned,
		* button commands assigned). This tracks if that has been performed and prevents
		* repeats.
		*/
		_torWarningInitialized: false,

		/** The panel is not linked to downloads data yet. */
		get kStateUninitialized() {
		return 0;
		@@ -110,6 +121,15 @@ var DownloadsPanel = {
		);
		}

		let showDownloadWarning = Services.prefs.getBoolPref(
		PREF_SHOW_DOWNLOAD_WARNING
		);
		if (!showDownloadWarning) {
		document.getElementById("downloadsWarning").hidden = true;
		} else {
		document.getElementById("downloadsWarning").hidden = false;
		}

		if (this._state != this.kStateUninitialized) {
		DownloadsCommon.log("DownloadsPanel is already initialized.");
		return;
		@@ -133,6 +153,43 @@ var DownloadsPanel = {
		DownloadsSummary
		);

		if (!this._torWarningInitialized) {
		document.getElementById("downloadsWarningHeaderTitle").textContent =
		this._getString("torbutton.download.warning.title");
		let tailsBrandName = this._getString(
		"torbutton.download.warning.tails_brand_name"
		);

		let warningDescriptionText = this._getString(
		"torbutton.download.warning.description"
		);
		let [head, rest] = warningDescriptionText.split("%S");
		const tailsLink = document.createElement("a");
		tailsLink.setAttribute("href", TAILS_URI);
		tailsLink.textContent = tailsBrandName.trim();
		tailsLink.addEventListener("click", event => {
		event.preventDefault();
		this.hidePanel();
		openWebLinkIn(TAILS_URI, "tab");
		});

		let downloadsWarningDescription = document.getElementById(
		"downloadsWarningDescription"
		);
		downloadsWarningDescription.append(head, tailsLink, rest);

		let dismissBtn = document.getElementById("downloadWarningDismiss");
		dismissBtn.textContent = this._getString(
		"torbutton.download.warning.dismiss"
		);
		dismissBtn.addEventListener("click", event => {
		Services.prefs.setBoolPref(PREF_SHOW_DOWNLOAD_WARNING, false);
		document.getElementById("downloadsWarning").hidden = true;
		this._focusPanel(true);
		});
		this._torWarningInitialized = true;
		}

		DownloadsCommon.log(
		"DownloadsView attached - the panel for this window",
		"should now see download items come in."
		@@ -506,8 +563,11 @@ var DownloadsPanel = {
		/**
		* Move focus to the main element in the downloads panel, unless another
		* element in the panel is already focused.
		*
		* @param {bool} [forceFocus=false] - Whether to force move the focus.
		*/
		_focusPanel() {
		_focusPanel(forceFocus = false) {
		if (!forceFocus) {
		// We may be invoked while the panel is still waiting to be shown.
		if (this._state != this.kStateShown) {
		return;
		@@ -520,6 +580,8 @@ var DownloadsPanel = {
		) {
		return;
		}
		}

		let focusOptions = {};
		if (this._preventFocusRing) {
		focusOptions.focusVisible = false;
		@@ -643,6 +705,30 @@ var DownloadsPanel = {
		}
		}, 0);
		},

		/**
		* Get a string from the properties bundle.
		*
		* @param {string} name - The string name.
		*
		* @return {string} The string.
		*/
		_getString(name) {
		if (!this._stringBundle) {
		this._stringBundle = Services.strings.createBundle(
		"chrome://torbutton/locale/torbutton.properties"
		);
		}
		try {
		return this._stringBundle.GetStringFromName(name);
		} catch {}
		if (!this._fallbackStringBundle) {
		this._fallbackStringBundle = Services.strings.createBundle(
		"resource://torbutton/locale/en-US/torbutton.properties"
		);
		}
		return this._fallbackStringBundle.GetStringFromName(name);
		},
		};

		XPCOMUtils.defineConstant(this, "DownloadsPanel", DownloadsPanel);

browser/components/downloads/content/downloadsPanel.inc.xhtml

+15 −0

Original line number	Diff line number	Diff line
		@@ -124,6 +124,21 @@
		disablekeynav="true">

		<panelview id="downloadsPanel-mainView">
		<vbox id="downloadsWarning">
		<vbox role="alert"
		aria-labelledby="downloadsWarningHeaderTitle"
		aria-describedby="downloadsWarningDescription">
		<html:p id="downloadsWarningHeaderTitle"></html:p>
		<html:p id="downloadsWarningDescription">
		</html:p>

		<html:div class="panel-footer">
		<html:button id="downloadWarningDismiss">
		</html:button>
		</html:div>
		</vbox>
		<toolbarseparator />
		</vbox>
		<vbox class="panel-view-body-unscrollable">
		<richlistbox id="downloadsListBox"
		data-l10n-id="downloads-panel-items"

browser/themes/shared/customizableui/panelUI-shared.css

+1 −0

Original line number	Diff line number	Diff line
		@@ -1269,6 +1269,7 @@ panelview .toolbarbutton-1 {
		}

		/* TODO: Check if we can/should merge with the above rule */
		#downloadsWarning toolbarseparator,
		#securityLevel-panel toolbarseparator {
		appearance: none;
		min-height: 0;

browser/themes/shared/downloads/downloads.inc.css

+1 −0

Original line number	Diff line number	Diff line
		@@ -266,6 +266,7 @@
		}

		/* Toolbarseparator */
		#downloadsWarning toolbarseparator,
		#downloadsFooterButtons > toolbarseparator {
		margin-inline: 0;
		}