Commit Graph

1322 Commits

Author SHA1 Message Date
David Anderson
995cadf738 Move TaskFactory from dom/plugins to ipc/glue. (bug 1271180 part 1, r=aklotz)
--HG--
rename : dom/plugins/ipc/TaskFactory.h => ipc/glue/TaskFactory.h
extra : rebase_source : 485012c5a08eb0ce2a4225ddfe9cbfb454fc4302
2016-06-10 15:43:35 -04:00
Jonathan Watt
b15368cfcb Bug 1279451 - Remove a lot of unnecessary includes of nsAutoPtr.h. rs=sparky 2016-06-07 21:10:18 +01:00
James Andreou
7c273b3eb3 Bug 1276328 - Remove private browsing flags r=baku
--HG--
extra : rebase_source : e13f6b834ae22986776ca7cc3dd0cee3fc945992
2016-06-07 13:50:00 +02:00
Christoph Diehl
053c555889 Bug 777067 - "Fuzzing: IPC Protocol Definition Language (IPDL) Protocols". r=wmccloskey
--HG--
extra : rebase_source : f256b70d6f81f82339e52ab4f6bc2d9b4b751c08
2016-05-30 16:33:00 +02:00
Bob Owen
5714578c95 Bug 1278528: Don't try to initialize the sandbox TargetServices when we are not sandboxed. r=jimm
MozReview-Commit-ID: EpXy9LYXwQL
2016-06-07 14:03:51 +01:00
Gabor Krizsanits
6c0e1dc69f Bug 1146873 - Handling sandbox policy setup failures. r=bobowen 2016-06-06 15:13:33 +02:00
Jed Davis
78e49e2efb Bug 1114647 - Use firefox for child processes instead of plugin-container. r=ted
Disabled on Mac (content processes need to use plugin-container.app for
UI reasons) and on Linux unless --disable-sandboxing (build issues).

Based on work by George Wright <george@mozilla.com>.

--HG--
extra : amend_source : 43986e25743de21e3ddfb7893e3ed550fe6eef76
2016-06-03 12:49:39 -07:00
Honza Bambas
8db609c916 Bug 1275117 - Fix static strings leaks when mozlogging is on. r=jduell
--HG--
extra : rebase_source : 2a08f4eee9a900a4e34d58084eb222aedd319464
2016-05-31 08:11:00 -04:00
Haik Aftandilian
c95d552240 Bug 1275430 - Add telemetry and logging to record content process failures to start; r=billm
MozReview-Commit-ID: LWeZbDBwfX5
2016-06-01 15:13:59 -07:00
Jonathan Hao
525c086187 Bug 1259871 - Replace getSimpleCodebasePrincipal with createCodebasePrincipal. r=sicking
MozReview-Commit-ID: Frx0CjBzuve

--HG--
extra : histedit_source : 036eb321d9ccb20e0e071ba588b0a1249eb34bdd
2016-05-24 18:01:34 +08:00
Andrea Marchesini
8d58a3b347 Bug 1274363 - BroadcastChannel should dispatch events in the correct order, r=smaug 2016-05-30 10:48:40 +02:00
Andrew McCreight
5848fe3b73 Bug 1276318, part 2 - Fix mode lines in ipc/glue. r=billm
MessageChannel.{h,cpp}, MessageLink.{h,cpp}, and ProtocolUtils.h are
using 4-space indent so I left those alone.
2016-05-27 14:54:31 -07:00
Andrew McCreight
9449c5e62d Bug 1276318, part 1 - Fix leading tabs in ipc/glue. r=billm
Also remove a weird mode line so the script can fix it more easily in
the next part.
2016-05-27 14:54:30 -07:00
Bill McCloskey
7dbc02ed0f Bug 1262671 - Use BufferList for Pickle (r=froydnj) 2016-05-27 09:57:41 -07:00
Bill McCloskey
2e4314c9a7 Bug 1262671 - IPC ReadData/ReadBytes elimination (r=froydnj) 2016-05-27 09:57:39 -07:00
Bill McCloskey
291c555f34 Bug 1262671 - void** -> PickleIterator (r=froydnj) 2016-05-27 09:57:38 -07:00
Bill McCloskey
1369c6edad Bug 1273307 - Remove copy constructor/assignment operator for Message/Pickle (r=froydnj) 2016-05-27 09:57:35 -07:00
Honza Bambas
ff31be3edd Bug 1270752 - Fix lifetime of buffer passed to PR_SetEnv(). r=jduell
--HG--
extra : rebase_source : 7666e7e3f864fddb5536793a91c94bccf6bb2877
2016-05-26 03:18:00 -04:00
Bob Owen
ecee115838 Bug 1250125: Make a 0 security.sandbox.content.level turn off the content process sandbox. r=TimAbraldes
This also fixes a bug where we weren't setting parts of the policy correctly for levels 3 to 9.

