Improve the TimedActivityLogger

- Get rid of the debug level parameter. Either we want to log the
  elapsed time or we don't. Enabling timing in verbose mode only
  is dubious anyway, as the debug output can influence the result.
- Log the time in more sensible places.

Change-Id: I1ced2a1d30cd015a9562d46079933db5ea69641c
Reviewed-by: Jake Petroules <jake.petroules@qt.io>

1 files changed, 3 insertions, 3 deletions

diff --git a/src/lib/corelib/api/internaljobs.cpp b/src/lib/corelib/api/internaljobs.cpp
index 21a2ee5df..8b0f5a5c7 100644
--- a/src/lib/corelib/api/internaljobs.cpp
+++ b/src/lib/corelib/api/internaljobs.cpp
@@ -77,8 +77,8 @@ private:
     void initialize(const QString &task, int maximum)
     {
         QBS_ASSERT(!m_timedLogger, delete m_timedLogger);
-        m_timedLogger = new TimedActivityLogger(m_job->logger(), task, QString(),
-                m_job->timed() ? LoggerInfo : LoggerDebug, m_job->timed());
+        if (m_job->timed())
+            m_timedLogger = new TimedActivityLogger(m_job->logger(), task, true);
         m_value = 0;
         m_maximum = maximum;
         m_canceled = false;
@@ -313,7 +313,7 @@ void InternalSetupProjectJob::resolveProjectFromScratch(ScriptEngine *engine)
 
 void InternalSetupProjectJob::resolveBuildDataFromScratch(const RulesEvaluationContextPtr &evalContext)
 {
-    TimedActivityLogger resolveLogger(logger(), QLatin1String("Resolving build project"));
+    TimedActivityLogger resolveLogger(logger(), QLatin1String("Resolving build project"), timed());
     BuildDataResolver(logger()).resolveBuildData(m_newProject, evalContext);
 }