Prefer a MAP that has AREAs over an empty one with the same name (in quirks mode). b=264624 r+sr=bzbarsky a=benjamin

git-svn-id: svn://10.0.0.236/trunk@176584 18797224-902f-48f8-a5cc-f745e15eee43

mozilla/content/html/document/src/nsHTMLDocument.cpp

@@ -1017,6 +1017,7 @@ nsHTMLDocument::GetImageMap(const nsAString& aMapName)
 {
   nsAutoString name;
   PRUint32 i, n = mImageMaps.Count();
+  nsIDOMHTMLMapElement *firstMatch = nsnull;
 
   for (i = 0; i < n; ++i) {
     nsIDOMHTMLMapElement *map = mImageMaps[i];
@@ -1036,11 +1037,27 @@ nsHTMLDocument::GetImageMap(const nsAString& aMapName)
     }
 
     if (match && NS_SUCCEEDED(rv)) {
+      // Quirk: if the first matching map is empty, remember it, but keep
+      // searching for a non-empty one, only use it if none was found (bug 264624).
+      if (mCompatMode == eCompatibility_NavQuirks) {
+        nsCOMPtr<nsIDOMHTMLCollection> mapAreas;
+        rv = map->GetAreas(getter_AddRefs(mapAreas));
+        if (NS_SUCCEEDED(rv) && mapAreas) {
+          PRUint32 length = 0;
+          mapAreas->GetLength(&length);
+          if (length == 0) {
+            if (!firstMatch) {
+              firstMatch = map;
+            }
+            continue;
+          }
+        }
+      }
       return map;
     }
   }
 
-  return nsnull;
+  return firstMatch;
 }
 
 nsCompatibility
@@ -3126,8 +3143,6 @@ FindNamedItems(const nsAString& aName, nsIContent *aContent,
   NS_ASSERTION(aEntry.mContentList != NAME_NOT_VALID,
                "Entry that should never have a list passed to FindNamedItems()!");
 
-  nsIAtom *tag = aContent->Tag();
-
   if (aContent->IsContentOfType(nsIContent::eTEXT)) {
     // Text nodes are not named items nor can they have children.
     return;