Rename version.is_sandboxed() to is_flatpak()

(cherry picked from commit 5ce8a9c9c19e2aaec591b191d3c3efebd1957fa7)

For easier backporting of changes.

6 files changed, 12 insertions, 8 deletions

diff --git a/qutebrowser/misc/elf.py b/qutebrowser/misc/elf.py
index 6d35eecb4..18cb9634c 100644
--- a/qutebrowser/misc/elf.py
+++ b/qutebrowser/misc/elf.py
@@ -310,7 +310,7 @@ def _parse_from_file(f: IO[bytes]) -> Versions:
 
 def parse_webenginecore() -> Optional[Versions]:
     """Parse the QtWebEngineCore library file."""
-    if version.is_sandboxed():
+    if version.is_flatpak():
         # Flatpak has Qt in /usr/lib/x86_64-linux-gnu, but QtWebEngine in /app/lib.
         library_path = pathlib.Path("/app/lib")
     else:
diff --git a/qutebrowser/utils/standarddir.py b/qutebrowser/utils/standarddir.py
index 447515ecd..d62f0ffdb 100644
--- a/qutebrowser/utils/standarddir.py
+++ b/qutebrowser/utils/standarddir.py
@@ -233,7 +233,7 @@ def _init_runtime(args: Optional[argparse.Namespace]) -> None:
         # maximum length of 104 chars), so we don't add the username here...
 
         from qutebrowser.utils import version
-        if version.is_sandboxed():
+        if version.is_flatpak():
             *parts, app_name = os.path.split(path)
             assert app_name == APPNAME, app_name
             path = os.path.join(*parts, 'app', os.environ['FLATPAK_ID'])
diff --git a/qutebrowser/utils/utils.py b/qutebrowser/utils/utils.py
index 03a3c7842..780ac3184 100644
--- a/qutebrowser/utils/utils.py
+++ b/qutebrowser/utils/utils.py
@@ -607,7 +607,7 @@ def open_file(filename: str, cmdline: str = None) -> None:
     # if we want to use the default
     override = config.val.downloads.open_dispatcher
 
-    if version.is_sandboxed():
+    if version.is_flatpak():
         if cmdline:
             message.error("Cannot spawn download dispatcher from sandbox")
             return
diff --git a/qutebrowser/utils/version.py b/qutebrowser/utils/version.py
index ffb50e054..b9ca21f6d 100644
--- a/qutebrowser/utils/version.py
+++ b/qutebrowser/utils/version.py
@@ -189,8 +189,12 @@ def distribution() -> Optional[DistributionInfo]:
         parsed=parsed, version=dist_version, pretty=pretty, id=dist_id)
 
 
-def is_sandboxed() -> bool:
-    """Whether the environment has restricted access to the host system."""
+def is_flatpak() -> bool:
+    """Whether qutebrowser is running via Flatpak.
+
+    If packaged via Flatpak, the environment is has restricted access to the host
+    system.
+    """
     current_distro = distribution()
     if current_distro is None:
         return False
diff --git a/tests/unit/utils/test_standarddir.py b/tests/unit/utils/test_standarddir.py
index b663846e0..88abcd255 100644
--- a/tests/unit/utils/test_standarddir.py
+++ b/tests/unit/utils/test_standarddir.py
@@ -211,7 +211,7 @@ class TestStandardDir:
 
         app_id = 'org.qutebrowser.qutebrowser'
 
-        monkeypatch.setattr(version, 'is_sandboxed', lambda: True)
+        monkeypatch.setattr(version, 'is_flatpak', lambda: True)
         monkeypatch.setenv('XDG_RUNTIME_DIR', str(runtime_path))
         monkeypatch.setenv('FLATPAK_ID', app_id)
 
diff --git a/tests/unit/utils/test_version.py b/tests/unit/utils/test_version.py
index c91017e84..7ad223e4b 100644
--- a/tests/unit/utils/test_version.py
+++ b/tests/unit/utils/test_version.py
@@ -314,9 +314,9 @@ def test_distribution(tmpdir, monkeypatch, os_release, expected):
         id='arch', parsed=version.Distribution.arch, version=None,
         pretty='Arch Linux'), False)
 ])
-def test_is_sandboxed(monkeypatch, distribution, expected):
+def test_is_flatpak(monkeypatch, distribution, expected):
     monkeypatch.setattr(version, "distribution", lambda: distribution)
-    assert version.is_sandboxed() == expected
+    assert version.is_flatpak() == expected
 
 
 class GitStrSubprocessFake: