[tbb-commits] [Git][tpo/applications/tor-browser][base-browser-115.11.0esr-13.5-1] 2 commits: Revert "Bug 41881: Don't persist custom network requests on private windows"
richard (@richard)
git at gitlab.torproject.org
Mon May 13 20:40:10 UTC 2024
richard pushed to branch base-browser-115.11.0esr-13.5-1 at The Tor Project / Applications / Tor Browser
Commits:
1e2cec74 by Pier Angelo Vendrame at 2024-05-13T20:39:35+00:00
Revert "Bug 41881: Don't persist custom network requests on private windows"
This reverts commit 6450a4e62e810832f35ee8732a4630ec2c2ba6f6.
- - - - -
fa7dafa6 by cypherpunks1 at 2024-05-13T20:39:43+00:00
Bug 1892052 - Do not persist custom network requests on PBM. r=devtools-reviewers,bomsy,nchevobbe
Differential Revision: https://phabricator.services.mozilla.com/D207762
- - - - -
3 changed files:
- devtools/client/netmonitor/src/components/new-request/HTTPCustomRequestPanel.js
- devtools/client/netmonitor/test/browser_net_new_request_panel_persisted_content.js
- devtools/client/netmonitor/test/head.js
Changes:
=====================================
devtools/client/netmonitor/src/components/new-request/HTTPCustomRequestPanel.js
=====================================
@@ -4,11 +4,6 @@
"use strict";
-const lazy = {};
-ChromeUtils.defineESModuleGetters(lazy, {
- PrivateBrowsingUtils: "resource://gre/modules/PrivateBrowsingUtils.sys.mjs",
-});
-
const {
Component,
createFactory,
@@ -127,7 +122,7 @@ class HTTPCustomRequestPanel extends Component {
async componentDidMount() {
let { connector, request } = this.props;
- if (!lazy.PrivateBrowsingUtils.isWindowPrivate(window)) {
+ if (!connector.currentTarget?.targetForm?.isPrivate) {
const persistedCustomRequest = await asyncStorage.getItem(
"devtools.netmonitor.customRequest"
);
@@ -198,7 +193,7 @@ class HTTPCustomRequestPanel extends Component {
}
componentWillUnmount() {
- if (!lazy.PrivateBrowsingUtils.isWindowPrivate(window)) {
+ if (!this.props.connector.currentTarget?.targetForm?.isPrivate) {
asyncStorage.setItem("devtools.netmonitor.customRequest", this.state);
}
}
=====================================
devtools/client/netmonitor/test/browser_net_new_request_panel_persisted_content.js
=====================================
@@ -9,30 +9,14 @@ const asyncStorage = require("resource://devtools/shared/async-storage.js");
* Test if content is still persisted after the panel is closed
*/
-add_task(async function () {
- // Turn true the pref
- await pushPref("devtools.netmonitor.features.newEditAndResend", true);
- // Reset the storage for the persisted custom request
- await asyncStorage.removeItem("devtools.netmonitor.customRequest");
-
- const { monitor } = await initNetMonitor(HTTPS_CUSTOM_GET_URL, {
- requestCount: 1,
- });
- info("Starting test... ");
-
- const { document, store, windowRequire } = monitor.panelWin;
-
- // Action should be processed synchronously in tests.
- const Actions = windowRequire("devtools/client/netmonitor/src/actions/index");
- store.dispatch(Actions.batchEnable(false));
-
- info("open the left panel");
- let waitForPanels = waitForDOM(
+async function addCustomRequestTestContent(tab, monitor, document) {
+ info("Open the left panel");
+ const waitForPanels = waitForDOM(
document,
".monitor-panel .network-action-bar"
);
- let HTTPCustomRequestButton = document.querySelector(
+ const HTTPCustomRequestButton = document.querySelector(
"#netmonitor-toolbar-container .devtools-http-custom-request-icon"
);
HTTPCustomRequestButton.click();
@@ -59,91 +43,166 @@ add_task(async function () {
"#http-custom-query .map-add-new-inputs .http-custom-input-name"
);
newParameterName.focus();
- EventUtils.sendString("My-param");
+ EventUtils.sendString("My-param", monitor.panelWin);
info("Adding new headers");
const newHeaderName = document.querySelector(
"#http-custom-headers .map-add-new-inputs .http-custom-input-name"
);
newHeaderName.focus();
- EventUtils.sendString("My-header");
+ EventUtils.sendString("My-header", monitor.panelWin);
const newHeaderValue = Array.from(
document.querySelectorAll(
"#http-custom-headers .http-custom-input .http-custom-input-value"
)
- ).pop();
+ ).at(-1);
newHeaderValue.focus();
- EventUtils.sendString("my-value");
+ EventUtils.sendString("my-value", monitor.panelWin);
const postValue = document.querySelector("#http-custom-postdata-value");
postValue.focus();
- EventUtils.sendString("{'Name': 'Value'}");
+ EventUtils.sendString("{'Name': 'Value'}", monitor.panelWin);
- // Close the panel
+ info("Close the panel");
const closePanel = document.querySelector(
".network-action-bar .tabs-navigation .sidebar-toggle"
);
closePanel.click();
+}
- // Open the panel again to see if the content is still there
- waitForPanels = waitUntil(
+async function runTests(tab, monitor, document, isPrivate = false) {
+ info("Open the panel again to see if the content is still there");
+ const waitForPanels = waitFor(
() =>
document.querySelector(".http-custom-request-panel") &&
document.querySelector("#http-custom-request-send-button").disabled ===
- false
+ isPrivate
);
- HTTPCustomRequestButton = document.querySelector(
+ const HTTPCustomRequestButton = document.querySelector(
"#netmonitor-toolbar-container .devtools-http-custom-request-icon"
);
HTTPCustomRequestButton.click();
await waitForPanels;
- is(
- methodValue.value,
- "POST",
- "The content should still be there after the user close the panel and re-opened"
- );
-
- is(
- url.value,
- "https://www.example.com?My-param=",
- "The url should still be there after the user close the panel and re-opened"
- );
+ // Wait a few seconds to make sure all the fields have been updated
+ await wait(1500);
+
+ const customMethod = document.querySelector("#http-custom-method-value");
+ const customUrl = document.querySelector(".http-custom-url-value");
+ const customQuery = document.querySelectorAll(
+ "#http-custom-query .tabpanel-summary-container.http-custom-input textarea"
+ );
+ const customHeaders = document.querySelectorAll(
+ "#http-custom-headers .tabpanel-summary-container.http-custom-input textarea"
+ );
+ const postDataValue = document.querySelector("#http-custom-postdata-value");
+
+ if (isPrivate) {
+ is(
+ customMethod.value,
+ "GET",
+ "The method should not be persisted after the user close the panel and re-opened in PBM"
+ );
+
+ is(
+ customUrl.value,
+ "",
+ "The url should not be there after the user close the panel and re-opened in PBM"
+ );
+
+ is(
+ customQuery.length,
+ 0,
+ "The Parameter should not be there after the user close the panel and re-opened in PBM"
+ );
+
+ is(
+ customHeaders.length,
+ 0,
+ "There should be no custom headers after the user close the panel and re-opened in PBM"
+ );
+
+ is(
+ postDataValue.value,
+ "",
+ "The post data should still be reset after the user close the panel and re-opened in PBM"
+ );
+ } else {
+ is(
+ customMethod.value,
+ "POST",
+ "The method should be persisted after the user close the panel and re-opened"
+ );
+
+ is(
+ customUrl.value,
+ "https://www.example.com?My-param=",
+ "The url should still be there after the user close the panel and re-opened"
+ );
+
+ const [nameParam] = Array.from(customQuery);
+ is(
+ nameParam.value,
+ "My-param",
+ "The Parameter name should still be there after the user close the panel and re-opened"
+ );
+
+ const [name, value] = Array.from(customHeaders);
+ is(
+ name.value,
+ "My-header",
+ "The header name should still be there after the user close the panel and re-opened"
+ );
+ is(
+ value.value,
+ "my-value",
+ "The header value should still be there after the user close the panel and re-opened"
+ );
+
+ is(
+ postDataValue.value,
+ "{'Name': 'Value'}",
+ "The content should still be there after the user close the panel and re-opened"
+ );
+ }
+}
+
+add_task(async function testRequestPanelPersistedContent() {
+ // Turn true the pref
+ await pushPref("devtools.netmonitor.features.newEditAndResend", true);
+ // Reset the storage for the persisted custom request
+ await asyncStorage.removeItem("devtools.netmonitor.customRequest");
- const [nameParam] = Array.from(
- document.querySelectorAll(
- "#http-custom-query .tabpanel-summary-container.http-custom-input textarea"
- )
- );
- is(
- nameParam.value,
- "My-param",
- "The Parameter name should still be there after the user close the panel and re-opened"
- );
+ const { tab, monitor } = await initNetMonitor(HTTPS_CUSTOM_GET_URL, {
+ requestCount: 1,
+ });
+ const { document, store, windowRequire } = monitor.panelWin;
+ const Actions = windowRequire("devtools/client/netmonitor/src/actions/index");
+ store.dispatch(Actions.batchEnable(false));
- const [name, value] = Array.from(
- document.querySelectorAll(
- "#http-custom-headers .tabpanel-summary-container.http-custom-input textarea"
- )
- );
- is(
- name.value,
- "My-header",
- "The header name should still be there after the user close the panel and re-opened"
- );
- is(
- value.value,
- "my-value",
- "The header value should still be there after the user close the panel and re-opened"
- );
+ info("Starting test... ");
+ info("Add initial custom request test content");
+ await addCustomRequestTestContent(tab, monitor, document);
+ await runTests(tab, monitor, document);
+ await teardown(monitor);
+});
- is(
- postValue.value,
- "{'Name': 'Value'}",
- "The content should still be there after the user close the panel and re-opened"
+add_task(async function testRequestPanelPersistedContentInPrivateWindow() {
+ await pushPref("devtools.netmonitor.features.newEditAndResend", true);
+ const { tab, monitor, privateWindow } = await initNetMonitor(
+ HTTPS_CUSTOM_GET_URL,
+ {
+ requestCount: 1,
+ openInPrivateWindow: true,
+ }
);
+ const { document, store, windowRequire } = monitor.panelWin;
+ const Actions = windowRequire("devtools/client/netmonitor/src/actions/index");
+ store.dispatch(Actions.batchEnable(false));
- await teardown(monitor);
+ info("Starting test in private window... ");
+ await runTests(tab, monitor, document, true);
+ await teardown(monitor, privateWindow);
});
=====================================
devtools/client/netmonitor/test/head.js
=====================================
@@ -321,6 +321,7 @@ function initNetMonitor(
expectedEventTimings,
waitForLoad = true,
enableCache = false,
+ openInPrivateWindow = false,
}
) {
info("Initializing a network monitor pane.");
@@ -341,7 +342,22 @@ function initNetMonitor(
],
});
- const tab = await addTab(url, { waitForLoad });
+ let tab = null;
+ let privateWindow = null;
+
+ if (openInPrivateWindow) {
+ privateWindow = await BrowserTestUtils.openNewBrowserWindow({
+ private: true,
+ });
+ ok(
+ PrivateBrowsingUtils.isContentWindowPrivate(privateWindow),
+ "window is private"
+ );
+ tab = BrowserTestUtils.addTab(privateWindow.gBrowser, url);
+ } else {
+ tab = await addTab(url, { waitForLoad });
+ }
+
info("Net tab added successfully: " + url);
const toolbox = await gDevTools.showToolboxForTab(tab, {
@@ -371,7 +387,7 @@ function initNetMonitor(
await clearNetworkEvents(monitor);
}
- return { tab, monitor, toolbox };
+ return { tab, monitor, toolbox, privateWindow };
})();
}
@@ -408,7 +424,7 @@ async function clearNetworkEvents(monitor) {
store.dispatch(Actions.clearRequests());
}
-function teardown(monitor) {
+function teardown(monitor, privateWindow) {
info("Destroying the specified network monitor.");
return (async function () {
@@ -419,6 +435,12 @@ function teardown(monitor) {
await monitor.toolbox.destroy();
await removeTab(tab);
+
+ if (privateWindow) {
+ const closed = BrowserTestUtils.windowClosed(privateWindow);
+ privateWindow.BrowserCommands.tryToCloseWindow();
+ await closed;
+ }
})();
}
View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/compare/3bed85e27d59187bdef1b4ca9ce9ec94187cbcb9...fa7dafa6f43fc7e9675a44c1b49b098a043babc3
--
This project does not include diff previews in email notifications.
View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/compare/3bed85e27d59187bdef1b4ca9ce9ec94187cbcb9...fa7dafa6f43fc7e9675a44c1b49b098a043babc3
You're receiving this email because of your account on gitlab.torproject.org.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.torproject.org/pipermail/tbb-commits/attachments/20240513/e2a31d65/attachment-0001.htm>
More information about the tbb-commits
mailing list