Skip to content

Commit f4705b2

Browse files
committed
Bug 1864896: Autofix unused function arguments (browser/components/newtab). r=omc-reviewers,aminomancer,thecount
Differential Revision: https://phabricator.services.mozilla.com/D202950
1 parent 70fc8c9 commit f4705b2

File tree

52 files changed

+110
-122
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

52 files changed

+110
-122
lines changed

browser/components/asrouter/actors/ASRouterChild.sys.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ export class ASRouterChild extends JSWindowActorChild {
4343
});
4444
}
4545

46-
handleEvent(event) {
46+
handleEvent() {
4747
// DOMDocElementCreated is only used to create the actor.
4848
}
4949

browser/components/asrouter/content-src/components/ASRouterAdmin/ASRouterAdmin.jsx

Lines changed: 10 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -258,7 +258,7 @@ export class ASRouterAdminInner extends React.PureComponent {
258258
ASRouterUtils.sendMessage({ type: "RESET_PROVIDER_PREF" });
259259
}
260260

261-
resetGroups(id, value) {
261+
resetGroups() {
262262
ASRouterUtils.sendMessage({
263263
type: "RESET_GROUPS_STATE",
264264
}).then(this.setStateFromParent);
@@ -387,7 +387,7 @@ export class ASRouterAdminInner extends React.PureComponent {
387387
}
388388

389389
// Simulate a copy event that sets to clipboard all targeting paramters and values
390-
onCopyTargetingParams(event) {
390+
onCopyTargetingParams() {
391391
const stringTargetingParameters = {
392392
...this.state.stringTargetingParameters,
393393
};
@@ -507,7 +507,7 @@ export class ASRouterAdminInner extends React.PureComponent {
507507
isBlocked ? null : isModified ? (
508508
<button
509509
className="button restore"
510-
onClick={e => this.resetJSON(msg)}
510+
onClick={() => this.resetJSON(msg)}
511511
>
512512
Reset
513513
</button>
@@ -523,7 +523,7 @@ export class ASRouterAdminInner extends React.PureComponent {
523523
{isBlocked ? null : (
524524
<button
525525
className="button modify"
526-
onClick={e => this.modifyJson(msg)}
526+
onClick={() => this.modifyJson(msg)}
527527
>
528528
Modify
529529
</button>
@@ -557,7 +557,7 @@ export class ASRouterAdminInner extends React.PureComponent {
557557
name={msg.id}
558558
className="general-textarea"
559559
disabled={isBlocked}
560-
onChange={e => this.handleChange(msg.id)}
560+
onChange={() => this.handleChange(msg.id)}
561561
>
562562
{JSON.stringify(msg, null, 2)}
563563
</textarea>
@@ -647,7 +647,7 @@ export class ASRouterAdminInner extends React.PureComponent {
647647
</button>
648648
<button
649649
className="ASRouterButton slim button"
650-
onClick={e => this.resetPBJSON(msg)}
650+
onClick={() => this.resetPBJSON(msg)}
651651
>
652652
Reset JSON
653653
</button>
@@ -698,7 +698,7 @@ export class ASRouterAdminInner extends React.PureComponent {
698698
<div>
699699
<button
700700
className="ASRouterButton slim"
701-
onClick={e => this.toggleAllMessages(messagesToShow)}
701+
onClick={() => this.toggleAllMessages(messagesToShow)}
702702
>
703703
Collapse/Expand All
704704
</button>
@@ -1046,7 +1046,7 @@ export class ASRouterAdminInner extends React.PureComponent {
10461046
});
10471047
}
10481048

1049-
setAttribution(e) {
1049+
setAttribution() {
10501050
ASRouterUtils.sendMessage({
10511051
type: "FORCE_ATTRIBUTION",
10521052
data: this.state.attributionParameters,
@@ -1307,7 +1307,7 @@ export class ASRouterAdminInner extends React.PureComponent {
13071307
<h2>Messages</h2>
13081308
<button
13091309
className="ASRouterButton slim button"
1310-
onClick={e => this.toggleAllMessages(messagesToShow)}
1310+
onClick={() => this.toggleAllMessages(messagesToShow)}
13111311
>
13121312
Collapse/Expand All
13131313
</button>
@@ -1359,10 +1359,7 @@ export class ASRouterAdminInner extends React.PureComponent {
13591359
<tbody>
13601360
{this.state.groups &&
13611361
this.state.groups.map(
1362-
(
1363-
{ id, enabled, frequency, userPreferences = [] },
1364-
index
1365-
) => (
1362+
({ id, enabled, frequency, userPreferences = [] }) => (
13661363
<Row key={id}>
13671364
<td>
13681365
<TogglePrefCheckbox

browser/components/asrouter/content-src/components/ASRouterAdmin/CopyButton.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ export const CopyButton = ({
2626
timeout.current = setTimeout(() => setCopied(false), 1500);
2727
}, [inputSelector, transformer]);
2828
return (
29-
<button className={className} onClick={e => onClick()} {...props}>
29+
<button className={className} onClick={() => onClick()} {...props}>
3030
{(copied && copiedLabel) || label}
3131
</button>
3232
);

browser/components/asrouter/modules/ASRouter.sys.mjs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -947,7 +947,7 @@ export class _ASRouter {
947947
return this.state;
948948
}
949949

950-
async _onLocaleChanged(subject, topic, data) {
950+
async _onLocaleChanged() {
951951
await this._maybeUpdateL10nAttachment();
952952
}
953953

@@ -1743,7 +1743,7 @@ export class _ASRouter {
17431743
}
17441744
// Update storage
17451745
this._storage.set("groupImpressions", newGroupImpressions);
1746-
return this.setState(({ groups }) => ({
1746+
return this.setState(() => ({
17471747
groupImpressions: newGroupImpressions,
17481748
}));
17491749
}

browser/components/asrouter/modules/ASRouterTargeting.sys.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -576,7 +576,7 @@ const TargetingGetters = {
576576
lazy.fxAccounts
577577
.getSignedInUser()
578578
.then(data => resolve(!!data))
579-
.catch(e => resolve(false));
579+
.catch(() => resolve(false));
580580
});
581581
},
582582
get sync() {

browser/components/asrouter/modules/ASRouterTriggerListeners.sys.mjs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -446,7 +446,7 @@ export const ASRouterTriggerListeners = new Map([
446446
}
447447
},
448448

449-
observe(aSubject, aTopic, aData) {
449+
observe(aSubject, aTopic) {
450450
if (aSubject.currentURI.asciiHost === "accounts.firefox.com") {
451451
// Don't notify about saved logins on the FxA login origin since this
452452
// trigger is used to promote login Sync and getting a recommendation
@@ -562,7 +562,7 @@ export const ASRouterTriggerListeners = new Map([
562562
_sessionPageLoad: 0,
563563
onLocationChange: null,
564564

565-
init(triggerHandler, params, patterns) {
565+
init(triggerHandler, params) {
566566
params.forEach(p => this._events.push(p));
567567

568568
if (!this._initialized) {
@@ -610,7 +610,7 @@ export const ASRouterTriggerListeners = new Map([
610610
this._sessionPageLoad = 0;
611611
},
612612

613-
observe(aSubject, aTopic, aData) {
613+
observe(aSubject, aTopic) {
614614
switch (aTopic) {
615615
case "SiteProtection:ContentBlockingEvent":
616616
const { browser, host, event } = aSubject.wrappedJSObject;
@@ -690,7 +690,7 @@ export const ASRouterTriggerListeners = new Map([
690690
this._triggerHandler = triggerHandler;
691691
},
692692

693-
observe(aSubject, aTopic, aData) {
693+
observe(aSubject, aTopic) {
694694
switch (aTopic) {
695695
case "captive-portal-login-success":
696696
const browser = Services.wm.getMostRecentBrowserWindow();

browser/components/asrouter/modules/CFRPageActions.sys.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -792,7 +792,7 @@ export class PageAction {
792792
* Respond to a user click on the recommendation by showing a doorhanger/
793793
* popup notification or running the action defined in the message
794794
*/
795-
async _cfrUrlbarButtonClick(event) {
795+
async _cfrUrlbarButtonClick() {
796796
const browser = this.window.gBrowser.selectedBrowser;
797797
if (!RecommendationMap.has(browser)) {
798798
// There's no recommendation for this browser, so the user shouldn't have

browser/components/asrouter/modules/FeatureCalloutBroker.sys.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -192,7 +192,7 @@ export class _FeatureCalloutBroker {
192192
return item;
193193
}
194194

195-
handleFeatureCalloutCallback(win, event, data) {
195+
handleFeatureCalloutCallback(win, event) {
196196
switch (event) {
197197
case "end":
198198
const item = this.#calloutMap.get(win);

browser/components/asrouter/modules/ToastNotification.sys.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ export const ToastNotification = {
123123
}
124124

125125
let shownPromise = Promise.withResolvers();
126-
let obs = (subject, topic, data) => {
126+
let obs = (subject, topic) => {
127127
if (topic === "alertshow") {
128128
shownPromise.resolve();
129129
}

browser/components/asrouter/modules/ToolbarBadgeHub.sys.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ export class _ToolbarBadgeHub {
8181
win.MozXULElement.insertFTLIfNeeded("browser/newtab/asrouter.ftl");
8282
}
8383

84-
executeAction({ id, data, message_id }) {
84+
executeAction({ id }) {
8585
switch (id) {
8686
case "show-whatsnew-button":
8787
lazy.ToolbarPanelHub.enableToolbarButton();

browser/components/asrouter/tests/browser/browser_asrouter_cfr.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -183,7 +183,7 @@ add_setup(async function () {
183183
// Store it in order to restore to the original value
184184
const { _fetchLatestAddonVersion } = CFRPageActions;
185185
// Prevent fetching the real addon url and making a network request
186-
CFRPageActions._fetchLatestAddonVersion = x => "http://example.com";
186+
CFRPageActions._fetchLatestAddonVersion = () => "http://example.com";
187187

188188
registerCleanupFunction(() => {
189189
CFRPageActions._fetchLatestAddonVersion = _fetchLatestAddonVersion;

browser/components/asrouter/tests/browser/browser_asrouter_experimentsAPILoader.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -444,7 +444,7 @@ add_task(async function test_emptyMessage() {
444444
);
445445

446446
// Clear all messages
447-
ASRouter.setState(state => ({
447+
ASRouter.setState(() => ({
448448
messages: [],
449449
}));
450450

browser/components/asrouter/tests/browser/browser_asrouter_targeting.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ add_task(async function return_nothing_for_no_matching_message() {
9393

9494
add_task(async function check_other_error_handling() {
9595
let called = false;
96-
function onError(...args) {
96+
function onError() {
9797
called = true;
9898
}
9999

browser/components/asrouter/tests/browser/browser_feature_callout_panel.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,7 @@ add_task(async function panel_feature_callout_anchor_open_attr() {
186186

187187
win.document.querySelector(calloutDismissSelector).click();
188188
},
189-
(win, calloutContainer) => {
189+
() => {
190190
ok(
191191
!anchor.hasAttribute("open"),
192192
"Callout container's anchor node should not have its [open] attribute set"

browser/components/asrouter/tests/browser/browser_trigger_listeners.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,10 @@ const mockIdleService = {
1616
},
1717
QueryInterface: ChromeUtils.generateQI(["nsIUserIdleService"]),
1818
idleTime: 1200000,
19-
addIdleObserver(observer, time) {
19+
addIdleObserver(observer) {
2020
this._observers.add(observer);
2121
},
22-
removeIdleObserver(observer, time) {
22+
removeIdleObserver(observer) {
2323
this._observers.delete(observer);
2424
},
2525
};

browser/components/asrouter/tests/unit/ASRouterTargeting.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,11 +32,11 @@ describe("#CachedTargetingGetter", () => {
3232
globals.set(
3333
"TargetingContext",
3434
class {
35-
static combineContexts(...args) {
35+
static combineContexts() {
3636
return sinon.stub();
3737
}
3838

39-
evalWithDefault(expr) {
39+
evalWithDefault() {
4040
return sinon.stub();
4141
}
4242
}

browser/components/asrouter/tests/unit/CFRPageActions.test.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -554,7 +554,7 @@ describe("CFRPageActions", () => {
554554
);
555555
const htmlFooterEl = fakeRemoteL10n.createElement.args.find(
556556
/* eslint-disable-next-line max-nested-callbacks */
557-
([doc, el, args]) =>
557+
([, , args]) =>
558558
args && args.content === fakeRecommendation.content.text
559559
);
560560
assert.ok(htmlFooterEl);
@@ -849,7 +849,7 @@ describe("CFRPageActions", () => {
849849
);
850850
const [, , headerElementArgs] = fakeRemoteL10n.createElement.args.find(
851851
/* eslint-disable-next-line max-nested-callbacks */
852-
([doc, el, args]) => args && args.content && args.attributes
852+
([, , args]) => args && args.content && args.attributes
853853
);
854854
assert.equal(
855855
headerElementArgs.content.string_id,
@@ -872,7 +872,7 @@ describe("CFRPageActions", () => {
872872
);
873873
const [, , headerElementArgs] = fakeRemoteL10n.createElement.args.find(
874874
/* eslint-disable-next-line max-nested-callbacks */
875-
([doc, el, args]) => args && args.content && args.attributes
875+
([, , args]) => args && args.content && args.attributes
876876
);
877877
assert.equal(
878878
headerElementArgs.content.string_id,

browser/components/asrouter/tests/unit/ToolbarPanelHub.test.js

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -392,28 +392,27 @@ describe("ToolbarPanelHub", () => {
392392
for (let message of messages) {
393393
assert.ok(
394394
fakeRemoteL10n.createElement.args.find(
395-
([doc, el, args]) =>
396-
args && args.classList === "whatsNew-message-title"
395+
([, , args]) => args && args.classList === "whatsNew-message-title"
397396
)
398397
);
399398
if (message.content.layout === "tracking-protections") {
400399
assert.ok(
401400
fakeRemoteL10n.createElement.args.find(
402-
([doc, el, args]) =>
401+
([, , args]) =>
403402
args && args.classList === "whatsNew-message-subtitle"
404403
)
405404
);
406405
}
407406
if (message.id === "WHATS_NEW_FINGERPRINTER_COUNTER_72") {
408407
assert.ok(
409408
fakeRemoteL10n.createElement.args.find(
410-
([doc, el, args]) => el === "h2" && args.content === 3
409+
([, el, args]) => el === "h2" && args.content === 3
411410
)
412411
);
413412
}
414413
assert.ok(
415414
fakeRemoteL10n.createElement.args.find(
416-
([doc, el, args]) =>
415+
([, , args]) =>
417416
args && args.classList === "whatsNew-message-content"
418417
)
419418
);
@@ -456,10 +455,9 @@ describe("ToolbarPanelHub", () => {
456455
// value as the `order` field of the message
457456
const titleEls = fakeRemoteL10n.createElement.args
458457
.filter(
459-
([doc, el, args]) =>
460-
args && args.classList === "whatsNew-message-title"
458+
([, , args]) => args && args.classList === "whatsNew-message-title"
461459
)
462-
.map(([doc, el, args]) => args.content);
460+
.map(([, , args]) => args.content);
463461
assert.deepEqual(titleEls, [1, 2, 3]);
464462
});
465463
it("should accept string for image attributes", async () => {
@@ -488,7 +486,7 @@ describe("ToolbarPanelHub", () => {
488486
await instance.renderMessages(fakeWindow, fakeDocument, "container-id");
489487

490488
const [, , args] = fakeRemoteL10n.createElement.args.find(
491-
([doc, el, elArgs]) => elArgs && elArgs.attributes
489+
([, , elArgs]) => elArgs && elArgs.attributes
492490
);
493491
assert.ok(args);
494492
// Currently this.state.contentArguments has 8 different entries
@@ -510,7 +508,7 @@ describe("ToolbarPanelHub", () => {
510508
await instance.renderMessages(fakeWindow, fakeDocument, "container-id");
511509

512510
const dateElements = fakeRemoteL10n.createElement.args.filter(
513-
([doc, el, args]) =>
511+
([, el, args]) =>
514512
el === "p" && args.classList === "whatsNew-message-date"
515513
);
516514
assert.lengthOf(dateElements, uniqueDates.length);

0 commit comments

Comments
 (0)