Remove support for the Strong Confirmation Password button as we never made use of it.
<rdar://problem/41795185>
Created attachment 344233 [details] Patch
Comment on attachment 344233 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=344233&action=review > Source/WebCore/accessibility/ios/WebAccessibilityObjectWrapperIOS.mm:1096 > AutoFillButtonType type = m_object->valueAutofillButtonType(); > - return type == AutoFillButtonType::StrongConfirmationPassword || type == AutoFillButtonType::StrongPassword; > + return type == AutoFillButtonType::StrongPassword; Temporary does not seem necessary.
Comment on attachment 344233 [details] Patch Attachment 344233 [details] did not pass ios-sim-ews (ios-simulator-wk2): Output: https://webkit-queues.webkit.org/results/8430170 New failing tests: accessibility/ios-simulator/strong-password-field.html
Created attachment 344247 [details] Archive of layout-test-results from ews125 for ios-simulator-wk2 The attached test failures were seen while running run-webkit-tests on the ios-sim-ews. Bot: ews125 Port: ios-simulator-wk2 Platform: Mac OS X 10.13.4
(In reply to Sam Weinig from comment #3) > Comment on attachment 344233 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=344233&action=review > > > Source/WebCore/accessibility/ios/WebAccessibilityObjectWrapperIOS.mm:1096 > > AutoFillButtonType type = m_object->valueAutofillButtonType(); > > - return type == AutoFillButtonType::StrongConfirmationPassword || type == AutoFillButtonType::StrongPassword; > > + return type == AutoFillButtonType::StrongPassword; > > Temporary does not seem necessary. Will remove before landing.
Committed r233578: <https://trac.webkit.org/changeset/233578>
(In reply to Build Bot from comment #4) > Comment on attachment 344233 [details] > Patch > > Attachment 344233 [details] did not pass ios-sim-ews (ios-simulator-wk2): > Output: https://webkit-queues.webkit.org/results/8430170 > > New failing tests: > accessibility/ios-simulator/strong-password-field.html Now that this change has landed, this test is failing on the bots: https://build.webkit.org/results/Apple%20iOS%2011%20Simulator%20Debug%20WK2%20(Tests)/r233581%20(5253)/results.html
(In reply to Ryan Haddad from comment #8) > (In reply to Build Bot from comment #4) > > Comment on attachment 344233 [details] > > Patch > > > > Attachment 344233 [details] did not pass ios-sim-ews (ios-simulator-wk2): > > Output: https://webkit-queues.webkit.org/results/8430170 > > > > New failing tests: > > accessibility/ios-simulator/strong-password-field.html > Now that this change has landed, this test is failing on the bots: > https://build.webkit.org/results/ > Apple%20iOS%2011%20Simulator%20Debug%20WK2%20(Tests)/r233581%20(5253)/ > results.html I rebaslined the test in https://trac.webkit.org/r233595