721863 - Source Editor needs to use the minimal requirejs implementation

Status: RESOLVED WONTFIX

(DevTools :: Source Editor, defect, P3)

Description

[Mihai Sucan [:msucan]]

Orion codebase uses requirejs and they currently have a minimal shim for define() and require(), but this is going to be removed. We need to update the Source Editor code to use our requirejs shim. Also, we should extract it from gcli.jsm and put into its own browser/devtools/shared/requirejs.jsm or something.

Comment 1

[Rob Campbell [:rc] (:robcee)]

I hear you like modules...

Comment 2

[Joe Walker [:jwalker]]

Currently our require shim lives in dryice, which makes a ton of sense from a dryice point of view - it's building script files that depend on it.

However from a mozilla pov, it makes no sense to have many copies of the same thing, so there is work to do. 

However the pain felt is duplication of a few ks of script. i.e. low.
Also, does the orion integration still need this?

P.S. Sup dawg.

Comment 3

[Mihai Sucan [:msucan]]

(In reply to Joe Walker from comment #2)
> Currently our require shim lives in dryice, which makes a ton of sense from
> a dryice point of view - it's building script files that depend on it.
> 
> However from a mozilla pov, it makes no sense to have many copies of the
> same thing, so there is work to do. 
> 
> However the pain felt is duplication of a few ks of script. i.e. low.
> Also, does the orion integration still need this?

This is a recent change in Orion. They still have the requirejs shim in their codebase, but as soon as they remove it, this bug is going to become pretty much P1 for any further Orion updates.

P4 works, as long as they don't remove the requirejs shim. ;)

In terms of duplication: the approach we use to fix this bug can vary. We can either move the code you have outside into a reusable jsm, or we can duplicate it in SourceEditor, or whatever we think it's best at that point.

Comment 4

[Joe Walker [:jwalker]]

(In reply to Mihai Sucan [:msucan] from comment #3)
> This is a recent change in Orion. They still have the requirejs shim in
> their codebase, but as soon as they remove it, this bug is going to become
> pretty much P1 for any further Orion updates.
> 
> P4 works, as long as they don't remove the requirejs shim. ;)
> 
> In terms of duplication: the approach we use to fix this bug can vary. We
> can either move the code you have outside into a reusable jsm, or we can
> duplicate it in SourceEditor, or whatever we think it's best at that point.

Thanks - that background helps.
The good news is that I think this should be quite simple (assuming their current shim is similar to ours)

Comment 5

[Joe Walker [:jwalker]]

GCLI Triage.

Comment 6

[Joe Walker [:jwalker]]

This bug is fixed by bug 742672. I'll probably mark this as a duplicate when that lands.

Comment 7

[Mihai Sucan [:msucan]]

(In reply to Joe Walker from comment #6)
> This bug is fixed by bug 742672. I'll probably mark this as a duplicate when
> that lands.

Great!

Please keep this bug open. We still need to update the Source Editor as well, to use the new require.jsm.

Thank you!

Comment 8

[Joe Walker [:jwalker]]

(In reply to Mihai Sucan [:msucan] from comment #7)
> (In reply to Joe Walker from comment #6)
> > This bug is fixed by bug 742672. I'll probably mark this as a duplicate when
> > that lands.
> 
> Great!
> 
> Please keep this bug open. We still need to update the Source Editor as
> well, to use the new require.jsm.

OK, in which case it doesn't block bug 745773.

Comment 9

[Mihai Sucan [:msucan]]

This is going to be fixed by bug 759351.

Comment 10

[Rob Campbell [:rc] (:robcee)]

Moving to Source Editor component.

Filter on CHELICERAE.

Comment 11

[Anton Kovalyov (:anton)]

Is this still an issue with SourceEditor in general or CodeMirror specifically?

Comment 12

[Mihai Sucan [:msucan]]

(In reply to Anton Kovalyov (:anton) from comment #11)
> Is this still an issue with SourceEditor in general or CodeMirror
> specifically?

This bug is no longer applicable. We had Orion integrated with its own requirejs shim. This bug was about switching from their requirejs shim to ours.