Bug 1451363 - part 6 - move GetIPCChannel into ProtocolState; r=mccr8 We can move this information into ProtocolState and save having two virtual functions for every protocol. Moving some bits out of the codegen'd IPC code is a nice bonus, though we keep the strange setup where toplevel protocols have two mChannel member variables.

Bug 1451363 - part 5 - remove ProtocolName virtual function; r=mccr8 ProtocolName() is only used for producing error messages and annotating crash reports. But examining actual crash reports that would have used the result of ProtocolName() indicates that we can always tell what the erroring protocol is due to the stack backtrace. So having this virtual function around just provides duplicate information, and it takes up too much space in the vtable besides. Let's get rid of it.

Bug 1451363 - part 4 - consolidate generated code into IProtocol; r=mccr8 lower.py generates repetitious: SetManager(...); Register(...); // Or RegisterID. SetIPCChannel(...); calls, which are moderately sized, given that the above call sequence requires virtual calls in several places. Instead of codegenning this sequence, let's consolidate the sequence into IProtocol and change the code generator to call into the consolidated function instead.

Bug 1451363 - part 3 - make OnChannelReceivedMessage defined in early beta only; r=mccr8 This function is only overriden in two places, both of which go away after early beta is done. We shouldn't be paying for its vtable entry after that point.

Bug 1451363 - part 2b - move protocol event target access into ProtocolState; r=mccr8 The reasoning here is the same as for the protocol register/lookup functions: these functions are all basic functionality that should not be overriden by subclasses.

Bug 1451363 - part 2a - move protocol register/lookup functions into ProtocolState; r=mccr8 This functionality is base functionality for top-level and non-toplevel protocols; nobody overrides this stuff, so it's safe to move into ProtocolState.

Bug 1451363 - part 1 - move Shmem-related IProtocol interfaces into an intermediate State class; r=mccr8 IProtocol, which is inherited by every generated IPDL protocol and every concrete protocol implementation in-tree, has a number of virtual methods that are only relevant when distinguishing between top-level protocols (IToplevelProtocol) and managed protocols (everything else). These virtual methods require pointers in every protocol's vtable, which is wasteful, and it's also somewhat confusing that many methods exist but don't really need to be overridable in any useful way. Let's clean this up, by creating a ProtocolState class to hold methods that solely differ between top-level protocols and everything else. This commit does that work and moves Shmem-related methods into this class as a proof that this can be done in a reasonable way.

Bug 1456192 - make {Read,Write}IPDLParam MOZ_NEVER_INLINE; r=froydnj MSVC's PGO inliner thinks these functions are hot enough to be inlined, but they're really not, and inlining them bloats xul.dll quite a bit.

Bug 1456192 - make Pickle::ReadSentinel MOZ_NEVER_INLINE; r=froydnj This function is just pure bloat when it gets inlined, and it will disappear on non-Nightly builds anyway. Make it MOZ_NEVER_INLINE so our size statistics on Nightly are somewhat more reflective of our size statistics on Release.

Bug 1437551 - [3.1] Move generic callback out of GeckoSession. r=snorp,jchen

Bug 1437551 - [2.5] Add GeckoRuntime telemetry API backend. r=jchen,Dexter

Bug 1437551 - [1.3] Add GeckoRuntime telemetry API. r=snorp,jchen

Bug 1448563 - Part 6: Add memory reporting for off-thread WASM. r=luke MozReview-Commit-ID: 1lXRj1JUJk2

Bug 1448563 - Part 5: Add memory reporting for Ion offthread. r=jandem MozReview-Commit-ID: KnuaP5HwNnm

Bug 1448563 - Part 4: Add memory reporting for off-thread parse. r=jandem MozReview-Commit-ID: 2qH6cwFRrfG

Bug 1448563 - Part 3b: Add memory reporting for JS helper threads. r=jandem MozReview-Commit-ID: JbOY0QRn0Wl

Bug 1448563 - Part 3a: Add iterator to js::Fifo. r=luke MozReview-Commit-ID: CoYdOBNnORg

Bug 1448563 - Part 2: Add memory reporting for Ion lazy linking. r=jandem MozReview-Commit-ID: 4tHDJXzJKY5

Bug 1448563 - Part 1: Use JS::CollectGlobalReports for non-runtime memory. r=jorendorff MozReview-Commit-ID: G4j6YKP0vEV

Bug 1436048 followup, fix for Windows build bustage. r=emilio(irc) on a CLOSED TREE