Jeff Griffiths (:canuckistani) (:⚡︎) Reporter
	Description • 8 years ago

Firefox OS has been end-of-lifed and Web IDE has never been used by a lot of people. We should probably remove it timed with the mass-removal of other B2G-related code.

	J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)
	Comment 1 • 8 years ago

WebIDE is also our primary remote device connection UI at the moment.  I would think we should hold off until we've ported this feature set into about:debugging.

That work is tracked in bug 1212802.

	Jeff Griffiths (:canuckistani) (:⚡︎) Reporter
	Comment 2 • 8 years ago

(In reply to J. Ryan Stinnett [:jryans] (use ni?) from comment #1)
> WebIDE is also our primary remote device connection UI at the moment.  I
> would think we should hold off until we've ported this feature set into
> about:debugging.

Totally agree.

	Patrick Brosset <:pbro>
	Comment 3 • 7 years ago

We're trying to stop using certain SDK APIs in bug 1350645, including sdk/system/child_process/subprocess which happens to be used in WebIDE. So removing WebIDE would help bug 1350645.

Now, if the blocking bug here (bug 1212802) takes too long to resolve and we want to remove sdk/system/child_process/subprocess before, then we'll need to figure something else and file another bug.

	Dietrich Ayala (:dietrich)
	Comment 4 • 7 years ago

Confirmed on IRC in #devtools, WebIDE is the only way to do wifi debugging of web content on Firefox Android from desktop Firefox.

It would be a bummer to lose that ability.

What are the alternative APIs for sdk/system/child_process/subprocess?

	J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)
	Comment 5 • 7 years ago

The usages of child_process are for building apps (not important anymore) and running Firefox OS simulators (also not important).

So, we can remove these bits as needed without deleting WebIDE entirely.

	Alexandre Poirot [:ochameau]
	Comment 6 • 7 years ago

This is also used by adbhelper to run adb/fastboot.
But I imagine we can bundle our copy of subprocess.js

	Julien Cristau [:jcristau]
	Comment 7 • 7 years ago

Mass wontfix for bugs affecting firefox 52.

	Tim Nguyen :ntim
	Comment 8 • 7 years ago

We should consider changing the WebIDE button to an about:debugging button.

	J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)
	Comment 9 • 7 years ago

(In reply to Tim Nguyen :ntim from comment #8)
> We should consider changing the WebIDE button to an about:debugging button.

If we keep the button, let's at least make sure we remove the strange "auto-insertion into main toolbar" behavior, which I think mostly just annoyed people.  (I believe this happens inside WebIDE itself.)

	Kris Maglione [:kmag]
	Comment 10 • 7 years ago

(In reply to Alexandre Poirot [:ochameau] from comment #6)
> This is also used by adbhelper to run adb/fastboot.
> But I imagine we can bundle our copy of subprocess.js

Please don't. That module has a mess of issues, which is why it was already removed from the SDK. You can use Subprocess.jsm directly instead.

	Patrick Brosset <:pbro>
	Comment 11 • 7 years ago

Quick summary of the state of things here:

Currently, WebIDE is our only way to discover and connect to remote devices.
We want to remove it, but before we do this, we want to move this functionality to the about:debugging page instead.

For this, we need
1. Bug 1243329 - This will bring the host/port connect part. We already have (rebased) patches for this and might be able to land this in Q2 2017.
2. Bug 1212802 - To discover devices, connect over wifi, etc. For now, we have not planned this work.

	Chris Mills [:cmills]
	Comment 12 • 7 years ago

Adding dev-doc-needed, so we can make sure to keep an eye on this and update docs when the time comes.