summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFlorian Bruhin <git@the-compiler.org>2018-06-28 11:22:44 +0200
committerFlorian Bruhin <git@the-compiler.org>2018-06-28 11:22:44 +0200
commita7af5195d14a0604e6809f58e97b168b78493dda (patch)
treea4b2972f88cad3829fcb92b78be6e935608ed5aa
parente9c78b29edf312faeb9d82bd6c0bd576df632cd5 (diff)
downloadqutebrowser-a7af5195d14a0604e6809f58e97b168b78493dda.tar.gz
qutebrowser-a7af5195d14a0604e6809f58e97b168b78493dda.zip
Set title when showing PDF.js error page
Fixes #3894
-rw-r--r--doc/changelog.asciidoc2
-rw-r--r--qutebrowser/browser/webkit/webpage.py3
2 files changed, 4 insertions, 1 deletions
diff --git a/doc/changelog.asciidoc b/doc/changelog.asciidoc
index f4c62a713..64a051ad0 100644
--- a/doc/changelog.asciidoc
+++ b/doc/changelog.asciidoc
@@ -116,6 +116,8 @@ Fixed
- The security fix in v1.3.3 caused URLs with ampersands
(`www.example.com?one=1&two=2`) to send the wrong arguments when clicked on
the `qute://history` page.
+- Crash when opening a PDF page with PDF.js enabled (on QtWebKit), but no
+ PDF.js installed.
Removed
~~~~~~~
diff --git a/qutebrowser/browser/webkit/webpage.py b/qutebrowser/browser/webkit/webpage.py
index 853ff1b81..a6c26c8ee 100644
--- a/qutebrowser/browser/webkit/webpage.py
+++ b/qutebrowser/browser/webkit/webpage.py
@@ -212,7 +212,8 @@ class BrowserPage(QWebPage):
page = pdfjs.generate_pdfjs_page(reply.url())
except pdfjs.PDFJSNotFound:
page = jinja.render('no_pdfjs.html',
- url=reply.url().toDisplayString())
+ url=reply.url().toDisplayString(),
+ title="PDF.js not found")
self.mainFrame().setContent(page.encode('utf-8'), 'text/html',
reply.url())
reply.deleteLater()