Warning: This build has drifted.
The coverage report for this pull request build may be inaccurate because its base commit is no longer the HEAD of its target branch.
This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

    • Learn more: For more information on this, see Tracking coverage changes for pull request builds.
    • Fix now: For a quick fix, rebase this PR at GitHub. Your next report should be accurate.
    • Prevent going forward: To avoid this issue with future PRs, see these Recommended CI Configurations.

fix: prevent panic in EvaluateHandle when returning an ElementHandle

EvaluateHandle, GetProperty and GetProperties on a JSHandle cast the
returned channel object to the concrete *jsHandleImpl. When the
evaluated expression returns a DOM element (e.g. element.parentElement),
the driver returns an ElementHandle, whose concrete type is
*elementHandleImpl, causing a panic:

  interface conversion: interface {} is *playwright.elementHandleImpl,
  not *playwright.jsHandleImpl

Cast to the JSHandle interface instead, which both *jsHandleImpl and
*elementHandleImpl satisfy. The same latent bug existed in
Worker.EvaluateHandle, Frame.WaitForFunction and ConsoleMessage.Args.

Fixes #332