Commit Graph

595 Commits

Author SHA1 Message Date
Andrew McCreight
470cef59ea Bug 1169382 - Don't generate commented out NS_INTERFACE_CLASS or NS_ABSTRACT_CLASS in IPDL. r=billm 2015-06-06 14:37:59 -07:00
Mike Hommey
806e0220a2 Bug 991983 - Use objdir-relative SOURCES instead of GENERATED_SOURCES. r=gps 2015-05-28 07:34:16 +09:00
Phil Ringnalda
198fda1fe8 Back out 7842a044a183 (bug 1167431) for widespread bustage
CLOSED TREE
2015-05-22 18:53:41 -07:00
Ben Turner
a53a3243ec Bug 1167431 - 'Properly clean up actors whose RecvFooConstructor method returns false', r=billm. 2015-05-22 16:10:02 -07:00
Birunthan Mohanathas
735ccdd101 Bug 968520 - Add mozilla::fallible to Fallible{Auto,}TArray::SetLength calls. r=froydnj 2015-05-18 13:50:35 -07:00
Blake Kaplan
25d39ad698 Bug 1155494 - Add a 'compressall' message flag. f=handyman r=billm (parts of this patch were written by handyman with r=billm) 2015-05-05 16:20:42 -07:00
Nathan Froyd
91108f0f63 Bug 1158905 - remove dead code from protocol Transition functions; r=bent
PFoo::Transition currently looks something like:

bool
Transition(
        State from,
        mozilla::ipc::Trigger trigger,
        State* next)
{
    switch (from) {
    ...
    default:
        NS_RUNTIMEABORT("corrupted actor state");
        return false;
    }
    (*(next)) = __Error;
    return false;
}

Coverity complains that the assignment (*(next)) = __Error will never be
reached, and rightfully so.  We can remove this fallthrough block when
there are no states declared in the individual protocol.
2015-04-27 13:29:37 -04:00
Nathan Froyd
b73bbd3af0 Bug 1158896 - prevent dead code from being generated in lower.py; r=bent
Coverity was complaining that we have things like:

    if ((!(actor))) {
        return false;
    }

    if ((!(actor))) {
        return false;
    }

in the generated code, as the second return will clearly never be hit.  To
address this, let's remove a redundant call to dtorPrologue.
2015-04-27 13:08:12 -04:00
Jim Blandy
6afdd6c96b Bug 1145051: Use MOZ_WARN_UNUSED_RESULT instead of NS_WARN_UNUSED_RESULT. r=glandium
--HG--
extra : rebase_source : d210822225d307f3e3685cb5f95b461c48c0be7e
2015-03-19 00:46:40 -07:00
Bill McCloskey
79abd6e01a Bug 1128457 - Change handling of urgent messages during timeout (r=dvander) 2015-04-13 18:11:49 -07:00
Ben Turner
51ec0b56dc Bug 1135344 - Don't let IPDL use names that are reserved for compilers, r=froydnj. 2015-04-09 11:35:18 -07:00
Nathan Froyd
7561c4089c Bug 1150266 - fix IPDL thinko for never-inline method declarations; r=bent 2015-04-01 20:40:16 -04:00
Bob Owen
283c8df550 Bug 1149971 Part 2: Cet rid of kCurrentProcessId as it makes lots of static initializers. r=dvander 2015-04-01 13:59:20 +01:00
Bob Owen
967713e4e9 Bug 1149971 Part 1: Follow-up to bug 1119878 to fix broken IPDL tests. r=dvander 2015-04-01 16:44:48 +01:00
Bob Owen
e4f543bb58 Bug 1119878 Part 2: Change IPC code to hold ProcessID instead of ProcessHandle. r=billm, r=dvander, r=aklotz, r=cpearce 2015-04-01 09:40:35 +01:00
Ehsan Akhgari
bf2df719b1 Bug 1145631 - Part 4: Replace MOZ_FINAL with final in the IPDL codegen; r=froydnj 2015-03-21 12:28:36 -04:00
Ehsan Akhgari
883849ee32 Bug 1145631 - Part 1: Replace MOZ_OVERRIDE and MOZ_FINAL with override and final in the tree; r=froydnj
This patch was automatically generated using the following script:

