2011-06-25 Joseph Pecoraro <joepeck@webkit.org>
authorjoepeck@webkit.org <joepeck@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 26 Jun 2011 04:04:07 +0000 (04:04 +0000)
committerjoepeck@webkit.org <joepeck@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 26 Jun 2011 04:04:07 +0000 (04:04 +0000)
        Reviewed by Darin Adler.

        Fix some minor style issues
        https://bugs.webkit.org/show_bug.cgi?id=63391

        * inspector/DOMNodeHighlighter.cpp:
        (WebCore::DOMNodeHighlighter::DrawNodeHighlight):
        The explicit WebCore namespace usage can be dropped.

        * rendering/RenderView.cpp:
        (WebCore::RenderView::mapLocalToContainer):
        Use ASSERT_UNUSED for a parameter that we want to ASSERT
        on for a debug build, but is UNUSED in a release build.

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

Source/WebCore/ChangeLog
Source/WebCore/inspector/DOMNodeHighlighter.cpp
Source/WebCore/rendering/RenderView.cpp

index bd87095..4d9f47e 100644 (file)
@@ -1,5 +1,21 @@
 2011-06-25  Joseph Pecoraro  <joepeck@webkit.org>
 
+        Reviewed by Darin Adler.
+
+        Fix some minor style issues
+        https://bugs.webkit.org/show_bug.cgi?id=63391
+
+        * inspector/DOMNodeHighlighter.cpp:
+        (WebCore::DOMNodeHighlighter::DrawNodeHighlight):
+        The explicit WebCore namespace usage can be dropped.
+
+        * rendering/RenderView.cpp:
+        (WebCore::RenderView::mapLocalToContainer):
+        Use ASSERT_UNUSED for a parameter that we want to ASSERT
+        on for a debug build, but is UNUSED in a release build.
+
+2011-06-25  Joseph Pecoraro  <joepeck@webkit.org>
+
         Unreviewed build fix.
 
         Use UNUSED_PARAM on the correct param!
index 3d4d4d2..25f9654 100644 (file)
@@ -132,7 +132,7 @@ inline IntSize frameToMainFrameOffset(Frame* frame)
     return mainFramePoint - IntPoint();
 }
 
-void drawElementTitle(GraphicsContext& context, Node* node, const IntRect& boundingBox, const IntRect& anchorBox, const FloatRect& overlayRect, WebCore::Settings* settings)
+void drawElementTitle(GraphicsContext& context, Node* node, const IntRect& boundingBox, const IntRect& anchorBox, const FloatRect& overlayRect, Settings* settings)
 {
     static const int rectInflatePx = 4;
     static const int fontHeightPx = 12;
@@ -297,7 +297,7 @@ void DrawNodeHighlight(GraphicsContext& context, Node* node, HighlightMode mode)
     if (!node->isElementNode())
         return;
 
-    WebCore::Settings* settings = containingFrame->settings();
+    Settings* settings = containingFrame->settings();
     if (mode == DOMNodeHighlighter::HighlightAll)
         drawElementTitle(context, node, boundingBox, titleAnchorBox, overlayRect, settings);
 }
index 999ebfa..08bd94d 100644 (file)
@@ -141,8 +141,7 @@ void RenderView::mapLocalToContainer(RenderBoxModelObject* repaintContainer, boo
     // If a container was specified, and was not 0 or the RenderView,
     // then we should have found it by now.
     ASSERT_ARG(repaintContainer, !repaintContainer || repaintContainer == this);
-    ASSERT(!wasFixed || *wasFixed == fixed);
-    UNUSED_PARAM(wasFixed);
+    ASSERT_UNUSED(wasFixed, !wasFixed || *wasFixed == fixed);
 
     if (!repaintContainer && useTransforms && shouldUseTransformFromContainer(0)) {
         TransformationMatrix t;