Move webkit.http to webkit.httpheaders

flake8 got a new warning about a module name shadowing a builtin module: https://github.com/gforcada/flake8-builtins/pull/121

Probably would have been safe enough to ignore it. But I don't think
moving it is that hard anyway. Hopefully I didn't miss anything!

6 files changed, 22 insertions, 22 deletions

diff --git a/qutebrowser/browser/qtnetworkdownloads.py b/qutebrowser/browser/qtnetworkdownloads.py
index 9dd507ab5..0360eed66 100644
--- a/qutebrowser/browser/qtnetworkdownloads.py
+++ b/qutebrowser/browser/qtnetworkdownloads.py
@@ -19,7 +19,7 @@ from qutebrowser.config import config, websettings
 from qutebrowser.utils import message, usertypes, log, urlutils, utils, debug, objreg, qtlog
 from qutebrowser.misc import quitter
 from qutebrowser.browser import downloads
-from qutebrowser.browser.webkit import http
+from qutebrowser.browser.webkit import httpheaders
 from qutebrowser.browser.webkit.network import networkmanager
 
 
@@ -533,7 +533,7 @@ class DownloadManager(downloads.AbstractDownloadManager):
             try:
                 suggested_filename = target.suggested_filename()
             except downloads.NoFilenameError:
-                _, suggested_filename = http.parse_content_disposition(reply)
+                _, suggested_filename = httpheaders.parse_content_disposition(reply)
         log.downloads.debug("fetch: {} -> {}".format(reply.url(),
                                                      suggested_filename))
         download = DownloadItem(reply, manager=self)
diff --git a/qutebrowser/browser/webkit/http.py b/qutebrowser/browser/webkit/httpheaders.py
index 95b7b7104..95b7b7104 100644
--- a/qutebrowser/browser/webkit/http.py
+++ b/qutebrowser/browser/webkit/httpheaders.py
diff --git a/qutebrowser/browser/webkit/webpage.py b/qutebrowser/browser/webkit/webpage.py
index ea19174ec..595432dc9 100644
--- a/qutebrowser/browser/webkit/webpage.py
+++ b/qutebrowser/browser/webkit/webpage.py
@@ -18,7 +18,7 @@ from qutebrowser.qt.webkitwidgets import QWebPage, QWebFrame
 
 from qutebrowser.config import websettings, config
 from qutebrowser.browser import pdfjs, shared, downloads, greasemonkey
-from qutebrowser.browser.webkit import http
+from qutebrowser.browser.webkit import httpheaders
 from qutebrowser.browser.webkit.network import networkmanager
 from qutebrowser.utils import message, usertypes, log, jinja, objreg
 from qutebrowser.qt import sip
@@ -263,14 +263,14 @@ class BrowserPage(QWebPage):
         At some point we might want to implement the MIME Sniffing standard
         here: https://mimesniff.spec.whatwg.org/
         """
-        inline, suggested_filename = http.parse_content_disposition(reply)
+        inline, suggested_filename = httpheaders.parse_content_disposition(reply)
         download_manager = objreg.get('qtnetwork-download-manager')
         if not inline:
             # Content-Disposition: attachment -> force download
             download_manager.fetch(reply,
                                    suggested_filename=suggested_filename)
             return
-        mimetype, _rest = http.parse_content_type(reply)
+        mimetype, _rest = httpheaders.parse_content_type(reply)
         if mimetype == 'image/jpg':
             # Some servers (e.g. the LinkedIn CDN) send a non-standard
             # image/jpg (instead of image/jpeg, defined in RFC 1341 section
diff --git a/scripts/dev/check_coverage.py b/scripts/dev/check_coverage.py
index 38a8f6ca1..e1d0d8642 100644
--- a/scripts/dev/check_coverage.py
+++ b/scripts/dev/check_coverage.py
@@ -73,8 +73,8 @@ PERFECT_FILES = [
      'qutebrowser/browser/history.py'),
     ('tests/unit/browser/test_pdfjs.py',
      'qutebrowser/browser/pdfjs.py'),
-    ('tests/unit/browser/webkit/http/test_http.py',
-     'qutebrowser/browser/webkit/http.py'),
+    ('tests/unit/browser/webkit/http/test_httpheaders.py',
+     'qutebrowser/browser/webkit/httpheaders.py'),
     # ('tests/unit/browser/webkit/test_webkitelem.py',
     #  'qutebrowser/browser/webkit/webkitelem.py'),
     # ('tests/unit/browser/webkit/test_webkitelem.py',
diff --git a/tests/unit/browser/webkit/http/test_content_disposition.py b/tests/unit/browser/webkit/http/test_content_disposition.py
index 7cf80e3fd..4f3ef13c7 100644
--- a/tests/unit/browser/webkit/http/test_content_disposition.py
+++ b/tests/unit/browser/webkit/http/test_content_disposition.py
@@ -6,7 +6,7 @@ import logging
 
 import pytest
 
-from qutebrowser.browser.webkit import http
+from qutebrowser.browser.webkit import httpheaders
 
 
 DEFAULT_NAME = 'qutebrowser-download'
@@ -30,7 +30,7 @@ class HeaderChecker:
         """Check if the passed header has the given filename."""
         reply = self.stubs.FakeNetworkReply(
             headers={'Content-Disposition': header})
-        cd_inline, cd_filename = http.parse_content_disposition(reply)
+        cd_inline, cd_filename = httpheaders.parse_content_disposition(reply)
         assert cd_filename is not None
         assert cd_filename == filename
         assert cd_inline == expected_inline
@@ -40,7 +40,7 @@ class HeaderChecker:
         reply = self.stubs.FakeNetworkReply(
             headers={'Content-Disposition': header})
         with self.caplog.at_level(logging.ERROR, 'network'):
-            cd_inline, cd_filename = http.parse_content_disposition(reply)
+            cd_inline, cd_filename = httpheaders.parse_content_disposition(reply)
         assert cd_filename == DEFAULT_NAME
         assert cd_inline
 
@@ -48,7 +48,7 @@ class HeaderChecker:
         """Check if the passed header results in an unnamed attachment."""
         reply = self.stubs.FakeNetworkReply(
             headers={'Content-Disposition': header})
-        cd_inline, cd_filename = http.parse_content_disposition(reply)
+        cd_inline, cd_filename = httpheaders.parse_content_disposition(reply)
         assert cd_filename == DEFAULT_NAME
         assert not cd_inline
 
@@ -164,7 +164,7 @@ class TestAttachment:
         """
         reply = stubs.FakeNetworkReply(
             headers={'Content-Disposition': 'attachment'})