function convert() {
echo "Converting $1 to $2..."
find . \
       ! -wholename "*/.git*" \
       ! -wholename "obj-ff-dbg*" \
         -type f \
      \( -iname "*.cpp" \
         -o -iname "*.h" \
         -o -iname "*.c" \
         -o -iname "*.cc" \
         -o -iname "*.idl" \
         -o -iname "*.ipdl" \
         -o -iname "*.ipdlh" \
         -o -iname "*.mm" \) | \
    xargs -n 1 sed -i -e "s/\b$1\b/$2/g"
}

convert MOZ_OVERRIDE override
convert MOZ_FINAL final
2015-03-21 12:28:04 -04:00
Bill McCloskey
e26a687773 Bug 1121676 - Use a lock to protect the list of top-level actors (r=bent) 2015-03-20 12:02:41 -07:00
Bill McCloskey
ba40fd679d Bug 1142109 - Process incoming urgent messages before sending (r=dvander) 2015-03-17 13:43:34 -07:00
Bill McCloskey
1004991381 Bug 1142109 - Fix IPDL tests (r=dvander) 2015-03-17 13:43:28 -07:00
Peter Van der Beken
5914b2bdde Bug 1133748 - Fix ipdl code for bridged protocols that are also opened. r=billm.
--HG--
extra : rebase_source : 308d8e3ac3caea541b649329d5a35ecd31725de9
2015-02-09 21:54:12 +01:00
Botond Ballo
19a974637b Bug 1135591 - Fix build breakage from bug 1132153 in code conditional on '--enable-ipdl-tests'. r=dholbert 2015-02-23 11:42:16 -05:00
Botond Ballo
847508c681 Bug 1132153 - Replace 'using namespace base' with specific using-declarations. r=dholbert 2015-02-11 15:01:26 -05:00
Nicholas Nethercote
242708cf72 Bug 1127201 (attempt 2, part 1) - Replace most NS_ABORT_IF_FALSE calls with MOZ_ASSERT. r=Waldo.
--HG--
extra : rebase_source : 488e401ff87e31a2074c4108c4df0572d9536667
2015-02-09 14:34:50 -08:00
Andrew McCreight
d3826daa16 Back out Bug 1127201 (part 2) for various problems. 2015-02-06 15:04:32 -08:00
Nicholas Nethercote
d34f0301b8 Bug 1127201 (part 2) - Convert all NS_ABORT_IF_FALSE calls to MOZ_ASSERT. r=Waldo.
--HG--
extra : rebase_source : 99182e70335d2b5ff95f8c528ae992d37294be3a
2015-02-04 20:05:36 -08:00
Bill McCloskey
c04f106018 Bug 1124933 - Include crash reason when intentionally crashing content process (r=bent) 2015-02-03 09:09:27 -08:00
Bill McCloskey
ab0788b330 Bug 1109884 - Stop using intr messages in PContent (r=bent) 2015-01-28 11:53:46 -08:00
Bill McCloskey
376b92ae54 Bug 1122956 - Convert IPDL tests to take move references (r=bent) 2015-01-20 15:42:54 -08:00
Kyle Huey
7c82e6dd0a Bug 1121673: Use move references in IPDL. r=bent 2015-01-16 11:58:52 -08:00
Ben Turner
f62303f39a Bug 1116640, fix TestUrgentHangs on windows, r=billm. Not tested on current infra so DONTBUILD 2015-01-05 10:53:36 -08:00
Nathan Froyd
de2ba33faa Bug 1114596 - don't define GENERATED_INCLUDES in ipc/ipdl/moz.build; r=gps
The common include file /ipc/chromium/chromium-config.mozbuild already
defines this variable for us, and there's no point in telling the
compiler to look twice in the same directory for includes.
2014-12-22 09:59:34 -05:00
Bill McCloskey
73c276ef13 Bug 1110938 - Add timeout for CPOWs (default to 0) (r=dvander) 2014-12-18 17:35:44 -08:00
Aaron Klotz
20dba0f8d2 Bug 1112827: Add IPDL's implicit typedefs to the set of explicit typedefs, preventing duplicates in generated code; r=bent 2014-12-18 14:38:18 -07:00
Ryan VanderMeulen
8a94c3960b Backed out changeset f0d4cb5f206c (bug 1112827) for Mulet permacrashing. 2014-12-18 15:26:05 -05:00
Aaron Klotz
a425715aeb Bug 1112827: Add IPDL's implicit typedefs to the set of explicit typedefs, preventing duplicates in generated code; r=bent 2014-12-18 12:20:32 -07:00
Nathan Froyd
3f89e62115 Bug 1096487 - part 2 - make ipdl message constructors take their routing id; r=bent
Similarly to part, there's no need to say we don't know the routing id
when constructing a message, and then turn around and set the routing id
immediately afterwards.  Just let the constructor do its work.
2014-11-10 13:05:59 -05:00
Nathan Froyd
357e2a0e8c Bug 1096487 - part 1 - make ipdl message constructors set their priority directly; r=bent
The default constructor for ipdl-generated messages looks like:

    MOZ_IMPLICIT Msg_AsyncMessage() :
        IPC::Message(MSG_ROUTING_NONE, ID, IPC::Message::PRIORITY_NORMAL, COMPRESSION_NONE, "PContent::Msg_AsyncMessage")
    {
    }

