Bug 656165 - g_free accepts NULL so we do not need NULL checking before invoking it. r=karlt

toolkit/system/gnome/nsGConfService.cpp

@@ -214,8 +214,7 @@ nsGConfService::GetAppForProtocol(const nsACString &aScheme, PRBool *aEnabled,
   }
 
   aHandler.Assign(command);
-  if (command)
-    g_free(command);
+  g_free(command);
 
   if (err) {
     g_error_free(err);

toolkit/system/gnome/nsGnomeVFSService.cpp

@@ -258,7 +258,6 @@ nsGnomeVFSService::ShowURIForInput(const nsACString &aUri)
   if (gnome_vfs_url_show_with_env(spec, NULL) == GNOME_VFS_OK)
     rv = NS_OK;
 
-  if (spec)
-    g_free(spec);
+  g_free(spec);
   return rv;
 }

widget/src/gtk2/nsDragService.cpp

@@ -1140,8 +1140,7 @@ nsDragService::TargetResetData(void)
 {
     mTargetDragDataReceived = PR_FALSE;
     // make sure to free old data if we have to
-    if (mTargetDragData)
-      g_free(mTargetDragData);
+    g_free(mTargetDragData);
     mTargetDragData = 0;
     mTargetDragDataLen = 0;
 }