Add code comments & Improve code quality

Author: Jack251970

Plugins/Flow.Launcher.Plugin.BrowserBookmark/ChromiumBookmarkLoader.cs

@@ -132,6 +132,7 @@ private void LoadFaviconsFromDb(string dbPath, List<Bookmark> bookmarks)
         // Use a copy to avoid lock issues with the original file
         var tempDbPath = Path.Combine(_faviconCacheDir, $"tempfavicons_{Guid.NewGuid()}.db");
 
+        // Use a copy to avoid lock issues with the original file
         try
         {
             File.Copy(dbPath, tempDbPath, true);

Plugins/Flow.Launcher.Plugin.BrowserBookmark/FirefoxBookmarkLoader.cs

@@ -60,7 +60,7 @@ protected List<Bookmark> GetBookmarksFromPath(string placesPath)
             File.Copy(placesPath, tempDbPath, true);
 
             // Create the connection string and init the connection
-            string dbPath = string.Format(DbPathFormat, tempDbPath);
+            var dbPath = string.Format($"Data Source={tempDbPath};Mode=ReadOnly");
             using var dbConnection = new SqliteConnection(dbPath);
 
             // Open connection to the database file and execute the query
@@ -119,8 +119,10 @@ protected List<Bookmark> GetBookmarksFromPath(string placesPath)
 
     private void LoadFaviconsFromDb(string faviconDbPath, List<Bookmark> bookmarks)
     {
+        // Use a copy to avoid lock issues with the original file
         var tempDbPath = Path.Combine(_faviconCacheDir, $"tempfavicons_{Guid.NewGuid()}.sqlite");
 
+        // Use a copy to avoid lock issues with the original file
         try
         {
             // Use a copy to avoid lock issues with the original file