Add page.reload early context cancel detection

inancgumus · inancgumus · commit f00ad8c596c5 · 2024-06-19T11:30:19.000+03:00
Sometimes the context gets canceled and leads to NPE.
diff --git a/common/page.go b/common/page.go
@@ -1085,6 +1085,7 @@ func (p *Page) Reload(opts sobek.Value) (*Response, error) { //nolint:funlen,cyc
 	)
 	select {
 	case <-p.ctx.Done():
+		err = fmt.Errorf("reloading page: %w", p.ctx.Err())
 	case <-timeoutCtx.Done():
 		err = wrapTimeoutError(timeoutCtx.Err())
 	case event := <-waitForFrameNavigation:

Original file line number	Diff line number	Diff line change
`@@ -1085,6 +1085,7 @@ func (p Page) Reload(opts sobek.Value) (Response, error) { //nolint:funlen,cyc`
`1085`	`1085`	`)`
`1086`	`1086`	`select {`
`1087`	`1087`	`case <-p.ctx.Done():`
	`1088`	`+ err = fmt.Errorf("reloading page: %w", p.ctx.Err())`
`1088`	`1089`	`case <-timeoutCtx.Done():`
`1089`	`1090`	`err = wrapTimeoutError(timeoutCtx.Err())`
`1090`	`1091`	`case event := <-waitForFrameNavigation:`