Bug 203771

Summary: [WebAuthn] CtapAuthenticator::tryDowngrade should distinguish requestData().options
Product: WebKit Reporter: Jiewen Tan <jiewen_tan>
Component: WebKit Misc.Assignee: Jiewen Tan <jiewen_tan>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, commit-queue, jiewen_tan, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 181943    
Attachments:
Description Flags
Patch
bfulgham: review+
Patch for landing none

Description Jiewen Tan 2019-11-01 16:26:44 PDT
CtapAuthenticator::tryDowngrade should distinguish requestData().options.
Comment 1 Radar WebKit Bug Importer 2019-11-01 16:42:16 PDT
<rdar://problem/56832618>
Comment 2 Jiewen Tan 2019-11-01 17:06:53 PDT
Created attachment 382657 [details]
Patch
Comment 3 Brent Fulgham 2019-11-01 17:28:00 PDT
Comment on attachment 382657 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=382657&action=review

> Source/WebKit/ChangeLog:14
> +        No tests given there is no way to spoof google.com in our test infrastructures.

It might be better to say, "... no way to simulate google.com in our test infrastructure."
Comment 4 Jiewen Tan 2019-11-01 17:37:51 PDT
Comment on attachment 382657 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=382657&action=review

Thanks Brent for r+ this patch. Waiting EWS before landing.

>> Source/WebKit/ChangeLog:14
>> +        No tests given there is no way to spoof google.com in our test infrastructures.
> 
> It might be better to say, "... no way to simulate google.com in our test infrastructure."

Fixed.
Comment 5 Jiewen Tan 2019-11-03 22:34:00 PST
Created attachment 382716 [details]
Patch for landing
Comment 6 WebKit Commit Bot 2019-11-03 23:32:50 PST
The commit-queue encountered the following flaky tests while processing attachment 382716 [details]:

imported/w3c/web-platform-tests/websockets/bufferedAmount-unchanged-by-sync-xhr.any.worker.html bug 202003 (author: youennf@gmail.com)
The commit-queue is continuing to process your patch.
Comment 7 WebKit Commit Bot 2019-11-03 23:33:35 PST
Comment on attachment 382716 [details]
Patch for landing

Clearing flags on attachment: 382716

Committed r251979: <https://trac.webkit.org/changeset/251979>