tor-browser

commit 6908c6dc6cf97adcce9a4f539065d0f100dc6225
parent fde651f861f2afc281f5f9c4a581e93c85e1eca0
Author: Greg Tatum <tatum.creative@gmail.com>
Date:   Mon,  8 Dec 2025 15:34:01 +0000

Bug 1998170 - Drive-by remove removeBoilerplate option r=ai-ondevice-reviewers,tarek

I'm not convinced that removing boilerplate is better than
viewport/readermode capabilities. I'm going to remove it from the code
for now pending Bug 1995620.

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

Diffstat:
M
toolkit/components/pageextractor/PageExtractor.d.ts
 | 
2
--
M
toolkit/components/pageextractor/PageExtractorChild.sys.mjs
 | 
4
----

2 files changed, 0 insertions(+), 6 deletions(-)
diff --git a/toolkit/components/pageextractor/PageExtractor.d.ts b/toolkit/components/pageextractor/PageExtractor.d.ts
@@ -7,8 +7,6 @@ export type GetTextOptions = Partial<{
   // When set, extraction will stop when the text meets or exceeds this length.
   // When unset, the lenghth of the extracted text is unbounded.
   sufficientLength: number;
-  // Remove menus and other boilerplate.
-  removeBoilerplate: boolean;
   // Just include the viewport content.
   justViewport: boolean;
 }>;
diff --git a/toolkit/components/pageextractor/PageExtractorChild.sys.mjs b/toolkit/components/pageextractor/PageExtractorChild.sys.mjs
@@ -136,10 +136,6 @@ export class PageExtractorChild extends JSWindowActorChild {
       return "";
     }
 
-    if (options.removeBoilerplate) {
-      throw new Error("Boilerplate removal is not supported yet.");
-    }
-
     const text = lazy.extractTextFromDOM(document, options);
 
     lazy.console.log("GetText", options);