Skip to content
This repository has been archived by the owner on Aug 27, 2019. It is now read-only.

Commit

Permalink
Only enable “Save Page As…” if we can load the content script.
Browse files Browse the repository at this point in the history
  This avoids the issue with `about:`, `chrome://` and `resource://`
URLs not being downloadable.
  • Loading branch information
ExE-Boss committed Aug 20, 2017
1 parent 440ca02 commit 28de787
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
1 change: 0 additions & 1 deletion src/background.js
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,6 @@ async function handlePopupMessage(message) {
],
enable: [
"new*",
"saveAs",
"emailLink",
"devGetTools",
"openAddons",
Expand Down
3 changes: 2 additions & 1 deletion src/content/content.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@ if (!amLoaded) {
let result = {
disable: [],
enable: [
"print"
"print",
"saveAs"
]
}
if (document.documentElement.requestFullScreen || document.documentElement.mozRequestFullScreen) {
Expand Down

0 comments on commit 28de787

Please sign in to comment.