Apply suggestions from code review

Co-authored-by: İnanç Gümüş <[email protected]>

common/browser_context.go

@@ -217,7 +217,7 @@ func (b *BrowserContext) ExposeFunction(name string, callback goja.Callable) {
 }
 
 // GrantPermissions enables the specified permissions, all others will be disabled.
-func (b *BrowserContext) GrantPermissions(permissions []string, parsedOpts *GrantPermissionsOptions) error {
+func (b *BrowserContext) GrantPermissions(permissions []string, opts *GrantPermissionsOptions) error {
 	b.logger.Debugf("BrowserContext:GrantPermissions", "bctxid:%v", b.id)
 
 	permsToProtocol := map[string]cdpbrowser.PermissionType{
@@ -247,9 +247,9 @@ func (b *BrowserContext) GrantPermissions(permissions []string, parsedOpts *Gran
 		perms = append(perms, proto)
 	}
 
-	action := cdpbrowser.GrantPermissions(perms).WithOrigin(parsedOpts.Origin).WithBrowserContextID(b.id)
+	action := cdpbrowser.GrantPermissions(perms).WithOrigin(opts.Origin).WithBrowserContextID(b.id)
 	if err := action.Do(cdp.WithExecutor(b.ctx, b.browser.conn)); err != nil {
-		return fmt.Errorf("error granting browser permissions: %w", err)
+		return fmt.Errorf("granting browser permissions: %w", err)
 	}
 
 	return nil

common/browser_context_options.go

@@ -239,12 +239,12 @@ type GrantPermissionsOptions struct {
 	Origin string
 }
 
-// NewGrantPermissionsOptions creates a new instance of GrantPermissionsOptions.
+// NewGrantPermissionsOptions returns a new GrantPermissionsOptions.
 func NewGrantPermissionsOptions() *GrantPermissionsOptions {
 	return &GrantPermissionsOptions{}
 }
 
-// Parse will parse the options. It can parse origin out of the supplied object.
+// Parse parses the options from opts if opts exists in the Goja runtime.
 func (g *GrantPermissionsOptions) Parse(ctx context.Context, opts goja.Value) {
 	rt := k6ext.Runtime(ctx)