Bug 422484: Add parens around macro if conditions. Patch by David Mandelin <dmandelin@mozilla.com> r=benjamin a1.9=dsicore

layout/svg/base/src/nsSVGGlyphFrame.cpp

@@ -1359,7 +1359,7 @@ nsSVGGlyphFrame::nsSVGAutoGlyphHelperContext::nsSVGAutoGlyphHelperContext(
   Init(aSource, aText);
 
   nsresult rv = aSource->GetCharacterPosition(mCT, aText, cp);
-  if NS_FAILED(rv) {
+  if (NS_FAILED(rv)) {
     NS_WARNING("failed to get character position data");
   }
 }

layout/xul/base/src/nsXULPopupManager.cpp

@@ -1558,7 +1558,7 @@ nsXULPopupManager::HandleKeyboardNavigation(PRUint32 aKeyCode)
       mActiveMenuBar->ChangeMenuItem(nextItem, PR_TRUE);
       return PR_TRUE;
     }
-    else if NS_DIRECTION_IS_BLOCK(theDirection) {
+    else if (NS_DIRECTION_IS_BLOCK(theDirection)) {
       // Open the menu and select its first item.
       if (currentMenu) {
         nsCOMPtr<nsIContent> content = currentMenu->GetContent();