Bug 548545 - don't try to get an atom for a NULL string. r=roc

gfx/src/thebes/nsThebesFontEnumerator.cpp

@@ -74,7 +74,14 @@ nsThebesFontEnumerator::EnumerateFonts(const char *aLangGroup,
     else
         generic.SetIsVoid(PR_TRUE);
 
-    nsCOMPtr<nsIAtom> langGroupAtom = do_GetAtom(aLangGroup);
+    nsCOMPtr<nsIAtom> langGroupAtom;
+    if (aLangGroup) {
+        nsCAutoString lowered;
+        lowered.Assign(aLangGroup);
+        ToLowerCase(lowered);
+        langGroupAtom = do_GetAtom(lowered);
+    }
+
     nsresult rv = gfxPlatform::GetPlatform()->GetFontList(langGroupAtom, generic, fontList);
 
     if (NS_FAILED(rv)) {

gfx/thebes/src/gfxFontconfigUtils.cpp

@@ -282,8 +282,10 @@ gfxFontconfigUtils::GetFontList(nsIAtom *aLangGroup,
     aListOfFonts.Clear();
 
     nsTArray<nsCString> fonts;
-    nsCString langGroupStr;
+    nsCAutoString langGroupStr;
+    if (aLangGroup) {
         aLangGroup->ToUTF8String(langGroupStr);
+    }
     nsresult rv = GetFontListInternal(fonts, langGroupStr);
     if (NS_FAILED(rv))
         return rv;

gfx/thebes/src/gfxOS2Platform.cpp

@@ -115,8 +115,10 @@ gfxOS2Platform::GetFontList(nsIAtom *aLangGroup,
                             nsTArray<nsString>& aListOfFonts)
 {
 #ifdef DEBUG_thebes
-    const char *langgroup;
+    const char *langgroup = "(null)";
+    if (aLangGroup) {
         aLangGroup->GetUTF8String(&langgroup);
+    }
     char *family = ToNewCString(aGenericFamily);
     printf("gfxOS2Platform::GetFontList(%s, %s, ..)\n",
            langgroup, family);