Use NS_LITERAL_STRING instead of an nsAutoString to avoid copy/inflate on platforms that support two-byte wchar_t. r=attinasi
This commit is contained in:
@@ -3826,13 +3826,10 @@ PRBool IsSimpleXlink(nsIContent *aContent, nsString &aHREF)
|
|||||||
// first see if we have an XML element
|
// first see if we have an XML element
|
||||||
nsCOMPtr<nsIXMLContent> xml(do_QueryInterface(aContent));
|
nsCOMPtr<nsIXMLContent> xml(do_QueryInterface(aContent));
|
||||||
if (xml) {
|
if (xml) {
|
||||||
nsAutoString strSimple;
|
|
||||||
strSimple.AssignWithConversion("simple");
|
|
||||||
|
|
||||||
// see if it is type=simple (we don't deal with other types)
|
// see if it is type=simple (we don't deal with other types)
|
||||||
nsAutoString val;
|
nsAutoString val;
|
||||||
aContent->GetAttribute(kNameSpaceID_XLink, nsHTMLAtoms::type, val);
|
aContent->GetAttribute(kNameSpaceID_XLink, nsHTMLAtoms::type, val);
|
||||||
if (val == strSimple) {
|
if (val == NS_LITERAL_STRING("simple")) {
|
||||||
// see if there is an xlink namespace'd href attribute:
|
// see if there is an xlink namespace'd href attribute:
|
||||||
// - get it if there is, if not no big deal, it is not required for xlinks
|
// - get it if there is, if not no big deal, it is not required for xlinks
|
||||||
aContent->GetAttribute(kNameSpaceID_XLink, nsHTMLAtoms::href, aHREF);
|
aContent->GetAttribute(kNameSpaceID_XLink, nsHTMLAtoms::href, aHREF);
|
||||||
|
|||||||
@@ -3826,13 +3826,10 @@ PRBool IsSimpleXlink(nsIContent *aContent, nsString &aHREF)
|
|||||||
// first see if we have an XML element
|
// first see if we have an XML element
|
||||||
nsCOMPtr<nsIXMLContent> xml(do_QueryInterface(aContent));
|
nsCOMPtr<nsIXMLContent> xml(do_QueryInterface(aContent));
|
||||||
if (xml) {
|
if (xml) {
|
||||||
nsAutoString strSimple;
|
|
||||||
strSimple.AssignWithConversion("simple");
|
|
||||||
|
|
||||||
// see if it is type=simple (we don't deal with other types)
|
// see if it is type=simple (we don't deal with other types)
|
||||||
nsAutoString val;
|
nsAutoString val;
|
||||||
aContent->GetAttribute(kNameSpaceID_XLink, nsHTMLAtoms::type, val);
|
aContent->GetAttribute(kNameSpaceID_XLink, nsHTMLAtoms::type, val);
|
||||||
if (val == strSimple) {
|
if (val == NS_LITERAL_STRING("simple")) {
|
||||||
// see if there is an xlink namespace'd href attribute:
|
// see if there is an xlink namespace'd href attribute:
|
||||||
// - get it if there is, if not no big deal, it is not required for xlinks
|
// - get it if there is, if not no big deal, it is not required for xlinks
|
||||||
aContent->GetAttribute(kNameSpaceID_XLink, nsHTMLAtoms::href, aHREF);
|
aContent->GetAttribute(kNameSpaceID_XLink, nsHTMLAtoms::href, aHREF);
|
||||||
|
|||||||
@@ -3826,13 +3826,10 @@ PRBool IsSimpleXlink(nsIContent *aContent, nsString &aHREF)
|
|||||||
// first see if we have an XML element
|
// first see if we have an XML element
|
||||||
nsCOMPtr<nsIXMLContent> xml(do_QueryInterface(aContent));
|
nsCOMPtr<nsIXMLContent> xml(do_QueryInterface(aContent));
|
||||||
if (xml) {
|
if (xml) {
|
||||||
nsAutoString strSimple;
|
|
||||||
strSimple.AssignWithConversion("simple");
|
|
||||||
|
|
||||||
// see if it is type=simple (we don't deal with other types)
|
// see if it is type=simple (we don't deal with other types)
|
||||||
nsAutoString val;
|
nsAutoString val;
|
||||||
aContent->GetAttribute(kNameSpaceID_XLink, nsHTMLAtoms::type, val);
|
aContent->GetAttribute(kNameSpaceID_XLink, nsHTMLAtoms::type, val);
|
||||||
if (val == strSimple) {
|
if (val == NS_LITERAL_STRING("simple")) {
|
||||||
// see if there is an xlink namespace'd href attribute:
|
// see if there is an xlink namespace'd href attribute:
|
||||||
// - get it if there is, if not no big deal, it is not required for xlinks
|
// - get it if there is, if not no big deal, it is not required for xlinks
|
||||||
aContent->GetAttribute(kNameSpaceID_XLink, nsHTMLAtoms::href, aHREF);
|
aContent->GetAttribute(kNameSpaceID_XLink, nsHTMLAtoms::href, aHREF);
|
||||||
|
|||||||
Reference in New Issue
Block a user