Use EvaluateWithContext from page.close

Now we can use the EvaluateWithContext from page.close when we call
evaluate to hide the page. We're going to rely on the default timeout
for now, and refine the timeout if we see this occurring too often --
if we see this occur too often this will affect the test result
depending on the default timeout (which is 30 seconds unless
overridden by the user).

Author: ankur22

common/page.go

@@ -427,15 +427,20 @@ func (p *Page) Close(opts goja.Value) error {
 
 	// forcing the pagehide event to trigger web vitals metrics.
 	v := p.vu.Runtime().ToValue(`() => window.dispatchEvent(new Event('pagehide'))`)
-	_ = p.Evaluate(v)
+	ctx, cancel := context.WithTimeout(p.ctx, p.defaultTimeout())
+	defer cancel()
+	_, err := p.MainFrame().EvaluateWithContext(ctx, v)
+	if err != nil {
+		p.logger.Warnf("Page:Close", "failed to hide page: %v", err)
+	}
 
 	add := runtime.RemoveBinding(webVitalBinding)
 	if err := add.Do(cdp.WithExecutor(p.ctx, p.session)); err != nil {
 		return fmt.Errorf("internal error while removing binding from page: %w", err)
 	}
 
 	action := target.CloseTarget(p.targetID)
-	err := action.Do(cdp.WithExecutor(p.ctx, p.session))
+	err = action.Do(cdp.WithExecutor(p.ctx, p.session))
 	if err != nil {
 		// When a close target command is sent to the browser via CDP,
 		// the browser will start to cleanup and the first thing it