MozReview-Commit-ID: IXsg2nGOqoa

--HG--
extra : rebase_source : 65c76a581dcd498c7d7d5b01e4f4e140acdb244f
2016-05-25 09:06:23 +01:00
Eric Rahm
d2eb11dfd1 Bug 1274404 - Annotate IPC message name and size when crashing due to message size. r=mccr8 2016-05-23 11:15:47 -07:00
Carsten "Tomcat" Book
0ed94efabc Backed out changeset 0659c9235136 (bug 1274363) for memory leaks
--HG--
extra : rebase_source : 7fd1df522bb0cab48ca51d29140036f6a5a6a7cc
2016-05-24 09:37:23 +02:00
Andrea Marchesini
2d4ce92868 Bug 1274363 - BroadcastChannel should dispatch events in the correct order, r=smaug 2016-05-24 06:59:15 +02:00
Sebastian Hengst
63e3ff7608 Backed out changeset 2a2a4005e935 (bug 1274404) for Asan bustage: use of undeclared identifier 'CrashReporter'. r=backout on a CLOSED TREE 2016-05-21 19:36:27 +02:00
Eric Rahm
ff177872a0 Bug 1274404 - Annotate IPC message name and size when crashing due to message size. r=mccr8 2016-05-20 14:30:39 -07:00
Bob Owen
9d7707097f Bug 1271601: If a child process fails to duplicate a TransportDescriptor pipe handle then send it anyway and get the target to duplicate. r=gabor
MozReview-Commit-ID: 5CKrQdDxGrg
2016-05-21 16:16:33 +01:00
Nathan Froyd
a5f42fc63a Bug 1262937 - part 8 - factor out array length deserialization errors; r=jld 2016-05-21 04:07:56 -04:00
Nathan Froyd
647562c2d2 Bug 1262937 - part 7 - factor out union type deserialization errors; r=jld 2016-05-21 04:07:56 -04:00
Nathan Froyd
dcc74a62c9 Bug 1262937 - part 5 - factor out actor reading code to a common base class; r=jld
Actor reading from IPC message is codegen'd with a lot of repeated code.
We can improve that by moving the core actor reading code out of
subclasses into IProtocolmanager.  While we still need to codegen a bit
of code to cast the read actor to the proper type, the code overall is
smaller.  The lone downside is that if we do encounter an error reading
the actor id out of the message, the precision of our crash messages is
reduced somewhat: we no longer have the protocol name doing the reading,
nor do we get crash report annotations, since we can't tell whether
we're in the parent or child process.
2016-05-21 04:07:56 -04:00
Nathan Froyd
bdd874023f Bug 1262937 - part 4 - publically inherit from MessageListener in IProtocol; r=jld
IProtocolManager is templated over some listener type.  In our IPDL
code, that type is always IProtocol, which is a subclass of
MessageListener.  It's also important to note that IProtocol uses
protected inheritance from MessageListener; the generated code takes
advantage of this inheritance structure when it reads actors:

  // ChannelListener is typedef'd to MessageListener
  // Lookup here is IProtocolManager::Lookup
  ChannelListener* listener = Lookup(id);

Lookup returns a pointer to the type over which IProtocolManager is
templated.  As mentioned above, that type is always IProtocol.  But
thanks to the containing class inheriting from *both* IProtocolManager
and IProtocol, the returned pointer can be silently upcasted to
MessageListener thanks to C++ visibility rules.

It's not clear that this restricted inheritance structure is actually
benefitting anybody, or that the inheritance hierarchy of protocol
classes is the best way to do things.  This particular implementation
detail is getting in the way for the next improvement, so let's make the
protected inheritance public instead.
2016-05-21 04:07:56 -04:00
Cervantes Yu
f3f969af61 Bug 1258317 - Part 2: Remove unused privilege in mozilla::ipc::TransferHandleTorProcess(). r=krizsa
Also fix a handle leak in mozilla::ipc::AnnotateProcessInformation().

MozReview-Commit-ID: DuepDytfoD2

