From d431dd6965b54b223e14a294addf9467dd6fca78 Mon Sep 17 00:00:00 2001 From: jgraham Date: Thu, 12 Jan 2017 19:29:47 +0000 Subject: [PATCH] Ensure marionette screenshots cover the viewport. (#223) 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. --- wptrunner/executors/executormarionette.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/wptrunner/executors/executormarionette.py b/wptrunner/executors/executormarionette.py index 21606ddef2f87f..d04e9cf7c0acab 100644 --- a/wptrunner/executors/executormarionette.py +++ b/wptrunner/executors/executormarionette.py @@ -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]