Last Comment Bug 784726 - Remove old IPC::URI
: Remove old IPC::URI
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: IPC (show other bugs)
: unspecified
: All All
: -- normal (vote)
: mozilla17
Assigned To: Ben Turner (not reading bugmail, use the needinfo flag!)
:
Mentors:
Depends on: 789059 801882
Blocks: 792238
  Show dependency treegraph
 
Reported: 2012-08-22 10:49 PDT by Ben Turner (not reading bugmail, use the needinfo flag!)
Modified: 2012-10-15 14:59 PDT (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Patch, v1 (161.17 KB, patch)
2012-08-22 10:49 PDT, Ben Turner (not reading bugmail, use the needinfo flag!)
cjones.bugs: review+
khuey: review+
Details | Diff | Splinter Review

Description Ben Turner (not reading bugmail, use the needinfo flag!) 2012-08-22 10:49:43 PDT
Created attachment 654260 [details] [diff] [review]
Patch, v1

The old IPC::URI code is fragile and unsafe, just like bug 782649. Removing it all here. Once that was gone nsIIPCSerializableObsolete was no longer used and so that has been removed as well.

Chris, I just need review from you on ipc/ipdl bits.

Kyle, you get the rest :)
Comment 1 Ben Turner (not reading bugmail, use the needinfo flag!) 2012-08-23 12:36:30 PDT
https://hg.mozilla.org/integration/mozilla-inbound/rev/c813eeb62b92
Comment 2 Ryan VanderMeulen [:RyanVM] 2012-08-23 19:17:12 PDT
https://hg.mozilla.org/mozilla-central/rev/c813eeb62b92

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