Bug 1773174 Comment 13 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

For me, the biggest reason of the rename was to make these files target of the shim added by bug 1766761 (that supported only `*.jsm` extension),
so that in-tree migration becomes easier.

Then, I overlooked about the out-of-tree consumer, at least for services/, as pointed out in 1609269 comment #11,
and that requires supporting `*.js` in the shim, and that at the same time requires recognizing `.jsm.js` separately.
And those 2 are supported in bug 1773603.

Now that `*.js` and `.jsm.js` are supported in the shim, there's not much reason to rename all those files immediately.
I still prefer renaming them, but leaving them as is safer option.
So, I tentatively abandoned those patches.  If there's any file that's definitely not referred from out-of-tree, I'll revive the patch.
For me, the biggest reason of the rename was to make these files target of the shim added by bug 1766761 (that supported only `*.jsm` extension),
so that in-tree migration becomes easier.

Then, I overlooked about the out-of-tree consumer, at least for services/, as pointed out in bug 1609269 comment #8,
and that requires supporting `*.js` in the shim, and that at the same time requires recognizing `.jsm.js` separately.
And those 2 are supported in bug 1773603.

Now that `*.js` and `.jsm.js` are supported in the shim, there's not much reason to rename all those files immediately.
I still prefer renaming them, but leaving them as is safer option.
So, I tentatively abandoned those patches.  If there's any file that's definitely not referred from out-of-tree, I'll revive the patch.
For me, the biggest reason of the rename was to make these files target of the shim added by bug 1766761 (that supported only `*.jsm` extension),
so that in-tree migration becomes easier.

Then, I overlooked about the out-of-tree consumer, at least for services/, as pointed out in bug 1609269 comment #8,
and that requires supporting `*.js` in the shim, and that at the same time requires recognizing `.jsm.js` separately.
And those 2 are supported in bug 1773603.

Now that `*.js` and `.jsm.js` are supported in the shim, there's not much reason to rename all those files immediately.
I still prefer renaming them, but leaving them as is is safer option.
So, I tentatively abandoned those patches.  If there's any file that's definitely not referred from out-of-tree, I'll revive the patch.

Back to Bug 1773174 Comment 13