--HG--
extra : rebase_source : 0b0dd05a3728e64962a774a2dcc35dc7646aa4dc
2016-05-06 19:01:29 +08:00
Ben Kelly
4c8ab874ba Bug 1093357 P2 Add an RAII type to safely handle serialized stream actors. r=jld
* * *
Bug 1093357 P2 interdiff 001 address review feedback
* * *
Bug 1093357 P2 interdiff 002 refactor to IPCStream type
2016-05-15 10:32:09 -07:00
Ben Kelly
749d2c1626 Bug 1093357 P1 Implement an actor for streaming pipes from child to parent. r=jld
* * *
Bug 1093357 P1 interdiff 001 address review feedback
2016-05-15 10:32:09 -07:00
Chris Peterson
353ee65255 Bug 1272513 - Part 1: Suppress -Wshadow warnings-as-errors in some directories. r=glandium 2016-05-11 00:00:01 -07:00
Kyle Huey
223dd41b6f Bug 1269056: Part 3 - Consolidate XPCOM and chromium event queues for the main thread. r=froydnj 2016-05-12 15:15:43 -07:00
Kyle Huey
3425031e06 Bug 1269056: Part 2 - Consolidate XPCOM and chromium event queues for non-main nsThreads. r=froydnj 2016-05-12 15:15:43 -07:00
Kyle Huey
c8143d07d7 Bug 1272415: Don't include task.h everywhere. r=froydnj 2016-05-12 15:15:43 -07:00
Kyle Huey
d3cc4b02b6 Bug 1268160: Assert that PostIdleTask is only used on the main thread. r=jld 2016-05-12 15:15:43 -07:00
Mason Chang
1ed5c6b7b4 Bug 1120485. Add more logging to find failed uses of MessageChannel::mEvent. r=milan 2016-05-10 17:59:15 -07:00
Eric Rahm
9caf0110e8 Bug 1268616 - Part 2: Stop sending messages that are too large. r=billm 2016-05-04 17:38:53 -07:00
Kyle Huey
941ab1f522 Bug 1268313: Part 7 - Move NS_NewRunnableMethod and friends to mozilla::NewRunnableMethod. r=froydnj 2016-05-05 01:45:00 -07:00
Kyle Huey
7f54121b91 Bug 1268313: Part 3 - Replace some NewCancelableRunnableMethod with NS_NewNonOwningCancelableRunnableMethod. r=froydnj 2016-05-05 01:44:59 -07:00
Kyle Huey
7311b10562 Bug 1268313: Part 2 - Replace some NewRunnableMethods with NS_NewNonOwningRunnableMethod. r=froydnj 2016-05-05 01:44:59 -07:00
Kyle Huey
63a71e96b0 Bug 1268313: Part 1 - Be explicit about which NewRunnableMethod callers want to be able to cancel. r=froydnj 2016-05-05 01:44:59 -07:00
Mason Chang
c30cae967e Bug 1120485. Add logging to check if we're closing MessageChannel::mHandle twice. r=milan 2016-05-04 11:03:49 -07:00
Andrew McCreight
a1f8b0904f Bug 1269365, part 3 - Use infallible array allocation in implementSpecialArrayPickling. r=froydnj
As with part 2, this will turn some allocation failures during
deserialization from IPC FatalError crashes into OOM crashes.
2016-05-03 09:29:39 -07:00
Andrew McCreight
0b08b02edb Bug 1269365, part 2 - Make ParamTraits<nsTArray<E>>::Read use infallible allocation. r=froydnj
This will turn allocation failures during deserialization from IPC
FatalError crashes into OOM crashes.
2016-05-03 09:29:39 -07:00
Andrew McCreight
6266959835 Bug 1269365, part 1 - Swap fallible and infallible TArray ParamTraits. r=froydnj
This should not change the behavior, because fallible_t is passed to
the array methods.

Rename from InfallibleTArray to nsTArray, because that is more usual.

Remove unnecessary spaces from some template definitions.
2016-05-03 09:29:39 -07:00
Andrew McCreight
56f812a0d5 Bug 1268938 - Use the name of the original message in Send for reply telemetry. r=billm
aReply usually won't have a name properly set, but aMsg does. To avoid
accessing aMsg after it dies, copy the pointer to the name.
2016-05-02 20:37:40 -07:00
Thomas Zimmermann
42c137c01d Bug 1268432: Replace |Task| with |Runnable| in B2G code r=fabrice
This patch converts the remaining B2G code from |Task| to |Runnable| and
fixes related API calls. This is a follow-up to bug 1266595.
2016-05-02 10:27:15 -07:00
Carsten "Tomcat" Book
f13688180e Backed out changeset af51821b2fc5 (bug 1268313) for causing very frequent bc7 memory leak
--HG--
extra : rebase_source : 0af725bfab61b8f86e0243490aa970535f726503
2016-04-29 14:22:04 +02:00