927e50f5d4
At this point in time all browsers, and also Node.js, support standard `import`/`export` statements and we can now finally consider outputting modern JavaScript modules in the builds.[1] In order for this to work we can *only* use proper `import`/`export` statements throughout the main code-base, and (as expected) our Node.js support made this much more complicated since both the official builds and the GitHub Actions-based tests must keep working.[2] One remaining issue is that the `pdf.scripting.js` file cannot be built as a JavaScript module, since doing so breaks PDF scripting. Note that my initial goal was to try and split these changes into a couple of commits, however that unfortunately didn't really work since it turned out to be difficult for smaller patches to work correctly and pass (all) tests that way.[3] This is a classic case of every change requiring a couple of other changes, with each of those changes requiring further changes in turn and the size/scope quickly increasing as a result. One possible "issue" with these changes is that we'll now only output JavaScript modules in the builds, which could perhaps be a problem with older tools. However it unfortunately seems far too complicated/time-consuming for us to attempt to support both the old and modern module formats, hence the alternative would be to do "nothing" here and just keep our "old" builds.[4] --- [1] The final blocker was module support in workers in Firefox, which was implemented in Firefox 114; please see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/import#browser_compatibility [2] It's probably possible to further improve/simplify especially the Node.js-specific code, but it does appear to work as-is. [3] Having partially "broken" patches, that fail tests, as part of the commit history is *really not* a good idea in general. [4] Outputting JavaScript modules was first requested almost five years ago, see issue 10317, and nowadays there *should* be much better support for JavaScript modules in various tools.
47 lines
1.6 KiB
HTML
47 lines
1.6 KiB
HTML
<!DOCTYPE html>
|
|
<!--
|
|
Copyright 2015 Mozilla Foundation
|
|
|
|
Licensed under the Apache License, Version 2.0 (the "License");
|
|
you may not use this file except in compliance with the License.
|
|
You may obtain a copy of the License at
|
|
|
|
http://www.apache.org/licenses/LICENSE-2.0
|
|
|
|
Unless required by applicable law or agreed to in writing, software
|
|
distributed under the License is distributed on an "AS IS" BASIS,
|
|
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
See the License for the specific language governing permissions and
|
|
limitations under the License.
|
|
-->
|
|
<html>
|
|
<head>
|
|
<title>PDF.js test slave</title>
|
|
<meta charset="utf-8">
|
|
<script src="../build/generic/build/pdf.mjs" type="module"></script>
|
|
<script src="../build/components/pdf_viewer.mjs" type="module"></script>
|
|
|
|
<link rel="resource" type="application/l10n" href="../build/generic/web/locale/locale.properties">
|
|
</head>
|
|
<body>
|
|
<p>Inflight requests: <span id="inflight"></span></p>
|
|
<p>
|
|
<input type="checkbox" id="disableScrolling">
|
|
<label for="disableScrolling">Disable automatic scrolling</label>
|
|
</p>
|
|
<pre id="output" style="max-height: 800px; overflow-y: scroll;"></pre>
|
|
<div id="end"></div>
|
|
</body>
|
|
<script type="module">
|
|
import { Driver } from "./driver.js";
|
|
|
|
const driver = new Driver({
|
|
disableScrolling: document.getElementById('disableScrolling'),
|
|
inflight: document.getElementById('inflight'),
|
|
output: document.getElementById('output'),
|
|
end: document.getElementById('end')
|
|
});
|
|
driver.run();
|
|
</script>
|
|
</html>
|