diff options
author | toofar <toofar@spalge.com> | 2023-11-27 08:20:28 +1300 |
---|---|---|
committer | toofar <toofar@spalge.com> | 2023-11-27 08:20:28 +1300 |
commit | c02a15b6c7525226a8cea274b0e111d4a6518d54 (patch) | |
tree | f58a09c436dfc3f8ea2db17c4add5d2ca9d024e4 | |
parent | a39dd64c5ebece50b0ae3574a5bb44885b99cefd (diff) | |
download | qutebrowser-c02a15b6c7525226a8cea274b0e111d4a6518d54.tar.gz qutebrowser-c02a15b6c7525226a8cea274b0e111d4a6518d54.zip |
Cleanup pdfjs_name
Now we aren't using it for the patching we don't need it in this function.
Move the `get_pdfjs_basename()` call into `is_available()` too since having to
call two methods to check everything is there sounds like a pain.
-rw-r--r-- | qutebrowser/browser/pdfjs.py | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/qutebrowser/browser/pdfjs.py b/qutebrowser/browser/pdfjs.py index 1a13f5bda..4c344d727 100644 --- a/qutebrowser/browser/pdfjs.py +++ b/qutebrowser/browser/pdfjs.py @@ -50,8 +50,7 @@ def generate_pdfjs_page(filename, url): filename: The filename of the PDF to open. url: The URL being opened. """ - pdfjs_name = get_pdfjs_basename() - if pdfjs_name is None or not is_available(): + if not is_available(): pdfjs_dir = os.path.join(standarddir.data(), 'pdfjs') return jinja.render('no_pdfjs.html', url=url.toDisplayString(), @@ -220,7 +219,7 @@ def get_pdfjs_basename(): def is_available(): """Return true if certain parts of a pdfjs installation are available.""" try: - # get_pdfjs_res('build/pdf.mjs') + get_pdfjs_basename() get_pdfjs_res('web/viewer.html') except PDFJSNotFound: return False |