Skip to content

Commit 014e11b

Browse files
authored
Move preloadedAudios/preloadedImages inside Browser. NFC (#22855)
1 parent 6be9648 commit 014e11b

File tree

3 files changed

+12
-13
lines changed

3 files changed

+12
-13
lines changed

site/source/docs/api_reference/preamble.js.rst

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -389,8 +389,6 @@ The :ref:`emscripten-memory-model` uses a typed array buffer (``ArrayBuffer``) t
389389
function SAFE_HEAP_LOAD(dest, bytes, isFloat, unsigned)
390390
function SAFE_FT_MASK(value, mask)
391391
function CHECK_OVERFLOW(value, bits, ignore, sig)
392-
Module["preloadedImages"]
393-
Module["preloadedAudios"]
394392
395393
396394
.. PRIVATE NOTES (not rendered) :

src/library_browser.js

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -26,15 +26,16 @@ var LibraryBrowser = {
2626
Module["setCanvasSize"] = Browser.setCanvasSize;
2727
Module["getUserMedia"] = Browser.getUserMedia;
2828
Module["createContext"] = Browser.createContext;
29-
var preloadedImages = {};
30-
var preloadedAudios = {};`,
29+
`,
3130

3231
$Browser: {
3332
useWebGL: false,
3433
isFullscreen: false,
3534
pointerLock: false,
3635
moduleContextCreatedCallbacks: [],
3736
workers: [],
37+
preloadedImages: {},
38+
preloadedAudios: {},
3839

3940
init() {
4041
if (Browser.initted) return;
@@ -73,7 +74,7 @@ var LibraryBrowser = {
7374
canvas.height = img.height;
7475
var ctx = canvas.getContext('2d');
7576
ctx.drawImage(img, 0, 0);
76-
preloadedImages[name] = canvas;
77+
Browser.preloadedImages[name] = canvas;
7778
URL.revokeObjectURL(url);
7879
onload?.(byteArray);
7980
};
@@ -94,13 +95,13 @@ var LibraryBrowser = {
9495
function finish(audio) {
9596
if (done) return;
9697
done = true;
97-
preloadedAudios[name] = audio;
98+
Browser.preloadedAudios[name] = audio;
9899
onload?.(byteArray);
99100
}
100101
function fail() {
101102
if (done) return;
102103
done = true;
103-
preloadedAudios[name] = new Audio(); // empty shim
104+
Browser.preloadedAudios[name] = new Audio(); // empty shim
104105
onerror?.();
105106
}
106107
var b = new Blob([byteArray], { type: Browser.getMimetype(name) });
@@ -881,7 +882,7 @@ var LibraryBrowser = {
881882
$getPreloadedImageData: (path, w, h) => {
882883
path = PATH_FS.resolve(path);
883884

884-
var canvas = /** @type {HTMLCanvasElement} */(preloadedImages[path]);
885+
var canvas = /** @type {HTMLCanvasElement} */(Browser.preloadedImages[path]);
885886
if (!canvas) return 0;
886887

887888
var ctx = canvas.getContext("2d");

src/library_sdl.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2156,7 +2156,7 @@ var LibrarySDL = {
21562156
// We support JPG, PNG, TIF because browsers do
21572157
IMG_Init: (flags) => flags,
21582158

2159-
IMG_Load_RW__deps: ['SDL_LockSurface', 'SDL_FreeRW', '$PATH_FS', '$stackSave', '$stackRestore', '$stringToUTF8OnStack', '$stackAlloc'],
2159+
IMG_Load_RW__deps: ['$Browser', 'SDL_LockSurface', 'SDL_FreeRW', '$PATH_FS', '$stackSave', '$stackRestore', '$stringToUTF8OnStack', '$stackAlloc'],
21602160
IMG_Load_RW__proxy: 'sync',
21612161
IMG_Load_RW: (rwopsID, freeSrc) => {
21622162
var sp = stackSave();
@@ -2209,7 +2209,7 @@ var LibrarySDL = {
22092209

22102210
if (!raw) {
22112211
filename = PATH_FS.resolve(filename);
2212-
raw = preloadedImages[filename];
2212+
raw = Browser.preloadedImages[filename];
22132213
if (!raw) {
22142214
if (raw === null) err('Trying to reuse preloaded image, but freePreloadedMediaOnUse is set!');
22152215
#if STB_IMAGE
@@ -2222,7 +2222,7 @@ var LibrarySDL = {
22222222
return 0;
22232223
#endif
22242224
} else if (Module['freePreloadedMediaOnUse']) {
2225-
preloadedImages[filename] = null;
2225+
Browser.preloadedImages[filename] = null;
22262226
}
22272227
}
22282228

@@ -2702,7 +2702,7 @@ var LibrarySDL = {
27022702

27032703
if (rwops.filename !== undefined) {
27042704
filename = PATH_FS.resolve(rwops.filename);
2705-
var raw = preloadedAudios[filename];
2705+
var raw = Browser.preloadedAudios[filename];
27062706
if (!raw) {
27072707
if (raw === null) err('Trying to reuse preloaded audio, but freePreloadedMediaOnUse is set!');
27082708
if (!Module['noAudioDecoding']) warnOnce('Cannot find preloaded audio ' + filename);
@@ -2716,7 +2716,7 @@ var LibrarySDL = {
27162716
}
27172717
}
27182718
if (Module['freePreloadedMediaOnUse']) {
2719-
preloadedAudios[filename] = null;
2719+
Browser.preloadedAudios[filename] = null;
27202720
}
27212721
audio = raw;
27222722
}

0 commit comments

Comments
 (0)