1245914 - Add a -xpcshell option to SeaMonkey

Status: ASSIGNED

(SeaMonkey :: General, enhancement)

Description

[Philip Chee]

(In reply to Mike Hommey [:glandium] from bug 1238769 comment #0)
> Now that xpcshell is just a (literal) shell around XRE_XPCShellMain, we can
> actually just make Firefox itself work as the xpcshell.

Comment 1

[Tony Mechelynck [:tonymec]]

(In reply to Philip Chee from comment #0)
> (In reply to Mike Hommey [:glandium] from bug 1238769 comment #0)
> > Now that xpcshell is just a (literal) shell around XRE_XPCShellMain, we can
> > actually just make Firefox itself work as the xpcshell.

IIUC this bug consists of porting to SeaMonkey Firefox bug 1238769 (assigned to glandium and which, as of this writing, has been pushed to inbound, backed out, and then given an updated patch)? The patch consists of an added else-block in one if statement and a few sporadic lines elsewhere. I'm interested to try, but not before that other bug gets FIXED, and only if no one else wants to.

Comment 2

[rn10950]

I can take this.

Comment 3

[rn10950]

Attachment: bug1245914_xpcshell.diff

When I run the executable with -xpcshell, nothing happens, seamonkey.exe runs and closes with no error or any output in the parent CMD window, however, the latest Firefox Nightly (build 20160214030236) behaves the same way, so I am submitting this patch as-is. (bug 1238769's patch was merged to m-c on 2016-02-10) Otherwise, the suite runs fine, and there are no build errors.

Comment 4

[Tony Mechelynck [:tonymec]]

Comment on attachment 8719380 [details] [diff] [review]
bug1245914_xpcshell.diff

Why the additional indent at new line 128?   {nullptr, nullptr}

Comment 5

[rn10950]

(In reply to Tony Mechelynck [:tonymec] from comment #4)
> Comment on attachment 8719380 [details] [diff] [review]
> bug1245914_xpcshell.diff
> 
> Why the additional indent at new line 128?   {nullptr, nullptr}

For some reason, Notepad++ replaced the 4 spaces with a tab. I went back in and changed it.

Comment 6

[rn10950]

Attachment: patch with wierd indent fixed

Comment 7

[Tony Mechelynck [:tonymec]]

(In reply to rn10950 from comment #3)
> Created attachment 8719380 [details] [diff] [review]
> bug1245914_xpcshell.diff
> 
> When I run the executable with -xpcshell, nothing happens, seamonkey.exe
> runs and closes with no error or any output in the parent CMD window,
> however, the latest Firefox Nightly (build 20160214030236) behaves the same
> way, so I am submitting this patch as-is. (bug 1238769's patch was merged to
> m-c on 2016-02-10) Otherwise, the suite runs fine, and there are no build
> errors.

The added lines indicate that any command-line arguments after -xpcshell, and any environment variables, get passed to XRE_XPCShellMain() at new line 172. What these arguments and variables might be, the Firefox bug doesn't say, but I suppose they are described somewhere, maybe in the MDN or mozwiki documentation (if there is any) about xpcshell.

Comment 8

[Philip Chee]

Comment on attachment 8719564 [details] [diff] [review]
patch with wierd indent fixed

Neil is a better reviewer for C++ code.

Comment 9

[p.sonalipatro]

Hello, can I please work on this bug?

Comment 10

[Frank-Rainer Grahl (:frg)]

Comment on attachment 8719564 [details] [diff] [review]
patch with wierd indent fixed

Neil is no longer active here. rn10950 also sees to be gone from bugzilla. p.sonalipatro I think this bug is no no longer valid. nsSuiteGlue.app was aligned with the one in browser in several recent bugs. Could you check.