Closed
Bug 933105
Opened 11 years ago
Closed 10 years ago
Implement SetThisProcessName on OS X
Categories
(Core :: IPC, defect)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: reuben, Unassigned)
Details
Attachments
(1 file)
4.68 KB,
patch
|
gerv
:
review-
|
Details | Diff | Splinter Review |
It's kinda helpful for debugging: http://cl.ly/image/0Z3O3c1M2F2V Also, why not? Flagging bent for IPC, smichaud for Core Foundation usage, and gerv for licensing. gerv, the SetProcessTitle function is copied from the Chromium code, is this the right way to do this?
Attachment #825089 -
Flags: review?(smichaud)
Attachment #825089 -
Flags: review?(gerv)
Attachment #825089 -
Flags: review?(bent.mozilla)
Reporter | ||
Updated•11 years ago
|
Assignee: nobody → reuben.bmo
Comment 1•11 years ago
|
||
Comment on attachment 825089 [details] [diff] [review] Implement ipc::SetThisProcessName on OS X If this file is mostly Chromium code, then just give it the Chromium licence. Files with multiple different licenses are a pain. In the boilerplate, please replace the text "the LICENSE file" with "http://src.chromium.org/viewvc/chrome/trunk/src/LICENSE". Gerv
Attachment #825089 -
Flags: review?(gerv) → review-
Comment 2•11 years ago
|
||
Comment on attachment 825089 [details] [diff] [review] Implement ipc::SetThisProcessName on OS X In my quest to reduce Ben's review queue, he says smichaud's review will cover this.
Attachment #825089 -
Flags: review?(bent.mozilla)
Reporter | ||
Updated•10 years ago
|
Attachment #825089 -
Flags: review?(smichaud)
Reporter | ||
Updated•10 years ago
|
Assignee: reuben.bmo → nobody
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•