corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1970656 - Enable VerifyStream in early beta builds, a=dmeehan

Browse Source 
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
This commit is contained in:
Kershaw Chang
2025-06-11 16:53:16 +00:00
committed by dmeehan@mozilla.com
parent 59def6b8b7
commit c176345db2
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
netwerk/protocol/http/Http2Session.cpp
View File

@@ -1210,10 +1210,10 @@ bool Http2Session::VerifyStream(Http2StreamBase* aStream,
// This is annoying, but at least it is O(1) // This is annoying, but at least it is O(1)
MOZ_ASSERT(OnSocketThread(), "not on socket thread"); MOZ_ASSERT(OnSocketThread(), "not on socket thread");
#ifndef DEBUG #ifndef MOZ_DIAGNOSTIC_ASSERT_ENABLED
// Only do the real verification in debug builds // Only do the real verification in early beta builds
return true; return true;
#else // DEBUG #else // MOZ_DIAGNOSTIC_ASSERT_ENABLED
if (!aStream) return true; if (!aStream) return true;