Numerous debug build fixes
authorsteveblock@google.com <steveblock@google.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 27 Oct 2011 11:47:51 +0000 (11:47 +0000)
committersteveblock@google.com <steveblock@google.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 27 Oct 2011 11:47:51 +0000 (11:47 +0000)
Unreviewed build fix.

No new tests, buuld fix only.

* loader/icon/IconDatabase.cpp: urlForLogging() is used by both LOG() and LOG_ERROR()
* storage/AbstractDatabase.h: databaseDebugName() is used by both LOG() and LOG_ERROR()
* storage/DatabaseTask.cpp: m_complete is only available when !LOG_DISABLED

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

Source/WebCore/ChangeLog
Source/WebCore/loader/icon/IconDatabase.cpp
Source/WebCore/storage/AbstractDatabase.h
Source/WebCore/storage/DatabaseTask.cpp

index 521b1987cef9837f81cf60423e8764dde7b3c44c..1ff04a8a6cedd8168d9b5d3d9307784986ee27d0 100644 (file)
@@ -1,3 +1,15 @@
+2011-10-27  Steve Block  <steveblock@google.com>
+
+        Numerous debug build fixes
+
+        Unreviewed build fix.
+
+        No new tests, buuld fix only.
+
+        * loader/icon/IconDatabase.cpp: urlForLogging() is used by both LOG() and LOG_ERROR()
+        * storage/AbstractDatabase.h: databaseDebugName() is used by both LOG() and LOG_ERROR()
+        * storage/DatabaseTask.cpp: m_complete is only available when !LOG_DISABLED
+
 2011-10-27  Vsevolod Vlasov  <vsevik@chromium.org>
 
         Web Inspector: Refactor ScriptsPanel so that ConsoleView does not depend on it.
index 720e770f1dfa9dc50cd2c699443dce314c8ef579..a8507a97d089ad49401ab2cf4fd97f90f7934dbd 100644 (file)
@@ -72,7 +72,7 @@ static const int updateTimerDelay = 5;
 
 static bool checkIntegrityOnOpen = false;
 
-#if !LOG_DISABLED
+#if !LOG_DISABLED || !ERROR_DISABLED
 static String urlForLogging(const String& url)
 {
     static unsigned urlTruncationLength = 120;
index f08446f3f02e5970b270f970f015ecbdfca48736..fab3b2d44d2eb9ae7f78eb2786ba848741ce7db2 100644 (file)
@@ -36,7 +36,7 @@
 #include "SQLiteDatabase.h"
 #include <wtf/Forward.h>
 #include <wtf/ThreadSafeRefCounted.h>
-#if !LOG_DISABLED
+#if !LOG_DISABLED || !ERROR_DISABLED
 #include "SecurityOrigin.h"
 #endif
 
@@ -115,7 +115,7 @@ protected:
     unsigned long m_estimatedSize;
     String m_filename;
 
-#if !LOG_DISABLED
+#if !LOG_DISABLED || !ERROR_DISABLED
     String databaseDebugName() const { return m_contextThreadSecurityOrigin->toString() + "::" + m_name; }
 #endif
 
index 3a12380ffc5dbc8ac12c2c3cf6eb8da10ffe1585..a1c714a73c64591f9f100012c7934430138070dd 100644 (file)
@@ -62,7 +62,7 @@ void DatabaseTaskSynchronizer::taskCompleted()
 DatabaseTask::DatabaseTask(Database* database, DatabaseTaskSynchronizer* synchronizer)
     : m_database(database)
     , m_synchronizer(synchronizer)
-#ifndef NDEBUG
+#if !LOG_DISABLED
     , m_complete(false)
 #endif
 {
@@ -70,13 +70,17 @@ DatabaseTask::DatabaseTask(Database* database, DatabaseTaskSynchronizer* synchro
 
 DatabaseTask::~DatabaseTask()
 {
+#if !LOG_DISABLED
     ASSERT(m_complete || !m_synchronizer);
+#endif
 }
 
 void DatabaseTask::performTask()
 {
     // Database tasks are meant to be used only once, so make sure this one hasn't been performed before.
+#if !LOG_DISABLED
     ASSERT(!m_complete);
+#endif
 
     LOG(StorageAPI, "Performing %s %p\n", debugTaskName(), this);
 
@@ -86,7 +90,7 @@ void DatabaseTask::performTask()
     if (m_synchronizer)
         m_synchronizer->taskCompleted();
 
-#ifndef NDEBUG
+#if !LOG_DISABLED
     m_complete = true;
 #endif
 }