Last Comment Bug 660158 - Remove head_ipc_setup.js
: Remove head_ipc_setup.js
[good first bug][mentor=jdm]
Product: Core
Classification: Components
Component: Networking (show other bugs)
: Trunk
: All All
-- normal (vote)
: mozilla8
Assigned To: Paul Nanowiz
: Patrick McManus [:mcmanus]
Depends on: 534764
  Show dependency treegraph
Reported: 2011-05-26 23:55 PDT by Josh Matthews [:jdm]
Modified: 2011-07-21 00:12 PDT (History)
7 users (show)
emorley: in‑testsuite-
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Bug 660158 - Remove all references to head_ipc.js (355 bytes, text/plain)
2011-07-13 16:41 PDT, Paul Nanowiz
no flags Details
Bug 660158 - Remove all references to head_ipc.js (1.98 KB, patch)
2011-07-13 18:46 PDT, Paul Nanowiz
jduell.mcbugs: review+
Details | Diff | Splinter Review

Description User image Josh Matthews [:jdm] 2011-05-26 23:55:52 PDT
Bug 534764 landed, and the comments say to remove the hack when this has occurred.
Comment 1 User image Josh Matthews [:jdm] 2011-05-30 11:52:08 PDT
Really simple bug for someone to get used to the process of contributing a patch.
Comment 2 User image Paul Nanowiz 2011-07-09 23:46:50 PDT
Hello...I would like to fix this bug. I am new here and very interested in learning. I write C/C++ and many other languages. Please let me work on this bug as I really want to learn how this whole thing works.
Comment 3 User image Josh Matthews [:jdm] 2011-07-10 08:08:52 PDT
Great! All you should need to do is remove the file netwerk/test/unit_ipc/head_ipc_setup.js and netwerk/cookie/test/unit_ipc/head_ipc_setup.js, then remove the references to those files from netwerk/test/unit_ipc/xpcshell.ini and netwerk/cookie/test/unit_ipc/xpcshell.ini. Does that make sense?
Comment 4 User image Ed Morley [:emorley] 2011-07-10 16:03:55 PDT
Paul, if you haven't already come across then it may save some time/effort :-)

The following MXR searches should point you in the
Comment 6 User image Paul Nanowiz 2011-07-10 22:34:39 PDT
Hello Everyone:

Thanks for providing me direction that will help to resolve this bug.

@Josh: Yes, I understand your explanation and should be able to work myself through the code to fix the error.
Comment 7 User image Paul Nanowiz 2011-07-11 21:16:05 PDT
Here is what I have done:

1. I downloaded the source 6.0b1
2. Next, I proceeded as Josh Matthews described: Removed the files netwerk/test/unit_ipc/head_ipc_setup.js and netwerk/cookie/test/unit_ipc/head_ipc_setup.js, then removed the references to the files from netwerk/test/unit_ipc/xpcshell.ini and netwerk/cookie/test/unit_ipc/xpcshell.ini.
3. Then, I rebuilt it: make -f build.

Is the next thing is to create the patch file using the instructions at: ?
Comment 8 User image Josh Matthews [:jdm] 2011-07-11 21:28:55 PDT
That's one way, but probably not the best way. You should get the source with mercurial from, and that will allow you to create a patch much easier.
Comment 9 User image Paul Nanowiz 2011-07-12 19:01:49 PDT
I have now set up Mercurial on my computer and ready to build it I am not sure which version I should actually build. I have downloaded the mozilla-central (main development tree) version but I am also wondering if it should be the mozilla-beta (prerelease development tree) version. Please clarify so I can be sure I am working with the right version. 

Also, the "Assigned To" on this page says no one has been assigned to it, could you please assign to me? Thank you.
Comment 10 User image Josh Matthews [:jdm] 2011-07-12 19:04:40 PDT
mozilla-central is the place where all current development occurs, so that's the tree you want to be using.
Comment 11 User image Paul Nanowiz 2011-07-12 21:35:05 PDT
I have one more question: using the information provided at and , I created mercurial.hgrc in my home folder as follows:

