mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-11-24 21:31:04 +00:00
Bug 1478688: Remove test codes that does not use anymore. r=Honza
Depends on D83288 Differential Revision: https://phabricator.services.mozilla.com/D83289
This commit is contained in:
parent
f77008aa71
commit
6e9455cf79
@ -59,7 +59,6 @@ support-files =
|
||||
test_private.css
|
||||
test_private.html
|
||||
doc_empty.html
|
||||
doc_fetch_from_netmonitor.html
|
||||
doc_long_string.css
|
||||
doc_long.css
|
||||
doc_short_string.css
|
||||
|
@ -1,84 +0,0 @@
|
||||
/* Any copyright is dedicated to the Public Domain.
|
||||
http://creativecommons.org/publicdomain/zero/1.0/ */
|
||||
|
||||
"use strict";
|
||||
|
||||
// A test to ensure Style Editor only issues 1 request for each stylesheet (instead of 2)
|
||||
// by using the network monitor's request history (bug 1306892).
|
||||
|
||||
const EMPTY_TEST_URL = TEST_BASE_HTTP + "doc_empty.html";
|
||||
const TEST_URL = TEST_BASE_HTTP + "doc_fetch_from_netmonitor.html";
|
||||
|
||||
add_task(async function() {
|
||||
await pushPref("devtools.testing.netmonitor.want-all-requests", true);
|
||||
|
||||
info("Opening netmonitor");
|
||||
// Navigate first to an empty document in order to:
|
||||
// * avoid introducing a cross process navigation when calling navigateTo()
|
||||
// * properly wait for request updates when calling navigateTo, while showToolbox
|
||||
// won't necessarily wait for all pending requests. (If we were loading TEST_URL
|
||||
// in the tab, we might have pending updates in the netmonitor which won't be
|
||||
// awaited for by showToolbox)
|
||||
const tab = await addTab(EMPTY_TEST_URL);
|
||||
const target = await TargetFactory.forTab(tab);
|
||||
const toolbox = await gDevTools.showToolbox(target, "netmonitor");
|
||||
const monitor = toolbox.getPanel("netmonitor");
|
||||
const { store, windowRequire } = monitor.panelWin;
|
||||
const Actions = windowRequire("devtools/client/netmonitor/src/actions/index");
|
||||
const { getSortedRequests } = windowRequire(
|
||||
"devtools/client/netmonitor/src/selectors/index"
|
||||
);
|
||||
|
||||
store.dispatch(Actions.batchEnable(false));
|
||||
|
||||
info("Navigating to test page");
|
||||
await navigateTo(TEST_URL);
|
||||
|
||||
info("Opening Style Editor");
|
||||
const styleeditor = await toolbox.selectTool("styleeditor");
|
||||
const ui = styleeditor.UI;
|
||||
|
||||
info("Waiting for the sources to be loaded.");
|
||||
await ui.editors[0].getSourceEditor();
|
||||
await ui.selectStyleSheet(ui.editors[1].styleSheet);
|
||||
await ui.editors[1].getSourceEditor();
|
||||
|
||||
// Wait till there is 5 requests in Netmonitor store.
|
||||
// (i.e. the Styleeditor panel performed one request).
|
||||
await waitUntil(() => getSortedRequests(store.getState()).length == 5);
|
||||
|
||||
info("Checking Netmonitor contents.");
|
||||
const shortRequests = [];
|
||||
const longRequests = [];
|
||||
const hugeRequests = [];
|
||||
for (const item of getSortedRequests(store.getState())) {
|
||||
if (item.url.endsWith("doc_short_string.css")) {
|
||||
shortRequests.push(item);
|
||||
}
|
||||
if (item.url.endsWith("doc_long_string.css")) {
|
||||
longRequests.push(item);
|
||||
}
|
||||
if (item.url.endsWith("sjs_huge-css-server.sjs")) {
|
||||
hugeRequests.push(item);
|
||||
}
|
||||
}
|
||||
|
||||
is(
|
||||
shortRequests.length,
|
||||
1,
|
||||
"Got one request for doc_short_string.css after Style Editor was loaded."
|
||||
);
|
||||
is(
|
||||
longRequests.length,
|
||||
1,
|
||||
"Got one request for doc_long_string.css after Style Editor was loaded."
|
||||
);
|
||||
|
||||
// Requests with a response body size greater than 1MB cannot be fetched from the
|
||||
// netmonitor, the style editor should perform a separate request.
|
||||
is(
|
||||
hugeRequests.length,
|
||||
2,
|
||||
"Got two requests for sjs_huge-css-server.sjs after Style Editor was loaded."
|
||||
);
|
||||
});
|
@ -1,13 +0,0 @@
|
||||
<!doctype html>
|
||||
<html>
|
||||
<head>
|
||||
<title>Fetch from netmonitor testcase</title>
|
||||
<link rel="stylesheet" charset="UTF-8" type="text/css" media="screen" href="doc_short_string.css"/>
|
||||
<link rel="stylesheet" charset="UTF-8" type="text/css" media="screen" href="doc_long_string.css"/>
|
||||
<!-- This last CSS is generated by a SJS server to avoid adding a 300,000 lines stylesheet to the codebase. -->
|
||||
<link rel="stylesheet" charset="UTF-8" type="text/css" media="screen" href="sjs_huge-css-server.sjs"/>
|
||||
</head>
|
||||
<body>
|
||||
<div>Fetch from netmonitor</div>
|
||||
</body>
|
||||
</html>
|
@ -6,7 +6,6 @@
|
||||
|
||||
const { Cc, Ci, Cr, Cu } = require("chrome");
|
||||
const Services = require("Services");
|
||||
const flags = require("devtools/shared/flags");
|
||||
const {
|
||||
wildcardToRegExp,
|
||||
} = require("devtools/server/actors/network-monitor/utils/wildcard-to-regexp");
|
||||
@ -82,7 +81,6 @@ function matchRequest(channel, filters) {
|
||||
// Ignore requests from chrome or add-on code when we are monitoring
|
||||
// content.
|
||||
if (
|
||||
!flags.wantAllNetworkRequests &&
|
||||
channel.loadInfo &&
|
||||
channel.loadInfo.loadingDocument === null &&
|
||||
(channel.loadInfo.loadingPrincipal ===
|
||||
|
@ -50,16 +50,6 @@ function makePrefTrackedFlag(exports, name, pref) {
|
||||
});
|
||||
}
|
||||
|
||||
/**
|
||||
* Setting the "devtools.testing.netmonitor.want-all-requests" preference to true will
|
||||
* enable logging of all of network requests by the network monitor.
|
||||
*/
|
||||
makePrefTrackedFlag(
|
||||
exports,
|
||||
"wantAllNetworkRequests",
|
||||
"devtools.testing.netmonitor.want-all-requests"
|
||||
);
|
||||
|
||||
/**
|
||||
* Setting the "devtools.debugger.log" preference to true will enable logging of
|
||||
* the RDP calls to the devtools server.
|
||||
|
Loading…
Reference in New Issue
Block a user