Bug 1917468 - Remove the unused pdfjs.enableHighlightEditor preferenc…

…e. r=pdfjs-reviewers,robwu This preference was removed upstream in mozilla/pdf.js#18594, which was uplifted in https://bugzilla.mozilla.org/show_bug.cgi?id=1913880, hence the override should also be removed now. Differential Revision: https://phabricator.services.mozilla.com/D221411 UltraBlame original commit: c011b9b0def2c394bd90b1837935d502109db725
marco-c · Sep 16, 2024 · 6e54050 · 6e54050
1 parent e41d326
commit 6e54050
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 50 deletions.
diff --git a/toolkit/components/nimbus/FeatureManifest.yaml b/toolkit/components/nimbus/FeatureManifest.yaml
@@ -10082,38 +10082,6 @@ displayed
 .
 variables
 :
-addHighlight
-:
-description
-:
-Set
-to
-true
-to
-highlight
-some
-text
-or
-something
-else
-in
-an
-existing
-pdf
-.
-type
-:
-boolean
-setPref
-:
-branch
-:
-default
-pref
-:
-pdfjs
-.
-enableHighlightEditor
 enableAltText
 :
 description

diff --git a/toolkit/components/pdfjs/PdfJsOverridePrefs.js b/toolkit/components/pdfjs/PdfJsOverridePrefs.js
@@ -27,16 +27,6 @@ true
 ;
 #
 else
-pref
-(
-"
-pdfjs
-.
-enableHighlightEditor
-"
-true
-)
-;
 #
 if
 defined

diff --git a/toolkit/components/pdfjs/test/browser_pdfjs_editing_contextmenu.js b/toolkit/components/pdfjs/test/browser_pdfjs_editing_contextmenu.js
@@ -1753,14 +1753,6 @@ annotationEditorMode
 "
 0
 ]
-[
-"
-pdfjs
-.
-enableHighlightEditor
-"
-true
-]
 ]
 }
 )
-Original file line number
+Diff line change
@@ Expand Up / @@ -27,16 +27,6 @@ true @@
     ;
     #
     else
-    pref
-    (
-    "
-    pdfjs
-    .
-    enableHighlightEditor
-    "
-    true
-    )
-    ;
     #
     if
     defined
@@ Expand Down @@