234318 2021-12-14 15:26:49 -0800 Display list iteration should automatically stop if an item is null 2021-12-21 15:27:16 -0800 1 1 1 Unclassified WebKit New Bugs WebKit Nightly Build Unspecified Unspecified NEW InRadar P2 Normal --- 1 gnavamarino webkit-unassigned mmaxfield webkit-bug-importer oldest_to_newest 1824150 0 gnavamarino 2021-12-14 15:26:49 -0800 We should consider modifying DisplayList::Iterator::operator== so display list iteration automatically stops iterating if an item is null. As was previously recommended, this can be done by making DisplayList::Iterator::operator== return true when called with a null iterator and the end iterator by changing ``` bool operator==(const Iterator& other) const { return &m_displayList == &other.m_displayList && m_cursor == other.m_cursor; } ``` to ``` bool operator==(const Iterator& other) const { if (atEnd() && other.atEnd()) return true; return &m_displayList == &other.m_displayList && m_cursor == other.m_cursor; } ``` This requires updating the following two API tests: - TestWebKitAPI.DisplayListTests.InlineItemValidationFailure - TestWebKitAPI.DisplayListTests.OutOfLineItemDecodingFailure 1826138 1 webkit-bug-importer 2021-12-21 15:27:16 -0800 <rdar://problem/86786410>