748858 - Use the startPoint argument of mozIDOMApplication.launch

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

The point use the starPoint argument of mozIDOMApplication.launch

Comment 1

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Attachment: Patch

Comment 2

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/4776d75914f4

Comment 3

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/4776d75914f4

Comment 4

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 618359 [details] [diff] [review]
Patch

>+  fullLaunchPath: function(aStartPoint) {
>+    let launchPath = this._localeProp("launch_path") || "";
>+    return this._origin.resolve(launchPath + aStartPoint);

You are breaking any callers that do not pass in | aStartPoint | to fullLaunchPath(), which is desktop and mobile Firefox