Have HTMLElements / SVGElements constructors take a Document reference in argument

Source/core/html/HTMLMetaElement.cpp

Index: Source/core/html/HTMLMetaElement.cpp
diff --git a/Source/core/html/HTMLMetaElement.cpp b/Source/core/html/HTMLMetaElement.cpp
index 753724ba28755109845866b63b5bae48f26d022e..4218d13e9e878675a4c368ea780974f0e91dd1f9 100644
--- a/Source/core/html/HTMLMetaElement.cpp
+++ b/Source/core/html/HTMLMetaElement.cpp
@@ -30,19 +30,19 @@ namespace WebCore {
 
 using namespace HTMLNames;
 
-inline HTMLMetaElement::HTMLMetaElement(const QualifiedName& tagName, Document* document)
+inline HTMLMetaElement::HTMLMetaElement(const QualifiedName& tagName, Document& document)

[kenneth.r.christiansen, 2013/09/03 10:01:55]

This is great especially as the class uses Document& document() internally so
being able to pass in null here seems very wrong

     : HTMLElement(tagName, document)
 {
     ASSERT(hasTagName(metaTag));
     ScriptWrappable::init(this);
 }
 
-PassRefPtr<HTMLMetaElement> HTMLMetaElement::create(Document* document)
+PassRefPtr<HTMLMetaElement> HTMLMetaElement::create(Document& document)
 {
     return adoptRef(new HTMLMetaElement(metaTag, document));
 }
 
-PassRefPtr<HTMLMetaElement> HTMLMetaElement::create(const QualifiedName& tagName, Document* document)
+PassRefPtr<HTMLMetaElement> HTMLMetaElement::create(const QualifiedName& tagName, Document& document)
 {
     return adoptRef(new HTMLMetaElement(tagName, document));
 }