Permalink
Browse files

Move Page.content() to FrameBase.content()

Both Page and Frame have this function which is essentially the same,
but obviously the frame only returns the content in the frame and not
the whole page, which I think should be pretty self-explanatory.
  • Loading branch information...
zploskey committed Jun 8, 2018
1 parent 3433ed8 commit 6f1b2188e3c6e08f82162994fefd0945b75ad31b
Showing with 2 additions and 2 deletions.
  1. +2 −0 src/FrameBase.re
  2. +0 −2 src/Page.re
@@ -396,6 +396,8 @@ external click :
(string, ~options: Click.clickOptions=?, unit) => Js.Promise.t(unit) =
"";

[@bs.send] external content : t => Js.Promise.t(string) = "";

/**
* Fetches the first element matching `selector`, scrolls it into view if not
* already visible, then hovers over the center of the element using
@@ -155,8 +155,6 @@ external makeCloseOptions : (~runBeforeUnload: bool=?, unit) => closeOptions =
[@bs.send.pipe: t]
external close : (~options: closeOptions=?) => Js.Promise.t(unit) = "";

[@bs.send] external content : t => Js.Promise.t(string) = "";

[@bs.get] external coverage : t => Coverage.t = "";

/* TODO: versions handling args */

0 comments on commit 6f1b218

Please sign in to comment.