From c176345db23f33e31d136f61f380ae4b70a8552c Mon Sep 17 00:00:00 2001 From: Kershaw Chang Date: Wed, 11 Jun 2025 16:53:16 +0000 Subject: [PATCH] Bug 1970656 - Enable VerifyStream in early beta builds, a=dmeehan In case the preivous patch doesn't fix the issue, enabling VerifyStream should help us catch related crashes earlier. Original Revision: https://phabricator.services.mozilla.com/D252832 Differential Revision: https://phabricator.services.mozilla.com/D253326 --- netwerk/protocol/http/Http2Session.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/netwerk/protocol/http/Http2Session.cpp b/netwerk/protocol/http/Http2Session.cpp index 71462525d35f..8dfbc441d60a 100644 --- a/netwerk/protocol/http/Http2Session.cpp +++ b/netwerk/protocol/http/Http2Session.cpp @@ -1210,10 +1210,10 @@ bool Http2Session::VerifyStream(Http2StreamBase* aStream, // This is annoying, but at least it is O(1) MOZ_ASSERT(OnSocketThread(), "not on socket thread"); -#ifndef DEBUG - // Only do the real verification in debug builds +#ifndef MOZ_DIAGNOSTIC_ASSERT_ENABLED + // Only do the real verification in early beta builds return true; -#else // DEBUG +#else // MOZ_DIAGNOSTIC_ASSERT_ENABLED if (!aStream) return true;