Have DOMImplementation take the Document argument by reference

Have DOMImplementation take the Document argument by reference

Have DOMImplementation take the Document argument by reference as its
implementation expected the Document pointer to never be null. This
makes the code look safer.

R=adamk

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=162126

[eseidel, 2013-11-15 08:53:03]

lgtm

https://codereview.chromium.org/65043011/diff/1/Source/core/dom/DOMImplementa...
File Source/core/dom/DOMImplementation.cpp (right):

https://codereview.chromium.org/65043011/diff/1/Source/core/dom/DOMImplementa...
Source/core/dom/DOMImplementation.cpp:188: return
DocumentType::create(&m_document, qualifiedName, publicId, systemId);
I don't thnik you meant &m_doc... just m_doc

[Inactive, 2013-11-15 13:16:33]

https://codereview.chromium.org/65043011/diff/1/Source/core/dom/DOMImplementa...
File Source/core/dom/DOMImplementation.cpp (right):

https://codereview.chromium.org/65043011/diff/1/Source/core/dom/DOMImplementa...
Source/core/dom/DOMImplementation.cpp:188: return
DocumentType::create(&m_document, qualifiedName, publicId, systemId);
On 2013/11/15 08:53:03, eseidel wrote:
> I don't thnik you meant &m_doc... just m_doc

For now, the DocumentType constructor takes Document* in argument, not
Document&. I cannot update it yet because this is just 1 of 3 call sites and the
2 others still use pointers.

[commit-bot: I haz the power, 2013-11-15 16:29:45]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/65043011/100001

[commit-bot: I haz the power, 2013-11-15 18:36:19]

Change committed as 162126