Closed Bug 1378832 Opened 4 years ago Closed 4 years ago

Stop using SDK in WebIDE

Categories

(DevTools Graveyard :: WebIDE, enhancement, P1)

enhancement

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 56

People

(Reporter: zer0, Unassigned)

References

Details

(Whiteboard: [nosdk])

SDK modules used in WebIDE:

  sdk/system/child_process/subprocess
  sdk/system/environment
  sdk/system/runtime
Component: Developer Tools → Developer Tools: WebIDE
Depends on: 1378833
Depends on: 1378835
Shouldn't it be resolved invalid due to Bug 1314811?
Flags: qe-verify-
Whiteboard: [nosdk]
(In reply to Kuba Niewiarowski from comment #1)
> Shouldn't it be resolved invalid due to Bug 1314811?
Removing the WebIDE entirely is a bigger project than what we can take on now, and we need to migrate away from the add-on SDK quickly. So this bug still makes sense.
Regarding
http://searchfox.org/mozilla-central/source/devtools/client/webide/modules/build.js#9
I imagine we could remove this feature as it was really specific to apps and quite cryptic.

And for
http://searchfox.org/mozilla-central/source/devtools/client/webide/modules/simulator-process.js#12
We may use Subprocess.jsm, like the browser toolbox
http://searchfox.org/mozilla-central/source/devtools/client/framework/ToolboxProcess.jsm#17
Running simulators still work! and you can still debug all apps. This demonstrates that tool clients work still great on old actors.
We may also remove if that's easier as this is really specific to Firefox OS. But that may end up removing a lot of features, not necessarely well contained.
Is that everything we need to remove here?
Flags: needinfo?(pbrosset)
(In reply to J. Ryan Stinnett [:jryans] (use ni?) from comment #5)
> Is that everything we need to remove here?

I think so, fwd ni? to Matteo since Patrick is on PTO for 2 more weeks.
There are no references to the SDK left in devtools/client/webide.
Flags: needinfo?(pbrosset) → needinfo?(zer0)
Yes, we are good for WebIDE.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Flags: needinfo?(zer0)
Priority: P2 → P1
Target Milestone: --- → Firefox 56
Product: Firefox → DevTools
Product: DevTools → DevTools Graveyard
You need to log in before you can comment on or make changes to this bug.