Added some "const" to code that was retrieving a pointer to the internal

buffer managed by nsString

gfx/src/nsDeviceContext.cpp

@@ -449,7 +449,7 @@ public:
 PRUint32 FontAliasKey::HashValue(void) const
 {
   PRUint32 hash = 0;
-  PRUnichar* string = mString;
+  const PRUnichar* string = mString;
   PRUnichar ch;
   while ((ch = *string++) != 0) {
     // FYI: hash = hash*37 + ch

gfx/src/nsFont.cpp

@@ -103,7 +103,8 @@ PRBool nsFont::EnumerateFamilies(nsFontFamilyEnumFunc aFunc, void* aData) const
 
   familyList.Append(kNullCh);  // put an extra null at the end
 
-  PRUnichar* start = (PRUnichar*)familyList;
+  // XXX This code is evil...
+  PRUnichar* start = (PRUnichar*)(const PRUnichar*)familyList;
   PRUnichar* end   = start;
 
   while (running && (kNullCh != *start)) {