WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
197021
Display name is empty when getting credentials with the WebKit API.
https://bugs.webkit.org/show_bug.cgi?id=197021
Summary
Display name is empty when getting credentials with the WebKit API.
Per Arne Vollan
Reported
2019-04-17 10:55:26 PDT
When getting credentials with -[WKWebsiteDataStore fetchDataRecordsOfTypes:completionHandler:], the display name is empty.
Attachments
Patch
(1.90 KB, patch)
2019-04-17 11:01 PDT
,
Per Arne Vollan
no flags
Details
Formatted Diff
Diff
Patch
(1.85 KB, patch)
2019-04-19 14:21 PDT
,
Per Arne Vollan
pvollan
: review?
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2019-04-17 10:56:58 PDT
<
rdar://problem/49983718
>
Per Arne Vollan
Comment 2
2019-04-17 11:01:35 PDT
Created
attachment 367651
[details]
Patch
Ryosuke Niwa
Comment 3
2019-04-19 12:54:28 PDT
Comment on
attachment 367651
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=367651&action=review
> Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.cpp:361 > + if (!displayName) > + continue;
It doesn't seem right to skip the entry just because we're missing the display name??
Per Arne Vollan
Comment 4
2019-04-19 13:57:04 PDT
(In reply to Ryosuke Niwa from
comment #3
)
> Comment on
attachment 367651
[details]
> Patch > > View in context: >
https://bugs.webkit.org/attachment.cgi?id=367651&action=review
> > > Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.cpp:361 > > + if (!displayName) > > + continue; > > It doesn't seem right to skip the entry just because we're missing the > display name??
That makes sense. I will update the patch. Thanks for reviewing!
Per Arne Vollan
Comment 5
2019-04-19 14:21:28 PDT
Created
attachment 367826
[details]
Patch
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug