Bug 1308871 - removed dead code from nsXMLContentSerializer::SerializeAttr. r?baku draft
authorAndi-Bogdan Postelnicu <bpostelnicu@mozilla.com>
Mon, 10 Oct 2016 13:14:42 +0300
changeset 423131 7aaa451dede7e440edb1719d2f52b6243d4def45
parent 423090 d72cf6ecebaf707c159f6697d9f5f6f6a6feb7e1
child 533370 a72769a7a3f98c8fa7e3f078cc22a9acf7afce14
push id31810
push userbmo:bpostelnicu@mozilla.com
push dateMon, 10 Oct 2016 10:15:29 +0000
reviewersbaku
bugs1308871
milestone52.0a1
Bug 1308871 - removed dead code from nsXMLContentSerializer::SerializeAttr. r?baku MozReview-Commit-ID: AxxLN41TvZc
dom/base/nsXMLContentSerializer.cpp
--- a/dom/base/nsXMLContentSerializer.cpp
+++ b/dom/base/nsXMLContentSerializer.cpp
@@ -667,20 +667,16 @@ nsXMLContentSerializer::SerializeAttr(co
           break;
         }
       } else if (*iCurr == char16_t('"')) {
         bIncludesDouble = true;
         if (bIncludesSingle) {
           break;
         }
       }
-      // if both have been found we don't need to search further
-      if (bIncludesDouble && bIncludesSingle) {
-        break;
-      }
     }
 
     // Delimiter and escaping is according to the following table
     //    bIncludesDouble     bIncludesSingle     Delimiter       Escape Double Quote
     //    FALSE               FALSE               "               FALSE
     //    FALSE               TRUE                "               FALSE
     //    TRUE                FALSE               '               FALSE
     //    TRUE                TRUE                "               TRUE