Skip to content

Commit

Permalink
Ensure marionette screenshots cover the viewport. (#223)
Browse files Browse the repository at this point in the history
It turns out the default marionette screenshot mode will attempt to screenshot
the html element, including cropping below the viewport size. This isn't expected
behaviour for reftests, so disable it.
  • Loading branch information
jgraham authored and gsnedders committed Jan 12, 2017
1 parent f150522 commit d431dd6
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion wptrunner/executors/executormarionette.py
Original file line number Diff line number Diff line change
Expand Up @@ -508,7 +508,7 @@ def _screenshot(self, marionette, url, timeout):

marionette.execute_async_script(self.wait_script)

screenshot = marionette.screenshot()
screenshot = marionette.screenshot(full=False)
# strip off the data:img/png, part of the url
if screenshot.startswith("data:image/png;base64,"):
screenshot = screenshot.split(",", 1)[1]
Expand Down

0 comments on commit d431dd6

Please sign in to comment.