From e18f3cf8c98234ad24ad5260c796e416d9c191d6 Mon Sep 17 00:00:00 2001 From: Jeffrey Holland <124393578+jholland-livefront@users.noreply.github.com> Date: Tue, 23 Sep 2025 15:38:53 +0200 Subject: [PATCH] [PM-22417] Enable autofill overlay for siteone.com (#16457) * [PM-22417] Enable autofill overlay for siteone.com * Fix broken test * Remove the magic number --- .../services/collect-autofill-content.service.spec.ts | 2 +- .../autofill/services/collect-autofill-content.service.ts | 7 ++++++- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/apps/browser/src/autofill/services/collect-autofill-content.service.spec.ts b/apps/browser/src/autofill/services/collect-autofill-content.service.spec.ts index 1e6c38bdce..9ee329fa15 100644 --- a/apps/browser/src/autofill/services/collect-autofill-content.service.spec.ts +++ b/apps/browser/src/autofill/services/collect-autofill-content.service.spec.ts @@ -578,7 +578,7 @@ describe("CollectAutofillContentService", () => { const autofillFieldsData = await Promise.resolve(autofillFieldsPromise); expect(collectAutofillContentService["getAutofillFieldElements"]).toHaveBeenCalledWith( - 100, + 200, formFieldElements, ); expect(collectAutofillContentService["buildAutofillFieldItem"]).toHaveBeenCalledTimes(2); diff --git a/apps/browser/src/autofill/services/collect-autofill-content.service.ts b/apps/browser/src/autofill/services/collect-autofill-content.service.ts index 0629621fad..2ddee28904 100644 --- a/apps/browser/src/autofill/services/collect-autofill-content.service.ts +++ b/apps/browser/src/autofill/services/collect-autofill-content.service.ts @@ -278,7 +278,12 @@ export class CollectAutofillContentService implements CollectAutofillContentServ private async buildAutofillFieldsData( formFieldElements: FormFieldElement[], ): Promise { - const autofillFieldElements = this.getAutofillFieldElements(100, formFieldElements); + // Maximum number of form fields to process for autofill to prevent performance issues on pages with excessive fields + const autofillFieldsLimit = 200; + const autofillFieldElements = this.getAutofillFieldElements( + autofillFieldsLimit, + formFieldElements, + ); const autofillFieldDataPromises = autofillFieldElements.map(this.buildAutofillFieldItem); return Promise.all(autofillFieldDataPromises);