[GTK][WPE] Remove useless conditional in IconDatabase::setIconDataForIconURL
authorcarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 25 Jul 2017 06:45:53 +0000 (06:45 +0000)
committercarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 25 Jul 2017 06:45:53 +0000 (06:45 +0000)
https://bugs.webkit.org/show_bug.cgi?id=174792

Patch by Michael Catanzaro <mcatanzaro@igalia.com> on 2017-07-24
Reviewed by Carlos Garcia Campos.

We have in IconDatabase::setIconDataForIconURL some code in a conditional to check if it's
not being executed on the icon sync thread. But at the top of the function is an assertion
to ensure the code is never executed on the icon sync thread. The condition is therefore
useless and the code should be executed unconditionally.

* UIProcess/API/glib/IconDatabase.cpp:
(WebKit::IconDatabase::setIconDataForIconURL):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@219862 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit/ChangeLog
Source/WebKit/UIProcess/API/glib/IconDatabase.cpp

index e61c8d5..487b236 100644 (file)
@@ -1,3 +1,18 @@
+2017-07-24  Michael Catanzaro  <mcatanzaro@igalia.com>
+
+        [GTK][WPE] Remove useless conditional in IconDatabase::setIconDataForIconURL
+        https://bugs.webkit.org/show_bug.cgi?id=174792
+
+        Reviewed by Carlos Garcia Campos.
+
+        We have in IconDatabase::setIconDataForIconURL some code in a conditional to check if it's
+        not being executed on the icon sync thread. But at the top of the function is an assertion
+        to ensure the code is never executed on the icon sync thread. The condition is therefore
+        useless and the code should be executed unconditionally.
+
+        * UIProcess/API/glib/IconDatabase.cpp:
+        (WebKit::IconDatabase::setIconDataForIconURL):
+
 2017-07-24  Carlos Garcia Campos  <cgarcia@igalia.com>
 
         [GTK] Icon database error and crash
index 44b1914..26aaa1a 100644 (file)
@@ -581,16 +581,12 @@ void IconDatabase::setIconDataForIconURL(RefPtr<SharedBuffer>&& data, const Stri
     }
 
     // Send notification out regarding all PageURLs that retain this icon
-    // But not if we're on the sync thread because that implies this mapping
-    // comes from the initial import which we don't want notifications for
-    if (!IS_ICON_SYNC_THREAD()) {
-        // Start the timer to commit this change - or further delay the timer if it was already started
-        scheduleOrDeferSyncTimer();
+    // Start the timer to commit this change - or further delay the timer if it was already started
+    scheduleOrDeferSyncTimer();
 
-        for (auto& pageURL : pageURLs) {
-            LOG(IconDatabase, "Dispatching notification that retaining pageURL %s has a new icon", urlForLogging(pageURL).ascii().data());
-            m_client->didChangeIconForPageURL(pageURL);
-        }
+    for (auto& pageURL : pageURLs) {
+        LOG(IconDatabase, "Dispatching notification that retaining pageURL %s has a new icon", urlForLogging(pageURL).ascii().data());
+        m_client->didChangeIconForPageURL(pageURL);
     }
 }