From ebacb921dac6f8fd335d917100e6b04521a3f893 Mon Sep 17 00:00:00 2001 From: Andreas Kling Date: Tue, 23 Jul 2024 09:50:48 +0200 Subject: [PATCH] LibWeb: Verify something was removed in StyleSheetList::remove_sheet() This would have saved me an hour of debugging, so putting it here for the next person. :^) --- Userland/Libraries/LibWeb/CSS/StyleSheetList.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Userland/Libraries/LibWeb/CSS/StyleSheetList.cpp b/Userland/Libraries/LibWeb/CSS/StyleSheetList.cpp index 92190f93b52..3b3779e1627 100644 --- a/Userland/Libraries/LibWeb/CSS/StyleSheetList.cpp +++ b/Userland/Libraries/LibWeb/CSS/StyleSheetList.cpp @@ -115,7 +115,8 @@ void StyleSheetList::add_sheet(CSSStyleSheet& sheet) void StyleSheetList::remove_sheet(CSSStyleSheet& sheet) { sheet.set_style_sheet_list({}, nullptr); - m_sheets.remove_first_matching([&](auto& entry) { return entry.ptr() == &sheet; }); + bool did_remove = m_sheets.remove_first_matching([&](auto& entry) { return entry.ptr() == &sheet; }); + VERIFY(did_remove); if (sheet.rules().length() == 0) { // NOTE: If the removed sheet had no rules, we don't have to invalidate anything.