Have HTMLElements / SVGElements constructors take a Document reference in argument

Source/core/html/HTMLMetaElement.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2003, 2010 Apple Inc. All rights reserved.
  *
  * 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.
@@ skipping 12 matching lines @@
 #include "config.h"
 #include "core/html/HTMLMetaElement.h"
 
 #include "HTMLNames.h"
 #include "core/dom/Document.h"
 
 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));
 }
 
 void HTMLMetaElement::parseAttribute(const QualifiedName& name, const AtomicString& value)
 {
     if (name == http_equivAttr)
         process();
     else if (name == contentAttr)
         process();
@@ skipping 45 matching lines @@
 {
     return getAttribute(http_equivAttr);
 }
 
 String HTMLMetaElement::name() const
 {
     return getNameAttribute();
 }
 
 }