-        cd_inline, cd_filename = http.parse_content_disposition(reply)
+        cd_inline, cd_filename = httpheaders.parse_content_disposition(reply)
         assert not cd_inline
         assert cd_filename == DEFAULT_NAME
 
diff --git a/tests/unit/browser/webkit/http/test_http.py b/tests/unit/browser/webkit/http/test_httpheaders.py
index 210d79486..7368575e8 100644
--- a/tests/unit/browser/webkit/http/test_http.py
+++ b/tests/unit/browser/webkit/http/test_httpheaders.py
@@ -2,7 +2,7 @@
 #
 # SPDX-License-Identifier: GPL-3.0-or-later
 
-"""Tests for qutebrowser.browser.webkit.http."""
+"""Tests for qutebrowser.browser.webkit.httpheaders."""
 
 import logging
 
@@ -11,7 +11,7 @@ import hypothesis
 from hypothesis import strategies
 from qutebrowser.qt.core import QUrl
 
-from qutebrowser.browser.webkit import http
+from qutebrowser.browser.webkit import httpheaders
 
 
 @pytest.mark.parametrize('url, expected', [
@@ -24,7 +24,7 @@ from qutebrowser.browser.webkit import http
 ])
 def test_no_content_disposition(stubs, url, expected):
     reply = stubs.FakeNetworkReply(url=QUrl(url))
-    inline, filename = http.parse_content_disposition(reply)
+    inline, filename = httpheaders.parse_content_disposition(reply)
     assert inline
     assert filename == expected
 
@@ -40,8 +40,8 @@ def test_no_content_disposition(stubs, url, expected):
     # dropping QtWebKit.
 ])
 def test_parse_content_disposition_invalid(value):
-    with pytest.raises(http.ContentDispositionError):
-        http.ContentDisposition.parse(value)
+    with pytest.raises(httpheaders.ContentDispositionError):
+        httpheaders.ContentDisposition.parse(value)
 
 
 @pytest.mark.parametrize('template', [
@@ -58,16 +58,16 @@ def test_parse_content_disposition_hypothesis(caplog, template, stubs, s):
     header = template.format(s)
     reply = stubs.FakeNetworkReply(headers={'Content-Disposition': header})
     with caplog.at_level(logging.ERROR, 'network'):
-        http.parse_content_disposition(reply)
+        httpheaders.parse_content_disposition(reply)
 
 
 @hypothesis.given(strategies.binary())
 def test_content_disposition_directly_hypothesis(s):
     """Test rfc6266 parsing directly with binary data."""
     try:
-        cd = http.ContentDisposition.parse(s)
+        cd = httpheaders.ContentDisposition.parse(s)
         cd.filename()
-    except http.ContentDispositionError:
+    except httpheaders.ContentDispositionError:
         pass
 
 
@@ -83,7 +83,7 @@ def test_parse_content_type(stubs, content_type, expected_mimetype,
         reply = stubs.FakeNetworkReply()
     else:
         reply = stubs.FakeNetworkReply(headers={'Content-Type': content_type})
-    mimetype, rest = http.parse_content_type(reply)
+    mimetype, rest = httpheaders.parse_content_type(reply)
     assert mimetype == expected_mimetype
     assert rest == expected_rest
 
@@ -91,4 +91,4 @@ def test_parse_content_type(stubs, content_type, expected_mimetype,
 @hypothesis.given(strategies.text())
 def test_parse_content_type_hypothesis(stubs, s):
     reply = stubs.FakeNetworkReply(headers={'Content-Type': s})
-    http.parse_content_type(reply)
+    httpheaders.parse_content_type(reply)