Title: [132368] trunk/Source/WebCore
Revision
132368
Author
commit-qu...@webkit.org
Date
2012-10-24 10:29:50 -0700 (Wed, 24 Oct 2012)

Log Message

Incorrect conditional use of LogFTP
https://bugs.webkit.org/show_bug.cgi?id=100260

Patch by Cosmin Truta <ctr...@rim.com> on 2012-10-24
Reviewed by Alexey Proskuryakov.

Use LogFTP if !LOG_DISABLED, to allow toggling ASSERTIONS_DISABLED_DEFAULT
without breaking the build.

* html/FTPDirectoryDocument.cpp:
(WebCore::FTPDirectoryDocument::FTPDirectoryDocument):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (132367 => 132368)


--- trunk/Source/WebCore/ChangeLog	2012-10-24 17:28:14 UTC (rev 132367)
+++ trunk/Source/WebCore/ChangeLog	2012-10-24 17:29:50 UTC (rev 132368)
@@ -1,3 +1,16 @@
+2012-10-24  Cosmin Truta  <ctr...@rim.com>
+
+        Incorrect conditional use of LogFTP
+        https://bugs.webkit.org/show_bug.cgi?id=100260
+
+        Reviewed by Alexey Proskuryakov.
+
+        Use LogFTP if !LOG_DISABLED, to allow toggling ASSERTIONS_DISABLED_DEFAULT
+        without breaking the build.
+
+        * html/FTPDirectoryDocument.cpp:
+        (WebCore::FTPDirectoryDocument::FTPDirectoryDocument):
+
 2012-10-23  Alexey Proskuryakov  <a...@apple.com>
 
         Add a strategy for shared workers

Modified: trunk/Source/WebCore/html/FTPDirectoryDocument.cpp (132367 => 132368)


--- trunk/Source/WebCore/html/FTPDirectoryDocument.cpp	2012-10-24 17:28:14 UTC (rev 132367)
+++ trunk/Source/WebCore/html/FTPDirectoryDocument.cpp	2012-10-24 17:29:50 UTC (rev 132368)
@@ -431,7 +431,7 @@
 FTPDirectoryDocument::FTPDirectoryDocument(Frame* frame, const KURL& url)
     : HTMLDocument(frame, url)
 {
-#ifndef NDEBUG
+#if !LOG_DISABLED
     LogFTP.state = WTFLogChannelOn;
 #endif
 }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to