Skip to content
This repository was archived by the owner on Jan 11, 2023. It is now read-only.

Update source-map worker to use source-map 0.7.x - #7071

Merged
jasonLaster merged 5 commits into
firefox-devtools:masterfrom
loganfsmyth:source-map-worker-assets
Oct 3, 2018
Merged

jasonLaster merged 5 commits into
firefox-devtools:masterfrom
loganfsmyth:source-map-worker-assets

Conversation

@loganfsmyth

Copy link
Copy Markdown
Contributor

Fixes #5598

Summary of Changes

  • Moves the dwarf_to_json.wasm into devtools-source-map since that is where it is used.
  • Adds a general asset location for the source-map worker
  • Updates the source-map module to use the new wasm version.

The tests on this will fail until https://bugzilla.mozilla.org/show_bug.cgi?id=1495029 lands in MC, but I'm posting it now since it's ready otherwise.

@loganfsmyth
loganfsmyth force-pushed the source-map-worker-assets branch from d836274 to fe6ee94 Compare October 3, 2018 19:32
@jasonLaster
jasonLaster merged commit 0d0dbd3 into firefox-devtools:master Oct 3, 2018
@andreicristianpetcu

Copy link
Copy Markdown
Contributor

soooo..... is this in Nightly now?

@loganfsmyth
loganfsmyth deleted the source-map-worker-assets branch October 3, 2018 21:32
@loganfsmyth

Copy link
Copy Markdown
Contributor Author

@andreicristianpetcu It will be in nightly the next time changes from Github are synched back into Mozilla's primary repository, so not quite yet.

jasonLaster pushed a commit that referenced this pull request Oct 4, 2018
* Remove unused wrapper function in Webpack config generation.
* Update yarn.lock for recent package.json changes.
* Add a general concept of assets to the source-map worker.
* Upgrade the source-map module for WASM performance.
* Centralize sourcemap consumer construction.
egdbear pushed a commit to egdbear/debugger.html that referenced this pull request Oct 23, 2018
* Remove unused wrapper function in Webpack config generation.
* Update yarn.lock for recent package.json changes.
* Add a general concept of assets to the source-map worker.
* Upgrade the source-map module for WASM performance.
* Centralize sourcemap consumer construction.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants