Bug 1488808 Part 1 - Fix return value sets in Preamble_Veto, r=froydnj.
authorBrian Hackett <bhackett1024@gmail.com>
Wed, 17 Oct 2018 09:58:50 -0600
changeset 497779 42143e96a1ea13f9b5adaf0ae4e9b8f3c9cdb200
parent 497778 40a4e0406d3daf01b3e2d37767cd7adce5e11396
child 497780 33cf80c7214823e5e38307a666add7a0ef116e17
push id10002
push userarchaeopteryx@coole-files.de
push dateFri, 19 Oct 2018 23:09:29 +0000
treeherdermozilla-beta@01378c910610 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersfroydnj
bugs1488808
milestone64.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1488808 Part 1 - Fix return value sets in Preamble_Veto, r=froydnj.
toolkit/recordreplay/ProcessRedirect.h
--- a/toolkit/recordreplay/ProcessRedirect.h
+++ b/toolkit/recordreplay/ProcessRedirect.h
@@ -515,28 +515,28 @@ RR_OversizeRval(Stream& aEvents, CallArg
 {
   RR_WriteBufferFixedSize<0, ByteCount>(aEvents, aArguments, aError);
 }
 
 template <size_t ReturnValue>
 static inline PreambleResult
 Preamble_Veto(CallArguments* aArguments)
 {
-  aArguments->Rval<size_t>() = 0;
+  aArguments->Rval<size_t>() = ReturnValue;
   return PreambleResult::Veto;
 }
 
 template <size_t ReturnValue>
 static inline PreambleResult
 Preamble_VetoIfNotPassedThrough(CallArguments* aArguments)
 {
   if (AreThreadEventsPassedThrough()) {
     return PreambleResult::PassThrough;
   }
-  aArguments->Rval<size_t>() = 0;
+  aArguments->Rval<size_t>() = ReturnValue;
   return PreambleResult::Veto;
 }
 
 static inline PreambleResult
 Preamble_PassThrough(CallArguments* aArguments)
 {
   return PreambleResult::PassThrough;
 }