fix(browser): Call original function on early return from patched history API (#15576)

msurdi-a8c · web-flow · commit 97155e966bea · 2025-03-05T09:47:00.000-05:00
In #14696 an early `return` was introduced that alters the default behavior of the underlying history API that is being patched. Instead of just returning, the original/underlying function should be called to keep the default behavior.
diff --git a/packages/browser-utils/src/instrument/history.ts b/packages/browser-utils/src/instrument/history.ts
@@ -51,7 +51,7 @@ function instrumentHistory(): void {
         lastHref = to;
 
         if (from === to) {
-          return;
+          return originalHistoryFunction.apply(this, args);
         }
 
         const handlerData = { from, to } satisfies HandlerDataHistory;

Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ function instrumentHistory(): void {`
`51`	`51`	`lastHref = to;`
`52`	`52`
`53`	`53`	`if (from === to) {`
`54`		`- return;`
	`54`	`+ return originalHistoryFunction.apply(this, args);`
`55`	`55`	`}`
`56`	`56`
`57`	`57`	`const handlerData = { from, to } satisfies HandlerDataHistory;`