Handle errors while getting screenshot

1 files changed, 9 insertions, 1 deletions

diff --git a/qutebrowser/components/misccommands.py b/qutebrowser/components/misccommands.py
index 0ead38799..7d65a63a4 100644
--- a/qutebrowser/components/misccommands.py
+++ b/qutebrowser/components/misccommands.py
@@ -41,6 +41,9 @@ from qutebrowser.completion.models import miscmodels
 from qutebrowser.utils import utils
 
 
+_LOGGER = logging.getLogger('misc')
+
+
 @cmdutils.register(name='reload')
 @cmdutils.argument('tab', value=cmdutils.Value.count_tab)
 def reloadpage(tab: Optional[apitypes.Tab],
@@ -93,7 +96,7 @@ def _print_pdf(tab: apitypes.Tab, filename: str) -> None:
     if directory and not os.path.exists(directory):
         os.mkdir(directory)
     tab.printing.to_pdf(filename)
-    logging.getLogger('misc').debug("Print to file: {}".format(filename))
+    _LOGGER.debug("Print to file: {}".format(filename))
 
 
 @cmdutils.register(name='print')
@@ -183,10 +186,15 @@ def screenshot(
     qrect = QRect() if rect is None else utils.parse_rect(rect)
 
     pic = tab.grab_pixmap(qrect)
+    if pic is None:
+        raise cmdutils.CommandError("Getting screenshot failed")
+
     ok = pic.save(str(expanded))
     if not ok:
         raise cmdutils.CommandError(f"Saving to {filename} failed")
 
+    _LOGGER.debug(f"Screenshot saved to {filename}")
+
 
 @cmdutils.register(maxsplit=0)
 @cmdutils.argument('tab', value=cmdutils.Value.cur_tab)