Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add top level $eval(script=) helper method to Chromote Session #68

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

schloerke
Copy link
Collaborator

$eval() alters the default behavior to have

  • awaitPromise = TRUE: Wait for returned Promise object to resolve.
  • returnByValue = TRUE: Return serialized values, not the pointer to the execution.

Questions:

  • Should arguments be added? If so, this would require wrapping script in paste0("(function() { ", script, " }).apply(null, ", args_serialized, ");") which would have different scoping and return x; behavior.
  • Should code be wrapped in { to scope let/const variables?

@schloerke schloerke requested a review from wch January 10, 2022 16:13
* main:
  Update docs (#74)
  Use BootStrap responsive breakpoint of Large devices (#72)
* main:
  Take screenshots beyond viewport without resizing (#83)
  Quote browser path if not using Windows (#80)
  Do not `shQuote()` the browser path (#78)
  `$view()` should use local `Chrome` browser if available (#73)
  Add a reasonable `{chromote}` wall `timeout_` if Protocol `timeout` exists (#69)
  Remove `ChromoteSession$get_parent()` and lock `$protocol` (#76)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant