Ema/perf followups (#386)

* Safeguard regexes against huge strings Signed-off-by: Emanuele Feliziani <[email protected]> * Add perf regression testing Signed-off-by: Emanuele Feliziani <[email protected]> * Bump up the timeout Signed-off-by: Emanuele Feliziani <[email protected]> * Bump timeouts again Signed-off-by: Emanuele Feliziani <[email protected]> --------- Signed-off-by: Emanuele Feliziani <[email protected]>
duckduckgo · Oct 2, 2023 · 84127e0 · 84127e0
1 parent 55466f1
commit 84127e0
Show file tree

Hide file tree

Showing 13 changed files with 4,537 additions and 131 deletions.
diff --git a/dist/autofill-debug.js b/dist/autofill-debug.js
diff --git a/dist/autofill.js b/dist/autofill.js
diff --git a/integration-test/helpers/harness.js b/integration-test/helpers/harness.js
@@ -215,7 +215,7 @@ export function forwardConsoleMessages (page, _opts = {}) {
  */
 export async function performanceEntries (page, measureName) {
     // don't measure until the entries exist
-    await page.waitForFunction((measureName) => window.performance.getEntriesByName(measureName).length > 0, `${measureName}:end`)
+    await page.waitForFunction((measureName) => window.performance.getEntriesByName(measureName).length > 0, `${measureName}:end`, { timeout: 5000 })
     const result = await page.evaluate((measureName) => {
         window.performance?.measure?.(measureName, `${measureName}:start`, `${measureName}:end`)
         const entries = window.performance?.getEntriesByName(measureName)