Bug 1793995 - Part 3: Remove the unused includeGlobs and excludeGlobs getters, r=kmag

These getters are never called and will make making the core of MatchGlob
threadsafe more annoying.

Differential Revision: https://phabricator.services.mozilla.com/D158881

Author: mystor

dom/chrome-webidl/WebExtensionContentScript.webidl

@@ -74,21 +74,6 @@ interface MozDocumentMatcher {
   [Constant]
   readonly attribute MatchPatternSet? excludeMatches;
 
-  /**
-   * A set of glob matchers for URLs in which this script should run. If this
-   * list is present, the script will only run in URLs which match the
-   * `matches` pattern as well as one of these globs.
-   */
-  [Cached, Constant, Frozen]
-  readonly attribute sequence<MatchGlob>? includeGlobs;
-
-  /**
-   * A set of glob matchers for URLs in which this script should not run, even
-   * if they match other include patterns or globs.
-   */
-  [Cached, Constant, Frozen]
-  readonly attribute sequence<MatchGlob>? excludeGlobs;
-
   /**
    * The originAttributesPattern for which this script should be enabled for.
    */

toolkit/components/extensions/WebExtensionContentScript.h

@@ -137,13 +137,6 @@ class MozDocumentMatcher : public nsISupports, public nsWrapperCache {
   MatchPatternSet* GetExcludeMatches() { return mExcludeMatches; }
   const MatchPatternSet* GetExcludeMatches() const { return mExcludeMatches; }
 
-  void GetIncludeGlobs(Nullable<MatchGlobArray>& aGlobs) {
-    ToNullable(mExcludeGlobs, aGlobs);
-  }
-  void GetExcludeGlobs(Nullable<MatchGlobArray>& aGlobs) {
-    ToNullable(mExcludeGlobs, aGlobs);
-  }
-
   Nullable<uint64_t> GetFrameID() const { return mFrameID; }
 
   void GetOriginAttributesPatterns(JSContext* aCx,
@@ -179,25 +172,6 @@ class MozDocumentMatcher : public nsISupports, public nsWrapperCache {
   Nullable<uint64_t> mFrameID;
   bool mMatchAboutBlank;
   Nullable<dom::Sequence<OriginAttributesPattern>> mOriginAttributesPatterns;
-
- private:
-  template <typename T, typename U>
-  void ToNullable(const Nullable<T>& aInput, Nullable<U>& aOutput) {
-    if (aInput.IsNull()) {
-      aOutput.SetNull();
-    } else {
-      aOutput.SetValue(aInput.Value());
-    }
-  }
-
-  template <typename T, typename U>
-  void ToNullable(const Nullable<T>& aInput, Nullable<nsTArray<U>>& aOutput) {
-    if (aInput.IsNull()) {
-      aOutput.SetNull();
-    } else {
-      aOutput.SetValue(aInput.Value().Clone());
-    }
-  }
 };
 
 class WebExtensionContentScript final : public MozDocumentMatcher {