Changeset 29543 in webkit

Timestamp:

Jan 16, 2008, 3:36:44 PM (18 years ago)

Author:

sullivan@apple.com

Message:

Reviewed by Adam and Dan

• cleaned up some existing logging

• WebView/WebHTMLView.mm: (-[WebHTMLView setNeedsDisplay:]): add method name to log, use "YES" and "NO" instead of (int)flag (-[WebHTMLView setNeedsLayout:]): ditto (-[WebHTMLView setNeedsToApplyStyles:]): ditto

Location:

trunk/WebKit/mac

Files:

• ChangeLog (modified) (1 diff)
• WebView/WebHTMLView.mm (modified) (1 diff)

trunk/WebKit/mac/ChangeLog

@@ +1 @@
+2008-01-16  John Sullivan  <sullivan@apple.com>
+
+        Reviewed by Adam and Dan
+        
+        - cleaned up some existing logging
+
+        * WebView/WebHTMLView.mm:
+        (-[WebHTMLView setNeedsDisplay:]):
+        add method name to log, use "YES" and "NO" instead of (int)flag
+        (-[WebHTMLView setNeedsLayout:]):
+        ditto
+        (-[WebHTMLView setNeedsToApplyStyles:]):
+        ditto
+
 2008-01-15  Geoffrey Garen  <ggaren@apple.com>
 

trunk/WebKit/mac/WebView/WebHTMLView.mm

@@ -2734 +2734 @@
 - (void)setNeedsDisplay:(BOOL)flag
 {
-    LOG(View, "%@ flag = %d", self, (int)flag);
-    [super setNeedsDisplay: flag];
+    LOG(View, "%@ setNeedsDisplay:%@", self, flag ? @"YES" : @"NO");
+    [super setNeedsDisplay:flag];
 }
 
 - (void)setNeedsLayout: (BOOL)flag
 {
-    LOG(View, "%@ flag = %d", self, (int)flag);
+    LOG(View, "%@ setNeedsLayout:%@", self, flag ? @"YES" : @"NO");
     _private->needsLayout = flag;
 }
 
-
 - (void)setNeedsToApplyStyles: (BOOL)flag
 {
-    LOG(View, "%@ flag = %d", self, (int)flag);
+    LOG(View, "%@ setNeedsToApplyStyles:%@", self, flag ? @"YES" : @"NO");
     _private->needsToApplyStyles = flag;
 }