Allow modules to be compiled directly from UTF-8

RESOLVED FIXED in Firefox 69

Status

()

enhancement
P1
normal
RESOLVED FIXED
29 days ago
25 days ago

People

(Reporter: Waldo, Assigned: Waldo)

Tracking

(Blocks 1 bug)

Trunk
mozilla69
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox69 fixed)

Details

Attachments

(6 attachments)

Assignee

Description

29 days ago
No description provided.
Priority: -- → P1

Comment 6

26 days ago
Pushed by jwalden@mit.edu:
https://hg.mozilla.org/integration/mozilla-inbound/rev/38beafecef9b
Templatize {Script,Module}ParseTask by Unit as a step toward enabling off-thread parsing of UTF-8 source.  r=arai
https://hg.mozilla.org/integration/mozilla-inbound/rev/41adfcfb70bf
Make StartOffThreadParseTask accept UniquePtr<ParseTask> and not just a raw ParseTask* borrowed (gulp) from one.  r=arai
https://hg.mozilla.org/integration/mozilla-inbound/rev/3ce85949f2e1
Rename one CompileModule overload to ParseModule to make it easier to distinguish them, both mentally and in searches.  r=arai
https://hg.mozilla.org/integration/mozilla-inbound/rev/cb71e51f8cad
Add ParseModule and CompileModule overloads for UTF-8.  r=arai
https://hg.mozilla.org/integration/mozilla-inbound/rev/6fac68354ef4
Begin fleshing out off-thread script/module compilation support for UTF-8.  r=arai
https://hg.mozilla.org/integration/mozilla-inbound/rev/d22bd25523e5
Make JS::CompileModule return the module directly, rather than by outparam duplicative of a boolean return value.  r=arai
https://hg.mozilla.org/integration/mozilla-inbound/rev/d31060048c95
Add a JS::CompileModule overload for direct UTF-8 compilation.  r=arai
Assignee

Comment 7

26 days ago

Whoops, I added some JSAPI entrypoints but not implementations of them. Let's get on that.

Keywords: leave-open

Comment 9

25 days ago
Pushed by jwalden@mit.edu:
https://hg.mozilla.org/integration/mozilla-inbound/rev/78f45a2b47cb
Define off-thread compilation APIs, of modules and scripts, for UTF-8.  r=arai
Assignee

Updated

25 days ago
Keywords: leave-open
You need to log in before you can comment on or make changes to this bug.