Skip to content

Commit

Permalink
Bug 1919071 - Remove the unused pdfjs.enableFloatingToolbar preferenc…
Browse files Browse the repository at this point in the history
…e. r=pdfjs-reviewers,calixte

This preference was removed upstream in mozilla/pdf.js#16605, which was uplifted in https://bugzilla.mozilla.org/show_bug.cgi?id=1840990, hence the override should also be removed now.

Differential Revision: https://phabricator.services.mozilla.com/D222306
  • Loading branch information
Snuffleupagus committed Sep 16, 2024
1 parent 13652d5 commit cfcef29
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 7 deletions.
3 changes: 0 additions & 3 deletions toolkit/components/pdfjs/PdfJsOverridePrefs.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,6 @@
// Editing PDFs is not supported on mobile
pref("pdfjs.annotationEditorMode", -1);

// Enable the floating PDF.js toolbar on GeckoView (bug 1829366)
pref("pdfjs.enableFloatingToolbar", true);

#else

#if defined(EARLY_BETA_OR_EARLIER)
Expand Down
5 changes: 1 addition & 4 deletions toolkit/components/pdfjs/test/test_pdf_file_in_iframe.html
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,7 @@

async function test() {
await SpecialPowers.pushPrefEnv({
"set": [
["pdfjs.disabled", false],
["pdfjs.enableFloatingToolbar", false],
],
"set": [["pdfjs.disabled", false]],
});

const iframe = document.createElement("iframe");
Expand Down

0 comments on commit cfcef29

Please sign in to comment.