Alexandre Poirot [:ochameau] Assignee
	Description • 9 years ago

Once bug 1007061 lands, there will still be some resource in /devtools/app-manager/, files being still used by WebIDE.
We should integrate them into WebIDE folders or get rid of these deps.

	Alexandre Poirot [:ochameau] Assignee
	Comment 2 • 9 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=453b75353826

	Alexandre Poirot [:ochameau] Assignee
	Comment 3 • 9 years ago

Also includes some require(Services) / require(promise) cleanups
as well as some removal of app-manager references.
(that are not about the webide/modules/app-manager module!)

I also tried to clean bugzilla from some app-manager bugs/references.

	Alexandre Poirot [:ochameau] Assignee
	Comment 4 • 9 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=66e1a83fe740

	Alexandre Poirot [:ochameau] Assignee
	Comment 5 • 9 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=5482a4ab59a4

	Alexandre Poirot [:ochameau] Assignee
	Comment 6 • 9 years ago

Last time you are going to hear about the app manager ;-)

Wait?! No, there is this the webide module!!

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

Comment on attachment 8680711 [details] [diff] [review]
patch v3

Review of attachment 8680711 [details] [diff] [review]:
-----------------------------------------------------------------

Great work! :D

::: browser/locales/jar.mn
@@ +77,5 @@
>      locale/browser/devtools/connection-screen.dtd  (%chrome/browser/devtools/connection-screen.dtd)
>      locale/browser/devtools/connection-screen.properties (%chrome/browser/devtools/connection-screen.properties)
>      locale/browser/devtools/font-inspector.dtd     (%chrome/browser/devtools/font-inspector.dtd)
>      locale/browser/devtools/har.properties         (%chrome/browser/devtools/har.properties)
> +    locale/browser/devtools/app-validator.properties (%chrome/browser/devtools/app-validator.properties)

I believe this will cause all the strings to be re-translated... not sure it's worth it?  On the other hand, it's probably left untranslated in many locales.  I leave it up to you.

	Alexandre Poirot [:ochameau] Assignee
	Comment 8 • 9 years ago

Do not rename app-manager.properties.

	Alexandre Poirot [:ochameau] Assignee
	Comment 9 • 9 years ago

https://hg.mozilla.org/integration/fx-team/rev/a8cc3e63866f20e70104c6b426ea93463a0e02cb
Bug 1216590 - Merge last bits of app-manager into WebIDE. r=jryans

	Carsten Book [:Tomcat]
	Comment 10 • 9 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/a8cc3e63866f

	Carsten Book [:Tomcat]
	Comment 11 • 9 years ago
bugherder uplift

https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/a8cc3e63866f