Skip to content

Commit

Permalink
Merge pull request #19224 from Snuffleupagus/pr-18776-followup
Browse files Browse the repository at this point in the history
[api-minor] Remove deprecated `getDocument` options (PR 18776 follow-up)
  • Loading branch information
timvandermeij authored Dec 15, 2024
2 parents 3226169 + 490e740 commit ac81de6
Showing 1 changed file with 1 addition and 15 deletions.
16 changes: 1 addition & 15 deletions src/display/api.js
Original file line number Diff line number Diff line change
Expand Up @@ -43,15 +43,14 @@ import {
PrintAnnotationStorage,
SerializableEmpty,
} from "./annotation_storage.js";
import { FontFaceObject, FontLoader } from "./font_loader.js";
import {
deprecated,
isDataScheme,
isValidFetchUrl,
PageViewport,
RenderingCancelledException,
StatTimer,
} from "./display_utils.js";
import { FontFaceObject, FontLoader } from "./font_loader.js";
import {
NodeCanvasFactory,
NodeCMapReaderFactory,
Expand Down Expand Up @@ -339,19 +338,6 @@ function getDocument(src = {}) {
isValidFetchUrl(cMapUrl, document.baseURI) &&
isValidFetchUrl(standardFontDataUrl, document.baseURI));

if (typeof PDFJSDev === "undefined" || PDFJSDev.test("GENERIC")) {
if (src.canvasFactory) {
deprecated(
"`canvasFactory`-instance option, please use `CanvasFactory` instead."
);
}
if (src.filterFactory) {
deprecated(
"`filterFactory`-instance option, please use `FilterFactory` instead."
);
}
}

// Parameters only intended for development/testing purposes.
const styleElement =
typeof PDFJSDev === "undefined" || PDFJSDev.test("TESTING")
Expand Down

0 comments on commit ac81de6

Please sign in to comment.