username = Paul Nanowiz <>
merge = kdiff3 

kdiff3.args=--auto -L1 base --L2 local --L3 other $base $local $other -o $output

qnew = -Ue

[extensions] = 

git = 1
showfunc = 1
unified = 8

However, when I use the command: hg qnew bugname.patch, I get an error: hg: unknown command 'qnew'
'qnew' is provided by the following extension:

    mq  manage a stack of patches

use "hg help extensions" for information on enabling extensions.

mq extension is enabled but it still gives an error, could you please advise what is wrong with the .hgrc file? Thank you.
Comment 12 User image Paul Nanowiz 2011-07-13 16:41:39 PDT
Created attachment 545778 [details]
Bug 660158 - Remove all references to head_ipc.js

Hello Jason Duell,

I learned from #introduction that you will be reviewing this patch after checkin, hence I am sending it with your name on it. I have also run "make" after the patch and everything seemed fine.

Please let me know your review comments when they are available.

Thank you.

Comment 13 User image Jason Duell [:jduell] (needinfo me) 2011-07-13 18:12:38 PDT
Comment on attachment 545778 [details]
Bug 660158 - Remove all references to head_ipc.js

Paul, it looks like you accidentally attached your .hgrc file instead of the patch?   Or do you still need help getting your .hgrc to work correctly?

If not, and you've made the changes for this bug and are ready for a review, follow the instructions here

to create a patch, and mark it "review?" with my email address as the requestee. (don't mark anything checkin+: clearing that flag for this.  Reviews come before checkins).

Comment 14 User image Paul Nanowiz 2011-07-13 18:46:23 PDT
Created attachment 545807 [details] [diff] [review]
Bug 660158 - Remove all references to head_ipc.js

Hello Jason Duell,

Sorry for the error in attaching the wrong file. I have now attached the one at /home/paul/Projects/src/.hg/patches. 

Thank you and apologies for the inconvenience. 

Comment 15 User image Jason Duell [:jduell] (needinfo me) 2011-07-15 14:42:59 PDT
Comment on attachment 545807 [details] [diff] [review]
Bug 660158 - Remove all references to head_ipc.js

Review of attachment 545807 [details] [diff] [review]:


Sending to try server just for quick sanity check: xpcshell-only run:

Paul: this is just so we run the tests on all platforms before we check the code in.
Comment 16 User image Jason Duell [:jduell] (needinfo me) 2011-07-15 18:01:02 PDT
Try is green.  Landed on mozilla-inbound:

Paul: if that's green too, your code will "automatically" (by magical merge-sheriffs :) get checked into mozilla-central, and this will be marked FIXED.

Thanks for your help!
Comment 17 User image Joe Drew (not getting mail) 2011-07-16 18:48:49 PDT
This (along with most things committed on Friday afternoon) was backed out of mozilla-inbound in order to clear up orange.
Comment 18 User image Ed Morley [:emorley] 2011-07-17 00:48:54 PDT
Re-marking checkin-needed.

Paul: What Joe meant was that some of the automated tests on the mozilla-inbound branch were failing (so show up orange instead of green): . 

Normally it's possible to tell what caused the breakage and back just that out, but in this case mozilla-inbound ended up in a bit of a mess, so a bunch of stuff has been pulled out as the easiest solution.

Since we know this patch wasn't the cause (since it's already been tested via it's own dedicated try server test run in comment 15:, I've marked this bug as checkin-needed again (in the keywords field). This means it will show up for anyone who has commit privileges and is feeling altruistic - so that they can land it again :-)
Comment 19 User image :Ms2ger (⌚ UTC+1/+2) 2011-07-17 04:01:26 PDT
Thanks, Paul and Ed.
Comment 20 User image Joe Drew (not getting mail) 2011-07-17 17:35:48 PDT
Comment 21 User image Paul Nanowiz 2011-07-21 00:12:18 PDT
Thanks to everyone for help with my first will go hunting for more bugs and things to do:)

Note You need to log in before you can comment on or make changes to this bug.