Status

enhancement
P1
normal
RESOLVED FIXED
2 years ago
Last year

People

(Reporter: zer0, Unassigned)

Tracking

unspecified
Firefox 56
Dependency tree / graph
Bug Flags:
qe-verify -

Firefox Tracking Flags

(Not tracked)

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
Priority: -- → P2
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: 2 years ago
Resolution: --- → FIXED
Flags: needinfo?(zer0)
Priority: P2 → P1
Target Milestone: --- → Firefox 56
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.