IPC::Message::PRIORITY_NORMAL is 1, so any set_priority(1) calls are
just redudant.

Additionally, the priorities for messages are known at code generation
time, so we are always able to provide the right constants to the
message constructor, rather than using set_priority calls later.
2014-11-10 12:32:15 -05:00
Nathan Froyd
d676b698ca Bug 1096501 - include nsRefPtr.h instead of nsAutoPtr.h in ipdl generated files; r=bent 2014-11-10 14:18:09 -05:00
Akshendra Pratap Singh
938aa11999 Bug 1074211: Output a warning message when a IPDL constructor returns a null value; r=bent 2014-10-15 15:30:00 -07:00
Mike Hommey
47c853314f Bug 1077148 part 4 - Add and use new moz.build templates for Gecko programs and libraries. r=gps
There are, sadly, many combinations of linkage in use throughout the tree.
The main differentiator, though, is between program/libraries related to
Gecko or not. Kind of. Some need mozglue, some don't. Some need dependent
linkage, some standalone.

Anyways, these new templates remove the need to manually define the
right dependencies against xpcomglue, nspr, mozalloc and mozglue
in most cases.

Places that build programs and were resetting MOZ_GLUE_PROGRAM_LDFLAGS
or that build libraries and were resetting MOZ_GLUE_LDFLAGS can now
just not use those Gecko-specific templates.
2014-10-30 13:06:12 +09:00
Bill McCloskey
43b7a1a0c8 Bug 641685 - Make sure generated code for IPDL bridged includes all necessary forward decls (r=bsmedberg) 2014-10-29 08:04:51 -07:00
Bill McCloskey
2ab547d3ab Bug 1049879 - Remove urgent and rpc message types and replace with message priorities (r=dvander,bent,ehsan) 2014-10-07 21:32:45 -07:00
Carsten "Tomcat" Book
4985d0ee76 Backed out changeset e56bf4ea89fb (bug 1049879) for bustage in emulator debug builds on a CLOSED TREE 2014-10-08 08:03:57 +02:00
Bill McCloskey
281cb8db91 Bug 1049879 - Remove urgent and rpc message types and replace with message priorities (r=dvander,bent,ehsan) 2014-10-07 21:32:45 -07:00
Mike Hommey
ed70c5f377 Bug 1041941 - Use templates for programs, simple programs, libraries and C++ unit tests. r=gps 2014-09-03 14:10:54 +09:00
David Major
36c74581b8 Bug 1023941 - Part 5: Loader hook to redirect the missing import. r=bsmedberg
--HG--
extra : rebase_source : 7cf2e857f51dcfcc95bb806d49ef13b794f5ba5b
2014-08-28 14:53:38 +12:00
Bill McCloskey
4cedc603cb Bug 1050604 - Get IPDL tests passing again (r=bent) 2014-08-20 12:49:11 -07:00
Bill McCloskey
be42be0ea4 Bug 1051128 - Allow filtering the IPC log by toplevel protocol name (r=bent) 2014-08-20 12:49:11 -07:00
Andrew McCreight
4790cd3937 Bug 1024795, part 4 - Use a default value for initializing unions, except for nsTArray. r=bent
This restores the initialization behavior prior to bug 819791, except for the one case that bug requires.
2014-08-13 11:36:15 -07:00