From 1c9d0857cb296e2b00646f619e9d18c219a852d7 Mon Sep 17 00:00:00 2001 From: Florian Bruhin Date: Thu, 7 Jul 2016 16:08:59 +0200 Subject: [PATCH] Various cleanups --- qutebrowser/browser/tab.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/qutebrowser/browser/tab.py b/qutebrowser/browser/tab.py index d1d049d74..c279632a4 100644 --- a/qutebrowser/browser/tab.py +++ b/qutebrowser/browser/tab.py @@ -58,8 +58,8 @@ class WrapperLayout(QLayout): def sizeHint(self): return self._widget.sizeHint() - def itemAt(self, _index): - # FIXME why does this get called? + def itemAt(self, _index): # pragma: no cover + # For some reason this sometimes gets called by Qt. return None def takeAt(self, _index): @@ -113,7 +113,7 @@ class AbstractSearch(QObject): self.text = None self._flags = 0 - def search(self, text, *, ignore_case=False, wrap=False): + def search(self, text, *, ignore_case=False, wrap=False, reverse=False): """Find the given text on the page. Args: @@ -154,7 +154,7 @@ class AbstractZoom(QObject): self._init_neighborlist() objreg.get('config').changed.connect(self.on_config_changed) - # # FIXME is this needed? + # # FIXME:refactor is this needed? # # For some reason, this signal doesn't get disconnected automatically # # when the WebView is destroyed on older PyQt versions. # # See https://github.com/The-Compiler/qutebrowser/issues/390