Summary: | Correct serialization error in _WKApplicationManifestIcon | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Brent Fulgham <bfulgham> | ||||||
Component: | WebKit API | Assignee: | Brent Fulgham <bfulgham> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | bfulgham, hi, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=233715 | ||||||||
Bug Depends on: | 233350 | ||||||||
Bug Blocks: | |||||||||
Attachments: |
|
Description
Brent Fulgham
2021-11-29 15:42:37 PST
To make matters worse, I was using an NSString to decode an NSURL, which obviously didn't work. I've added a test case to exercise this code. Created attachment 445427 [details]
Patch
Comment on attachment 445427 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=445427&action=review r=me > Tools/TestWebKitAPI/Tests/WebKitCocoa/ApplicationManifest.mm:379 > + EXPECT_FALSE(error); NIT: `EXPECT_NULL`? > Tools/TestWebKitAPI/Tests/WebKitCocoa/ApplicationManifest.mm:382 > + EXPECT_FALSE(error); ditto (:379) Comment on attachment 445427 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=445427&action=review >> Tools/TestWebKitAPI/Tests/WebKitCocoa/ApplicationManifest.mm:379 >> + EXPECT_FALSE(error); > > NIT: `EXPECT_NULL`? Ah! Will do. >> Tools/TestWebKitAPI/Tests/WebKitCocoa/ApplicationManifest.mm:382 >> + EXPECT_FALSE(error); > > ditto (:379) Ditto. Created attachment 445441 [details]
Patch for landing
Committed r286311 (244670@main): <https://commits.webkit.org/244670@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 445441 [details]. |