Image Load Cancelation - Downgrade error log to warning and note the cause of it
authorAndrew Cox <andrew.cox@partner.samsung.com>
Thu, 5 Jun 2014 10:23:25 +0000 (11:23 +0100)
committerAdeel Kazmi <adeel.kazmi@samsung.com>
Tue, 10 Jun 2014 15:10:11 +0000 (16:10 +0100)
[problem] A rare event that will probably continue to happen until we do
some rewrites is firing as an error.
[solution] Turn error into a warning and document why it currently

Signed-off-by: Andrew Cox <andrew.cox@partner.samsung.com>
dali/integration-api/core.h
dali/internal/update/resources/resource-manager.cpp

index 780d096..1ba892d 100644 (file)
@@ -298,7 +298,7 @@ public:
 
   /**
    * Queue an event with Core.
-   * Pre-processing of events may be benificial e.g. a series of motion events could be throttled, so that only the last event is queued.
+   * Pre-processing of events may be beneficial e.g. a series of motion events could be throttled, so that only the last event is queued.
    * Multi-threading note: this method should be called from the main thread.
    * @param[in] event The new event.
    */
index 53f3e63..8e62e65 100644 (file)
@@ -857,7 +857,8 @@ void ResourceManager::LoadResponse( ResourceId id, ResourceTypeId type, Resource
   }
   else
   {
-    DALI_LOG_ERROR("ResourceManager::LoadResponse() received a stray load notification for a resource whose loading is not being tracked: (id:%u, status=%s)\n", id, loadStatus==RESOURCE_LOADING?"LOADING":loadStatus==RESOURCE_PARTIALLY_LOADED?"PARTIAL":"COMPLETE");
+    // This warning can fire if a cancelled load is forgotten here while already complete on a resource thread:
+    DALI_LOG_WARNING( "Received a notification for an untracked resource: (id:%u, status=%s)\n", id, loadStatus==RESOURCE_LOADING?"LOADING":loadStatus==RESOURCE_PARTIALLY_LOADED?"PARTIAL":"COMPLETE");
   }
 }