Bug 251040

Summary: [ iOS Debug ] fast/forms/ios/remove-view-after-focus.html is a flaky crash under SOAuthorizationCoordinator::tryAuthorize
Product: WebKit Reporter: Bri Harris <bharris9>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: NEW ---    
Severity: Normal CC: ap, bfulgham, pascoe, webkit-bot-watchers-bugzilla, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Safari Technology Preview   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Crash Log none

Description Bri Harris 2023-01-23 14:17:28 PST 
fast/forms/ios/remove-view-after-focus.html 

is a flaky crash on iOS Debug


HISTORY: 
https://results.webkit.org/?suite=layout-tests&test=fast%2Fforms%2Fios%2Fremove-view-after-focus.html


Crash Log is attached.
Comment 1 Bri Harris 2023-01-23 14:18:26 PST 
Created attachment 464612 [details]
Crash Log
Comment 2 Radar WebKit Bug Importer 2023-01-23 14:18:55 PST 
<rdar://problem/104573436>
Comment 3 Bri Harris 2023-01-23 14:46:01 PST 
Test is flaky; However, I was able to reproduce crash on iOS 16 Debug ToT running the test as follows:

Run-webkit-test --iOS-simulator fast/forms/ios/remove-view-after-focus.html --iteration 100

Running in iterations of 100 allowed us to reproduce the crash more than once during that test run.
Comment 4 Bri Harris 2023-01-23 14:52:27 PST 
It appears to have been a flaky crash since testing began on iOS 16.
Comment 5 Bri Harris 2023-01-23 15:11:50 PST 
Pull request: https://github.com/WebKit/WebKit/pull/8995
Comment 6 EWS 2023-01-23 15:18:02 PST 
Committed 259247@main (9e94ed68b0a1): <https://commits.webkit.org/259247@main>

Reviewed commits have been landed. Closing PR #8995 and removing active labels.
Comment 7 Ryan Haddad 2023-01-23 16:00:11 PST 
Reopening since the above was a test gardening commit.
Comment 8 Ryan Haddad 2023-01-23 16:04:02 PST 
ASSERTION FAILED: !HashTranslator::equal(KeyTraits::emptyValue(), key)
/Volumes/Data/worker/Apple-iOS-16-Simulator-Debug-Build/build/WebKitBuild/Debug-iphonesimulator/usr/local/include/wtf/HashTable.h(659) : void WTF::HashTable<unsigned int, WTF::KeyValuePair<unsigned int, OpaqueJSString *>, WTF::KeyValuePairKeyExtractor<WTF::KeyValuePair<unsigned int, OpaqueJSString *>>, WTF::DefaultHash<unsigned int>, WTF::HashMap<unsigned int, OpaqueJSString *>::KeyValuePairTraits, WTF::HashTraits<unsigned int>>::checkKey(const T &) [Key = unsigned int, Value = WTF::KeyValuePair<unsigned int, OpaqueJSString *>, Extractor = WTF::KeyValuePairKeyExtractor<WTF::KeyValuePair<unsigned int, OpaqueJSString *>>, HashFunctions = WTF::DefaultHash<unsigned int>, Traits = WTF::HashMap<unsigned int, OpaqueJSString *>::KeyValuePairTraits, KeyTraits = WTF::HashTraits<unsigned int>, HashTranslator = WTF::IdentityHashTranslator<WTF::HashMap<unsigned int, OpaqueJSString *>::KeyValuePairTraits, WTF::DefaultHash<unsigned int>>, T = unsigned int]
Comment 9 Alexey Proskuryakov 2023-01-24 18:58:23 PST 
Perhaps caused by a preceding test? Seems unlikely that this code gets triggered as part of generic layout test execution.