Fix context managers behavior on exceptions.

qutebrowser/utils/debug.py

@@ -230,7 +230,9 @@ def log_time(logger, action='operation'):
         action: A description of what's being done.
     """
     started = datetime.datetime.now()
+    try:
         yield
+    finally:
         finished = datetime.datetime.now()
         delta = (finished - started).total_seconds()
         logger.debug("{} took {} seconds.".format(action.capitalize(), delta))

qutebrowser/utils/log.py

@@ -159,7 +159,9 @@ def init_log(args):
 def disable_qt_msghandler():
     """Contextmanager which temporarily disables the Qt message handler."""
     old_handler = QtCore.qInstallMessageHandler(None)
+    try:
         yield
+    finally:
         QtCore.qInstallMessageHandler(old_handler)
 
 
@@ -319,7 +321,9 @@ def hide_qt_warning(pattern, logger='qt'):
     log_filter = QtWarningFilter(pattern)
     logger_obj = logging.getLogger(logger)
     logger_obj.addFilter(log_filter)
+    try:
         yield
+    finally:
         logger_obj.removeFilter(log_filter)
 
 

qutebrowser/utils/utils.py

@@ -438,7 +438,9 @@ def disabled_excepthook():
     """Run code with the exception hook temporarily disabled."""
     old_excepthook = sys.excepthook
     sys.excepthook = sys.__excepthook__
+    try:
         yield
+    finally:
         # If the code we did run did change sys.excepthook, we leave it
         # unchanged. Otherwise, we reset it.
         if sys.excepthook is sys.__excepthook__: