From fbaf0bb26397aa498eb9156f06d5a6fe34dd7dd8 Mon Sep 17 00:00:00 2001 From: Daniel Baumann Date: Fri, 19 Apr 2024 03:14:29 +0200 Subject: Merging upstream version 125.0.1. Signed-off-by: Daniel Baumann --- remote/cdp/CDP.sys.mjs | 2 -- remote/cdp/CDPConnection.sys.mjs | 6 +++++ remote/cdp/Error.sys.mjs | 4 +-- remote/cdp/domains/Domain.sys.mjs | 4 +-- remote/cdp/domains/content/Runtime.sys.mjs | 2 +- remote/cdp/domains/parent/Network.sys.mjs | 4 +-- remote/cdp/domains/parent/Page.sys.mjs | 8 ++---- remote/cdp/observers/ContextObserver.sys.mjs | 2 +- remote/cdp/observers/NetworkObserver.sys.mjs | 4 +-- remote/cdp/targets/TabTarget.sys.mjs | 4 +-- remote/cdp/targets/Target.sys.mjs | 2 +- remote/cdp/test/browser/browser_httpd.js | 4 +-- remote/cdp/test/browser/input/doc_events.html | 3 +++ remote/cdp/test/browser/log/browser_entryAdded.js | 4 +-- .../browser/network/browser_responseReceived.js | 20 +++++++++------ .../browser/network/browser_setCacheDisabled.js | 9 ++++--- remote/cdp/test/browser/network/head.js | 10 +++++--- .../test/browser/page/browser_captureScreenshot.js | 10 +++++--- .../test/browser/page/browser_getLayoutMetrics.js | 30 +++++++++++++--------- .../test/browser/page/browser_lifecycleEvent.js | 7 ++--- .../browser/runtime/browser_exceptionThrown.js | 7 +---- .../test/browser/runtime/browser_getProperties.js | 6 ++++- .../security/browser_setIgnoreCertificateErrors.js | 2 +- .../browser/systemInfo/browser_getProcessInfo.js | 10 +++++--- .../test/browser/target/browser_activateTarget.js | 4 +-- .../test/browser/target/browser_attachToTarget.js | 4 +-- .../cdp/test/browser/target/browser_closeTarget.js | 6 ++--- .../test/browser/target/browser_targetDestroyed.js | 2 +- 28 files changed, 100 insertions(+), 80 deletions(-) (limited to 'remote/cdp') diff --git a/remote/cdp/CDP.sys.mjs b/remote/cdp/CDP.sys.mjs index 0307c71149..cd9aedfecc 100644 --- a/remote/cdp/CDP.sys.mjs +++ b/remote/cdp/CDP.sys.mjs @@ -26,8 +26,6 @@ const RECOMMENDED_PREFS = new Map([ "browser.contentblocking.features.standard", "-tp,tpPrivate,cookieBehavior0,-cm,-fp", ], - // Accept all cookies (see behavior definitions in nsICookieService.idl) - ["network.cookie.cookieBehavior", 0], ]); /** diff --git a/remote/cdp/CDPConnection.sys.mjs b/remote/cdp/CDPConnection.sys.mjs index 1d2eb3e77c..78638899a9 100644 --- a/remote/cdp/CDPConnection.sys.mjs +++ b/remote/cdp/CDPConnection.sys.mjs @@ -40,6 +40,12 @@ export class CDPConnection extends WebSocketConnection { * The session to register. */ registerSession(session) { + lazy.logger.warn( + `Support for the Chrome DevTools Protocol (CDP) in Firefox will be deprecated after Firefox 128 (ESR) ` + + `and will be removed in a later release. CDP users should consider migrating ` + + `to WebDriver BiDi. See https://bugzilla.mozilla.org/show_bug.cgi?id=1872254` + ); + // CDP is not compatible with Fission by default, check the appropriate // preferences are set to ensure compatibility. if ( diff --git a/remote/cdp/Error.sys.mjs b/remote/cdp/Error.sys.mjs index b047285649..37bd33e2e0 100644 --- a/remote/cdp/Error.sys.mjs +++ b/remote/cdp/Error.sys.mjs @@ -44,8 +44,8 @@ export class RemoteAgentError extends Error { * The error must be of this form: * * {"message": "TypeError: foo is not a function\n - * execute@chrome://remote/content/cdp/sessions/Session.jsm:73:39\n - * onMessage@chrome://remote/content/cdp/sessions/TabSession.jsm:65:20"} + * execute@chrome://remote/content/cdp/sessions/Session.sys.mjs:73:39\n + * onMessage@chrome://remote/content/cdp/sessions/TabSession.sys.mjs:65:20"} * * This approach has the notable deficiency that it cannot deal * with causes to errors because of the unstructured nature of CDP diff --git a/remote/cdp/domains/Domain.sys.mjs b/remote/cdp/domains/Domain.sys.mjs index d9b36c62b8..d753c35863 100644 --- a/remote/cdp/domains/Domain.sys.mjs +++ b/remote/cdp/domains/Domain.sys.mjs @@ -29,8 +29,8 @@ export class Domain { /** * Execute the provided method in the child domain that has the same domain - * name. eg. calling this.executeInChild from domains/parent/Input.jsm will - * attempt to execute the method in domains/content/Input.jsm. + * name. eg. calling this.executeInChild from domains/parent/Input.sys.mjs will + * attempt to execute the method in domains/content/Input.sys.mjs. * * This can only be called from parent domains managed by a TabSession. * diff --git a/remote/cdp/domains/content/Runtime.sys.mjs b/remote/cdp/domains/content/Runtime.sys.mjs index 35e0f16710..44c36914d4 100644 --- a/remote/cdp/domains/content/Runtime.sys.mjs +++ b/remote/cdp/domains/content/Runtime.sys.mjs @@ -613,7 +613,7 @@ export class Runtime extends ContentProcessDomain { * @param {nsIConsoleMessage} subject * Console message. */ - observe(subject, topic, data) { + observe(subject) { if (subject instanceof Ci.nsIScriptError && subject.hasException) { let entry = fromScriptError(subject); this._emitExceptionThrown(entry); diff --git a/remote/cdp/domains/parent/Network.sys.mjs b/remote/cdp/domains/parent/Network.sys.mjs index 4d36cf994e..18ce65e0c3 100644 --- a/remote/cdp/domains/parent/Network.sys.mjs +++ b/remote/cdp/domains/parent/Network.sys.mjs @@ -146,12 +146,10 @@ export class Network extends Domain { * * Depending on the backend support, will return detailed cookie information in the cookies field. * - * @param {object} options - * * @returns {Array} * Array of cookie objects. */ - async getAllCookies(options = {}) { + async getAllCookies() { const cookies = []; for (const cookie of Services.cookies.cookies) { cookies.push(_buildCookie(cookie)); diff --git a/remote/cdp/domains/parent/Page.sys.mjs b/remote/cdp/domains/parent/Page.sys.mjs index d050277aff..420f99af39 100644 --- a/remote/cdp/domains/parent/Page.sys.mjs +++ b/remote/cdp/domains/parent/Page.sys.mjs @@ -672,12 +672,8 @@ export class Page extends Domain { * When file chooser interception is enabled, * the native file chooser dialog is not shown. * Instead, a protocol event Page.fileChooserOpened is emitted. - * - * @param {object} options - * @param {boolean=} options.enabled - * Enabled state of file chooser interception. */ - setInterceptFileChooserDialog(options = {}) {} + setInterceptFileChooserDialog() {} _getCurrentHistoryIndex() { const { window } = this.session.target; @@ -719,7 +715,7 @@ export class Page extends Domain { */ _onDialogLoaded(e, data) { const { message, type } = data; - // XXX: We rely on the common-dialog-loaded event (see DialogHandler.jsm) + // XXX: We rely on the common-dialog-loaded event (see DialogHandler.sys.mjs) // which is inconsistent with the name "javascriptDialogOpening". // For correctness we should rely on an event fired _before_ the prompt is // visible, such as DOMWillOpenModalDialog. However the payload of this diff --git a/remote/cdp/observers/ContextObserver.sys.mjs b/remote/cdp/observers/ContextObserver.sys.mjs index d0d6fd1f93..14ad98108f 100644 --- a/remote/cdp/observers/ContextObserver.sys.mjs +++ b/remote/cdp/observers/ContextObserver.sys.mjs @@ -121,7 +121,7 @@ export class ContextObserver { } } - observe(subject, topic, data) { + observe(subject, topic) { switch (topic) { case "document-element-inserted": const window = subject.defaultView; diff --git a/remote/cdp/observers/NetworkObserver.sys.mjs b/remote/cdp/observers/NetworkObserver.sys.mjs index ffd8028ba7..10d5ba4588 100644 --- a/remote/cdp/observers/NetworkObserver.sys.mjs +++ b/remote/cdp/observers/NetworkObserver.sys.mjs @@ -186,7 +186,7 @@ export class NetworkObserver { this._redirectMap.set(newChannel, oldChannel); } - _onRequest(channel, topic) { + _onRequest(channel) { const httpChannel = channel.QueryInterface(Ci.nsIHttpChannel); const loadContext = getLoadContext(httpChannel); const browser = loadContext?.topFrameElement; @@ -242,7 +242,7 @@ export class NetworkObserver { }); } - _onResponse(fromCache, httpChannel, topic) { + _onResponse(fromCache, httpChannel) { const loadContext = getLoadContext(httpChannel); if ( !loadContext || diff --git a/remote/cdp/targets/TabTarget.sys.mjs b/remote/cdp/targets/TabTarget.sys.mjs index a35b0c3f07..016b934cb3 100644 --- a/remote/cdp/targets/TabTarget.sys.mjs +++ b/remote/cdp/targets/TabTarget.sys.mjs @@ -94,7 +94,7 @@ export class TabTarget extends Target { /** @returns {Promise} */ get faviconUrl() { - return new Promise((resolve, reject) => { + return new Promise(resolve => { lazy.Favicons.getFaviconURLForPage(this.browser.currentURI, url => { if (url) { resolve(url.spec); @@ -143,7 +143,7 @@ export class TabTarget extends Target { // nsIObserver - observe(subject, topic, data) { + observe(subject) { if (subject === this.mm && subject == "message-manager-disconnect") { // disconnect debugging target if is disconnected, // otherwise this is just a host process change diff --git a/remote/cdp/targets/Target.sys.mjs b/remote/cdp/targets/Target.sys.mjs index 9264110c37..ee6353983b 100644 --- a/remote/cdp/targets/Target.sys.mjs +++ b/remote/cdp/targets/Target.sys.mjs @@ -21,7 +21,7 @@ export class Target { */ constructor(targetList, sessionClass) { // Save a reference to TargetList instance in order to expose it to: - // domains/parent/Target.jsm + // domains/parent/Target.sys.mjs this.targetList = targetList; // When a new connection is made to this target, diff --git a/remote/cdp/test/browser/browser_httpd.js b/remote/cdp/test/browser/browser_httpd.js index fb74ae8b70..f3d7b6581f 100644 --- a/remote/cdp/test/browser/browser_httpd.js +++ b/remote/cdp/test/browser/browser_httpd.js @@ -188,7 +188,7 @@ add_task(async function json_activate_target({ client, tab }) { is(invalidResponse, "No such target id: does-not-exist"); }); -add_task(async function json_close_target({ CDP, client, tab }) { +add_task(async function json_close_target({ CDP, client }) { const { Target } = client; const { targetInfo, newTab } = await openTab(Target); @@ -224,7 +224,7 @@ add_task(async function json_close_target({ CDP, client, tab }) { target => target.id === targetInfo.targetId ); - ok(afterTarget == null, "New target is gone"); + Assert.equal(afterTarget, null, "New target is gone"); const invalidResponse = await requestJSON("/json/close/does-not-exist", { status: 404, diff --git a/remote/cdp/test/browser/input/doc_events.html b/remote/cdp/test/browser/input/doc_events.html index 74df931233..0cedc82e38 100644 --- a/remote/cdp/test/browser/input/doc_events.html +++ b/remote/cdp/test/browser/input/doc_events.html @@ -133,6 +133,9 @@

KeyReporter

+ +
diff --git a/remote/cdp/test/browser/log/browser_entryAdded.js b/remote/cdp/test/browser/log/browser_entryAdded.js index 1c6ba2e0a8..7eb85a4e49 100644 --- a/remote/cdp/test/browser/log/browser_entryAdded.js +++ b/remote/cdp/test/browser/log/browser_entryAdded.js @@ -100,7 +100,7 @@ add_task(async function eventsForScriptErrorContent({ client }) { is(events[0].lineNumber, 2, "Got expected line number"); }); -async function runEntryAddedTest(client, eventCount, callback, options = {}) { +async function runEntryAddedTest(client, eventCount, callback) { const { Log } = client; const EVENT_ENTRY_ADDED = "Log.entryAdded"; @@ -109,7 +109,7 @@ async function runEntryAddedTest(client, eventCount, callback, options = {}) { history.addRecorder({ event: Log.entryAdded, eventName: EVENT_ENTRY_ADDED, - messageFn: payload => `Received "${EVENT_ENTRY_ADDED}"`, + messageFn: () => `Received "${EVENT_ENTRY_ADDED}"`, }); const timeBefore = Date.now(); diff --git a/remote/cdp/test/browser/network/browser_responseReceived.js b/remote/cdp/test/browser/network/browser_responseReceived.js index 41e854de8a..5319f0eb65 100644 --- a/remote/cdp/test/browser/network/browser_responseReceived.js +++ b/remote/cdp/test/browser/network/browser_responseReceived.js @@ -74,8 +74,9 @@ add_task(async function documentNavigationWithResource({ client }) { "127.0.0.1", "Document response has the expected IP address" ); - ok( - typeof docResponse.response.remotePort == "number", + Assert.equal( + typeof docResponse.response.remotePort, + "number", "Document response has a remotePort" ); } @@ -112,8 +113,9 @@ add_task(async function documentNavigationWithResource({ client }) { docResponse.response.remoteIPAddress, "Script response has same IP address as document response" ); - ok( - typeof scriptResponse.response.remotePort == "number", + Assert.equal( + typeof scriptResponse.response.remotePort, + "number", "Script response has a remotePort" ); } @@ -166,8 +168,9 @@ add_task(async function documentNavigationWithResource({ client }) { "127.0.0.1", "Subdocument response has the expected IP address" ); - ok( - typeof frameDocResponse.response.remotePort == "number", + Assert.equal( + typeof frameDocResponse.response.remotePort, + "number", "Subdocument response has a remotePort" ); } @@ -207,8 +210,9 @@ add_task(async function documentNavigationWithResource({ client }) { docResponse.response.remoteIPAddress, "Script response has same IP address as document response" ); - ok( - typeof frameScriptResponse.response.remotePort == "number", + Assert.equal( + typeof frameScriptResponse.response.remotePort, + "number", "Script response has a remotePort" ); } diff --git a/remote/cdp/test/browser/network/browser_setCacheDisabled.js b/remote/cdp/test/browser/network/browser_setCacheDisabled.js index 4af9f0fc07..a7272bfc54 100644 --- a/remote/cdp/test/browser/network/browser_setCacheDisabled.js +++ b/remote/cdp/test/browser/network/browser_setCacheDisabled.js @@ -18,7 +18,7 @@ add_task(async function cacheEnabledAfterDisabled({ client }) { await waitForLoadFlags(); }); -add_task(async function cacheEnabledByDefault({ Network }) { +add_task(async function cacheEnabledByDefault() { await watchLoadFlags(LOAD_NORMAL, TEST_PAGE); await loadURL(TEST_PAGE); await waitForLoadFlags(); @@ -67,7 +67,7 @@ function watchLoadFlags(flags, url) { ); }, - onStateChange(webProgress, request, flags, status) { + onStateChange(webProgress, request, flags) { // We are checking requests - if there isn't one, ignore it. if (!request) { return; @@ -99,8 +99,9 @@ function watchLoadFlags(flags, url) { if (request.name == this.url && (flags & stopFlags) == stopFlags) { this.docShell.removeProgressListener(this); - ok( - this.requestCount > 1, + Assert.greater( + this.requestCount, + 1, this.url + " saw " + this.requestCount + " requests" ); this.callback(); diff --git a/remote/cdp/test/browser/network/head.js b/remote/cdp/test/browser/network/head.js index 3347e79e0f..2910f1c080 100644 --- a/remote/cdp/test/browser/network/head.js +++ b/remote/cdp/test/browser/network/head.js @@ -47,14 +47,16 @@ function assertEventOrder(first, second, options = {}) { const firstDescription = getDescriptionForEvent(first); const secondDescription = getDescriptionForEvent(second); - ok( - first.index < second.index, + Assert.less( + first.index, + second.index, `${firstDescription} received before ${secondDescription})` ); if (!ignoreTimestamps) { - ok( - first.payload.timestamp <= second.payload.timestamp, + Assert.lessOrEqual( + first.payload.timestamp, + second.payload.timestamp, `Timestamp of ${firstDescription}) is earlier than ${secondDescription})` ); } diff --git a/remote/cdp/test/browser/page/browser_captureScreenshot.js b/remote/cdp/test/browser/page/browser_captureScreenshot.js index ecd688fd14..1ccd54bc36 100644 --- a/remote/cdp/test/browser/page/browser_captureScreenshot.js +++ b/remote/cdp/test/browser/page/browser_captureScreenshot.js @@ -135,12 +135,14 @@ add_task(async function asJPEGFormatAndQuality({ client }) { is(info10.height, (viewport.height - viewport.y) * scale); // Images of different quality result in different content sizes - ok( - info100.length > infoDefault.length, + Assert.greater( + info100.length, + infoDefault.length, "Size of quality 100 is larger than default" ); - ok( - info10.length < infoDefault.length, + Assert.less( + info10.length, + infoDefault.length, "Size of quality 10 is smaller than default" ); }); diff --git a/remote/cdp/test/browser/page/browser_getLayoutMetrics.js b/remote/cdp/test/browser/page/browser_getLayoutMetrics.js index db8b3e8f3c..ad17e97aec 100644 --- a/remote/cdp/test/browser/page/browser_getLayoutMetrics.js +++ b/remote/cdp/test/browser/page/browser_getLayoutMetrics.js @@ -21,12 +21,14 @@ add_task(async function documentSmallerThanViewport({ client }) { layoutViewport.pageY, "Y position of content is equal to layout viewport" ); - ok( - contentSize.width <= layoutViewport.clientWidth, + Assert.lessOrEqual( + contentSize.width, + layoutViewport.clientWidth, "Width of content is smaller than the layout viewport" ); - ok( - contentSize.height <= layoutViewport.clientHeight, + Assert.lessOrEqual( + contentSize.height, + layoutViewport.clientHeight, "Height of content is smaller than the layout viewport" ); }); @@ -49,12 +51,14 @@ add_task(async function documentLargerThanViewport({ client }) { layoutViewport.pageY, "Y position of content is equal to layout viewport" ); - ok( - contentSize.width > layoutViewport.clientWidth, + Assert.greater( + contentSize.width, + layoutViewport.clientWidth, "Width of content is larger than the layout viewport" ); - ok( - contentSize.height > layoutViewport.clientHeight, + Assert.greater( + contentSize.height, + layoutViewport.clientHeight, "Height of content is larger than the layout viewport" ); }); @@ -81,12 +85,14 @@ add_task(async function documentLargerThanViewportScrolledXY({ client }) { contentSize.y + 100, "Y position of content is equal to layout viewport" ); - ok( - contentSize.width > layoutViewport.clientWidth, + Assert.greater( + contentSize.width, + layoutViewport.clientWidth, "Width of content is larger than the layout viewport" ); - ok( - contentSize.height > layoutViewport.clientHeight, + Assert.greater( + contentSize.height, + layoutViewport.clientHeight, "Height of content is larger than the layout viewport" ); }); diff --git a/remote/cdp/test/browser/page/browser_lifecycleEvent.js b/remote/cdp/test/browser/page/browser_lifecycleEvent.js index da5f5da9e8..eecf18f310 100644 --- a/remote/cdp/test/browser/page/browser_lifecycleEvent.js +++ b/remote/cdp/test/browser/page/browser_lifecycleEvent.js @@ -178,9 +178,10 @@ async function runPageLifecycleTest(client, expectedEventSets, callback) { // Check data as exposed by each of these events let lastTimestamp = frameEvents[0].payload.timestamp; - frameEvents.forEach(({ payload }, index) => { - ok( - payload.timestamp >= lastTimestamp, + frameEvents.forEach(({ payload }) => { + Assert.greaterOrEqual( + payload.timestamp, + lastTimestamp, "timestamp succeeds the one from the former event" ); lastTimestamp = payload.timestamp; diff --git a/remote/cdp/test/browser/runtime/browser_exceptionThrown.js b/remote/cdp/test/browser/runtime/browser_exceptionThrown.js index 23e6dc2de9..ad46ae25a9 100644 --- a/remote/cdp/test/browser/runtime/browser_exceptionThrown.js +++ b/remote/cdp/test/browser/runtime/browser_exceptionThrown.js @@ -76,12 +76,7 @@ add_task(async function eventsForScriptErrorWithException({ client }) { is(callFrames[1].url, PAGE_CONSOLE_EVENTS, "Got expected url"); }); -async function runExceptionThrownTest( - client, - eventCount, - callback, - options = {} -) { +async function runExceptionThrownTest(client, eventCount, callback) { const { Runtime } = client; const EVENT_EXCEPTION_THROWN = "Runtime.exceptionThrown"; diff --git a/remote/cdp/test/browser/runtime/browser_getProperties.js b/remote/cdp/test/browser/runtime/browser_getProperties.js index f897c3dfcd..f004480a22 100644 --- a/remote/cdp/test/browser/runtime/browser_getProperties.js +++ b/remote/cdp/test/browser/runtime/browser_getProperties.js @@ -102,7 +102,11 @@ async function testGetPrototypeProperties({ Runtime }, contextId) { objectId: result.objectId, ownProperties: false, }); - ok(result2.length > 1, "We have more properties than just the object one"); + Assert.greater( + result2.length, + 1, + "We have more properties than just the object one" + ); const foo = result2.find(p => p.name == "foo"); ok(foo, "The object property is described"); ok(foo.isOwn, "and is reported as 'own' property"); diff --git a/remote/cdp/test/browser/security/browser_setIgnoreCertificateErrors.js b/remote/cdp/test/browser/security/browser_setIgnoreCertificateErrors.js index d21e737ebe..969541f842 100644 --- a/remote/cdp/test/browser/security/browser_setIgnoreCertificateErrors.js +++ b/remote/cdp/test/browser/security/browser_setIgnoreCertificateErrors.js @@ -68,7 +68,7 @@ function isSecurityState(browser, expectedState) { ); } -add_task(async function testDefault({ Security }) { +add_task(async function testDefault() { for (const url of BAD_CERTS) { info(`Navigating to ${url}`); const loaded = BrowserTestUtils.waitForErrorPage(gBrowser.selectedBrowser); diff --git a/remote/cdp/test/browser/systemInfo/browser_getProcessInfo.js b/remote/cdp/test/browser/systemInfo/browser_getProcessInfo.js index fb491e248f..d2ee0096e9 100644 --- a/remote/cdp/test/browser/systemInfo/browser_getProcessInfo.js +++ b/remote/cdp/test/browser/systemInfo/browser_getProcessInfo.js @@ -51,9 +51,13 @@ function assertProcesses(processInfo, tabs) { ok(Array.isArray(processInfo), "Process info is an array"); for (const info of processInfo) { - ok(typeof info.id === "number", "Info has a numeric id"); - ok(typeof info.type === "string", "Info has a string type"); - ok(typeof info.cpuTime === "number", "Info has a numeric cpuTime"); + Assert.strictEqual(typeof info.id, "number", "Info has a numeric id"); + Assert.strictEqual(typeof info.type, "string", "Info has a string type"); + Assert.strictEqual( + typeof info.cpuTime, + "number", + "Info has a numeric cpuTime" + ); } const getByType = type => processInfo.filter(info => info.type === type); diff --git a/remote/cdp/test/browser/target/browser_activateTarget.js b/remote/cdp/test/browser/target/browser_activateTarget.js index 03c5a96e07..9f4762dfae 100644 --- a/remote/cdp/test/browser/target/browser_activateTarget.js +++ b/remote/cdp/test/browser/target/browser_activateTarget.js @@ -3,7 +3,7 @@ "use strict"; -add_task(async function raisesWithoutArguments({ client, tab }) { +add_task(async function raisesWithoutArguments({ client }) { const { Target } = client; await Assert.rejects( @@ -13,7 +13,7 @@ add_task(async function raisesWithoutArguments({ client, tab }) { ); }); -add_task(async function raisesWithUnknownTargetId({ client, tab }) { +add_task(async function raisesWithUnknownTargetId({ client }) { const { Target } = client; await Assert.rejects( diff --git a/remote/cdp/test/browser/target/browser_attachToTarget.js b/remote/cdp/test/browser/target/browser_attachToTarget.js index 944d7a9984..2ffebc15a9 100644 --- a/remote/cdp/test/browser/target/browser_attachToTarget.js +++ b/remote/cdp/test/browser/target/browser_attachToTarget.js @@ -3,7 +3,7 @@ "use strict"; -add_task(async function raisesWithoutArguments({ client, tab }) { +add_task(async function raisesWithoutArguments({ client }) { const { Target } = client; await Assert.rejects( @@ -13,7 +13,7 @@ add_task(async function raisesWithoutArguments({ client, tab }) { ); }); -add_task(async function raisesWithUnknownTargetId({ client, tab }) { +add_task(async function raisesWithUnknownTargetId({ client }) { const { Target } = client; await Assert.rejects( diff --git a/remote/cdp/test/browser/target/browser_closeTarget.js b/remote/cdp/test/browser/target/browser_closeTarget.js index 694994148b..b1a980b8e0 100644 --- a/remote/cdp/test/browser/target/browser_closeTarget.js +++ b/remote/cdp/test/browser/target/browser_closeTarget.js @@ -3,7 +3,7 @@ "use strict"; -add_task(async function raisesWithoutArguments({ client, tab }) { +add_task(async function raisesWithoutArguments({ client }) { const { Target } = client; await Assert.rejects( @@ -13,7 +13,7 @@ add_task(async function raisesWithoutArguments({ client, tab }) { ); }); -add_task(async function raisesWithUnknownTargetId({ client, tab }) { +add_task(async function raisesWithUnknownTargetId({ client }) { const { Target } = client; await Assert.rejects( @@ -23,7 +23,7 @@ add_task(async function raisesWithUnknownTargetId({ client, tab }) { ); }); -add_task(async function triggersTargetDestroyed({ client, tab }) { +add_task(async function triggersTargetDestroyed({ client }) { const { Target } = client; const { targetInfo, newTab } = await openTab(Target); diff --git a/remote/cdp/test/browser/target/browser_targetDestroyed.js b/remote/cdp/test/browser/target/browser_targetDestroyed.js index 2ad657b135..6668787ddc 100644 --- a/remote/cdp/test/browser/target/browser_targetDestroyed.js +++ b/remote/cdp/test/browser/target/browser_targetDestroyed.js @@ -3,7 +3,7 @@ "use strict"; -add_task(async function eventFiredWhenTabIsClosed({ client, tab }) { +add_task(async function eventFiredWhenTabIsClosed({ client }) { const { Target } = client; const { newTab } = await openTab(Target); -- cgit v1.2.3