Revert "[Tizen] Add some logs to check performance"
authortscholb <scholb.kim@samsung.com>
Wed, 2 Nov 2022 07:38:20 +0000 (16:38 +0900)
committertscholb <scholb.kim@samsung.com>
Wed, 2 Nov 2022 07:38:20 +0000 (16:38 +0900)
This reverts commit 84d96e087438998f24e16ffd69b24183e629298f.

dali/internal/common/core-impl.cpp
dali/internal/event/common/notification-manager.cpp
dali/internal/event/size-negotiation/relayout-controller-impl.cpp

index 9dda5a8..c37dd21 100644 (file)
@@ -365,73 +365,61 @@ void Core::UnregisterProcessor(Integration::Processor& processor, bool postProce
 
 void Core::RunProcessors()
 {
-  if(mProcessors.Count() != 0)
-  {
-    DALI_LOG_RELEASE_INFO("Start RunProcessors\n");
-
-    // Copy processor pointers to prevent changes to vector affecting loop iterator.
-    Dali::Vector<Integration::Processor*> processors(mProcessors);
+  // Copy processor pointers to prevent changes to vector affecting loop iterator.
+  Dali::Vector<Integration::Processor*> processors(mProcessors);
 
-    // To prevent accessing processor unregistered during the loop
-    mProcessorUnregistered = false;
+  // To prevent accessing processor unregistered during the loop
+  mProcessorUnregistered = false;
 
-    for(auto processor : processors)
+  for(auto processor : processors)
+  {
+    if(processor)
     {
-      if(processor)
+      if(!mProcessorUnregistered)
       {
-        if(!mProcessorUnregistered)
+        processor->Process(false);
+      }
+      else
+      {
+        // Run processor if the processor is still in the list.
+        // It may be removed during the loop.
+        auto iter = std::find(mProcessors.Begin(), mProcessors.End(), processor);
+        if(iter != mProcessors.End())
         {
           processor->Process(false);
         }
-        else
-        {
-          // Run processor if the processor is still in the list.
-          // It may be removed during the loop.
-          auto iter = std::find(mProcessors.Begin(), mProcessors.End(), processor);
-          if(iter != mProcessors.End())
-          {
-            processor->Process(false);
-          }
-        }
       }
     }
-    DALI_LOG_RELEASE_INFO("End RunProcessors\n");
   }
 }
 
 void Core::RunPostProcessors()
 {
-  if(mPostProcessors.Count() != 0)
-  {
-    DALI_LOG_RELEASE_INFO("Start RunPostProcessors\n");
-
-    // Copy processor pointers to prevent changes to vector affecting loop iterator.
-    Dali::Vector<Integration::Processor*> processors(mPostProcessors);
+  // Copy processor pointers to prevent changes to vector affecting loop iterator.
+  Dali::Vector<Integration::Processor*> processors(mPostProcessors);
 
-    // To prevent accessing processor unregistered during the loop
-    mPostProcessorUnregistered = false;
+  // To prevent accessing processor unregistered during the loop
+  mPostProcessorUnregistered = false;
 
-    for(auto processor : processors)
+  for(auto processor : processors)
+  {
+    if(processor)
     {
-      if(processor)
+      if(!mPostProcessorUnregistered)
       {
-        if(!mPostProcessorUnregistered)
+        processor->Process(true);
+      }
+      else
+      {
+        // Run processor if the processor is still in the list.
+        // It may be removed during the loop.
+        auto iter = std::find(mPostProcessors.Begin(), mPostProcessors.End(), processor);
+        if(iter != mPostProcessors.End())
         {
           processor->Process(true);
         }
-        else
-        {
-          // Run processor if the processor is still in the list.
-          // It may be removed during the loop.
-          auto iter = std::find(mPostProcessors.Begin(), mPostProcessors.End(), processor);
-          if(iter != mPostProcessors.End())
-          {
-            processor->Process(true);
-          }
-        }
       }
     }
-    DALI_LOG_RELEASE_INFO("End RunPostProcessors\n");
   }
 }
 
index c0530cc..dad673f 100644 (file)
@@ -21,7 +21,6 @@
 // INTERNAL INCLUDES
 #include <dali/devel-api/common/owner-container.h>
 #include <dali/devel-api/threading/mutex.h>
-#include <dali/integration-api/debug.h>
 #include <dali/internal/common/message.h>
 #include <dali/internal/event/common/complete-notification-interface.h>
 #include <dali/internal/event/common/property-notification-impl.h>
@@ -165,14 +164,9 @@ void NotificationManager::ProcessMessages()
 
   MessageContainer::Iterator       iter = mImpl->eventMessageQueue.Begin();
   const MessageContainer::Iterator end  = mImpl->eventMessageQueue.End();
-  if(iter != end)
+  for(; iter != end; ++iter)
   {
-    DALI_LOG_RELEASE_INFO("Start ProcessMessages\n");
-    for(; iter != end; ++iter)
-    {
-      (*iter)->Process(0u /*ignored*/);
-    }
-    DALI_LOG_RELEASE_INFO("End ProcessMessages\n");
+    (*iter)->Process(0u /*ignored*/);
   }
   // release the processed messages from event side queue
   mImpl->eventMessageQueue.Clear();
index c3deac4..497f515 100644 (file)
@@ -436,10 +436,8 @@ void RelayoutController::Relayout()
     // 2. Iterate through the stack until it's empty.
     if(mRelayoutStack->Size() > 0)
     {
-      DALI_LOG_RELEASE_INFO("Start Relayout\n");
       PRINT_HIERARCHY;
 
-      int count = 0;
       while(mRelayoutStack->Size() > 0)
       {
         Dali::Actor actor;
@@ -447,7 +445,6 @@ void RelayoutController::Relayout()
         mRelayoutStack->Get(mRelayoutStack->Size() - 1, actor, size);
         Actor& actorImpl = GetImplementation(actor);
         mRelayoutStack->PopBack();
-        count++;
 
         if(actorImpl.RelayoutRequired() && actorImpl.OnScene())
         {
@@ -463,7 +460,6 @@ void RelayoutController::Relayout()
       mRelayoutInfoAllocator.ResetMemoryPool();
 
       PRINT_HIERARCHY;
-      DALI_LOG_RELEASE_INFO("End Relayout [%d]\n", count);
     }
 
     mPerformingRelayout = false;