Skip to content

Commit 97fd2da

Browse files
committed
Backed out 5 changesets (bug 1864896) for causing newtab failures related to bundles. CLOSED TREE
Backed out changeset 2cd1cc279f99 (bug 1864896) Backed out changeset e48d6928bdcb (bug 1864896) Backed out changeset 1abbcaf91693 (bug 1864896) Backed out changeset 06a05e5257d5 (bug 1864896) Backed out changeset b1955ae3e9e2 (bug 1864896)
1 parent 47a03bc commit 97fd2da

File tree

128 files changed

+339
-298
lines changed

Some content is hidden

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

128 files changed

+339
-298
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() {
46+
handleEvent(event) {
4747
// DOMDocElementCreated is only used to create the actor.
4848
}
4949

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

Lines changed: 13 additions & 10 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() {
261+
resetGroups(id, value) {
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() {
390+
onCopyTargetingParams(event) {
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={() => this.resetJSON(msg)}
510+
onClick={e => 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={() => this.modifyJson(msg)}
526+
onClick={e => 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={() => this.handleChange(msg.id)}
560+
onChange={e => 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={() => this.resetPBJSON(msg)}
650+
onClick={e => 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={() => this.toggleAllMessages(messagesToShow)}
701+
onClick={e => 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() {
1049+
setAttribution(e) {
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={() => this.toggleAllMessages(messagesToShow)}
1310+
onClick={e => this.toggleAllMessages(messagesToShow)}
13111311
>
13121312
Collapse/Expand All
13131313
</button>
@@ -1359,7 +1359,10 @@ export class ASRouterAdminInner extends React.PureComponent {
13591359
<tbody>
13601360
{this.state.groups &&
13611361
this.state.groups.map(
1362-
({ id, enabled, frequency, userPreferences = [] }) => (
1362+
(
1363+
{ id, enabled, frequency, userPreferences = [] },
1364+
index
1365+
) => (
13631366
<Row key={id}>
13641367
<td>
13651368
<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={() => onClick()} {...props}>
29+
<button className={className} onClick={e => 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() {
950+
async _onLocaleChanged(subject, topic, data) {
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(() => ({
1746+
return this.setState(({ groups }) => ({
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(() => resolve(false));
579+
.catch(e => 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) {
449+
observe(aSubject, aTopic, aData) {
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) {
565+
init(triggerHandler, params, patterns) {
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) {
613+
observe(aSubject, aTopic, aData) {
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) {
693+
observe(aSubject, aTopic, aData) {
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() {
795+
async _cfrUrlbarButtonClick(event) {
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) {
195+
handleFeatureCalloutCallback(win, event, data) {
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) => {
126+
let obs = (subject, topic, data) => {
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 }) {
84+
executeAction({ id, data, message_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 = () => "http://example.com";
186+
CFRPageActions._fetchLatestAddonVersion = x => "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(() => ({
447+
ASRouter.setState(state => ({
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() {
96+
function onError(...args) {
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-
() => {
189+
(win, calloutContainer) => {
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) {
19+
addIdleObserver(observer, time) {
2020
this._observers.add(observer);
2121
},
22-
removeIdleObserver(observer) {
22+
removeIdleObserver(observer, time) {
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() {
35+
static combineContexts(...args) {
3636
return sinon.stub();
3737
}
3838

39-
evalWithDefault() {
39+
evalWithDefault(expr) {
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-
([, , args]) =>
557+
([doc, el, 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-
([, , args]) => args && args.content && args.attributes
852+
([doc, el, 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-
([, , args]) => args && args.content && args.attributes
875+
([doc, el, 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: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -392,27 +392,28 @@ describe("ToolbarPanelHub", () => {
392392
for (let message of messages) {
393393
assert.ok(
394394
fakeRemoteL10n.createElement.args.find(
395-
([, , args]) => args && args.classList === "whatsNew-message-title"
395+
([doc, el, args]) =>
396+
args && args.classList === "whatsNew-message-title"
396397
)
397398
);
398399
if (message.content.layout === "tracking-protections") {
399400
assert.ok(
400401
fakeRemoteL10n.createElement.args.find(
401-
([, , args]) =>
402+
([doc, el, args]) =>
402403
args && args.classList === "whatsNew-message-subtitle"
403404
)
404405
);
405406
}
406407
if (message.id === "WHATS_NEW_FINGERPRINTER_COUNTER_72") {
407408
assert.ok(
408409
fakeRemoteL10n.createElement.args.find(
409-
([, el, args]) => el === "h2" && args.content === 3
410+
([doc, el, args]) => el === "h2" && args.content === 3
410411
)
411412
);
412413
}
413414
assert.ok(
414415
fakeRemoteL10n.createElement.args.find(
415-
([, , args]) =>
416+
([doc, el, args]) =>
416417
args && args.classList === "whatsNew-message-content"
417418
)
418419
);
@@ -455,9 +456,10 @@ describe("ToolbarPanelHub", () => {
455456
// value as the `order` field of the message
456457
const titleEls = fakeRemoteL10n.createElement.args
457458
.filter(
458-
([, , args]) => args && args.classList === "whatsNew-message-title"
459+
([doc, el, args]) =>
460+
args && args.classList === "whatsNew-message-title"
459461
)
460-
.map(([, , args]) => args.content);
462+
.map(([doc, el, args]) => args.content);
461463
assert.deepEqual(titleEls, [1, 2, 3]);
462464
});
463465
it("should accept string for image attributes", async () => {
@@ -486,7 +488,7 @@ describe("ToolbarPanelHub", () => {
486488
await instance.renderMessages(fakeWindow, fakeDocument, "container-id");
487489

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

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

0 commit comments

Comments
 (0)