Have SVGURIReference API deal with Document references, not pointers

Source/core/svg/SVGURIReference.cpp

 /*
  * Copyright (C) 2004, 2005, 2008 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006 Rob Buis <buis@kde.org>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
@@ skipping 25 matching lines @@
     }
 
     return false;
 }
 
 bool SVGURIReference::isKnownAttribute(const QualifiedName& attrName)
 {
     return attrName.matches(XLinkNames::hrefAttr);
 }
 
-String SVGURIReference::fragmentIdentifierFromIRIString(const String& url, Document* document)
+String SVGURIReference::fragmentIdentifierFromIRIString(const String& url, const Document& document)
 {
-    ASSERT(document);
     size_t start = url.find('#');
     if (start == notFound)
         return emptyString();
 
-    KURL base = start ? KURL(document->baseURI(), url.substring(0, start)) : document->baseURI();
-    if (equalIgnoringFragmentIdentifier(base, document->url()))
+    KURL base = start ? KURL(document.baseURI(), url.substring(0, start)) : document.baseURI();
+    if (equalIgnoringFragmentIdentifier(base, document.url()))
         return url.substring(start + 1);
 
     return emptyString();
 }
 
-static inline KURL urlFromIRIStringWithFragmentIdentifier(const String& url, Document* document, String& fragmentIdentifier)
+static inline KURL urlFromIRIStringWithFragmentIdentifier(const String& url, const Document& document, String& fragmentIdentifier)
 {
-    ASSERT(document);
     size_t startOfFragmentIdentifier = url.find('#');
     if (startOfFragmentIdentifier == notFound)
         return KURL();
 
     // Exclude the '#' character when determining the fragmentIdentifier.
     fragmentIdentifier = url.substring(startOfFragmentIdentifier + 1);
     if (startOfFragmentIdentifier) {
-        KURL base(document->baseURI(), url.substring(0, startOfFragmentIdentifier));
+        KURL base(document.baseURI(), url.substring(0, startOfFragmentIdentifier));
         return KURL(base, url.substring(startOfFragmentIdentifier));
     }
 
-    return KURL(document->baseURI(), url.substring(startOfFragmentIdentifier));
+    return KURL(document.baseURI(), url.substring(startOfFragmentIdentifier));
 }
 
-Element* SVGURIReference::targetElementFromIRIString(const String& iri, Document* document, String* fragmentIdentifier, Document* externalDocument)
+Element* SVGURIReference::targetElementFromIRIString(const String& iri, const Document& document, String* fragmentIdentifier, Document* externalDocument)
 {
     // If there's no fragment identifier contained within the IRI string, we can't lookup an element.
     String id;
     KURL url = urlFromIRIStringWithFragmentIdentifier(iri, document, id);
     if (url == KURL())
         return 0;
 
     if (fragmentIdentifier)
         *fragmentIdentifier = id;
 
     if (id.isEmpty())
         return 0;
 
     if (externalDocument) {
         // Enforce that the referenced url matches the url of the document that we've loaded for it!
         ASSERT(equalIgnoringFragmentIdentifier(url, externalDocument->url()));
         return externalDocument->getElementById(id);
     }
 
     // Exit early if the referenced url is external, and we have no externalDocument given.
     if (isExternalURIReference(iri, document))
         return 0;
 
-    return document->getElementById(id);
+    return document.getElementById(id);
 }
 
 void SVGURIReference::addSupportedAttributes(HashSet<QualifiedName>& supportedAttributes)
 {
     supportedAttributes.add(XLinkNames::hrefAttr);
 }
 
 }