renun/NetNewsWire
1
0
Fork 0
mirror of https://github.com/Ranchero-Software/NetNewsWire synced 2025-08-12 06:26:36 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge pull request #1368 from Wevah/favicon-master-fix

Browse Source 
Favicon master fix
This commit is contained in:
Maurice Parker
2019-11-27 16:32:44 -06:00
committed by GitHub
parent 24634dfaaa 64af8840fe
commit aebb33420e
2 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
Shared/Favicons/FaviconDownloader.swift
View File

@@ -136,6 +136,7 @@ final class FaviconDownloader {
if (!hasIcons) {
self.homePageURLsWithNoFaviconURLCache.insert(url)
self.homePageURLsWithNoFaviconURLCacheDirty = true
}
}
@@ -169,6 +170,7 @@ final class FaviconDownloader {
if let url = singleFaviconDownloader.homePageURL {
if self.homePageToFaviconURLCache[url] == nil {
self.homePageToFaviconURLCache[url] = singleFaviconDownloader.faviconURL
self.homePageToFaviconURLCacheDirty = true
}
}

3
Shared/Favicons/SingleFaviconDownloader.swift
View File

@@ -85,8 +85,9 @@ private extension SingleFaviconDownloader {
if let image = image {
self.iconImage = IconImage(image)
self.postDidLoadFaviconNotification()
}
self.postDidLoadFaviconNotification()
}
}