Have TreeScrope::adoptIfNeeded() take a reference instead of a pointer

Source/core/dom/TreeScopeAdopter.h

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
  * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 15 matching lines @@
 #define TreeScopeAdopter_h
 
 #include "core/dom/Node.h"
 
 namespace WebCore {
 
 class TreeScope;
 
 class TreeScopeAdopter {
 public:
-    explicit TreeScopeAdopter(Node* toAdopt, TreeScope* newScope);
+    TreeScopeAdopter(Node& toAdopt, TreeScope& newScope);
 
     void execute() const { moveTreeToNewScope(m_toAdopt); }
     bool needsScopeChange() const { return m_oldScope != m_newScope; }
 
 #ifdef NDEBUG
     static void ensureDidMoveToNewDocumentWasCalled(Document&) { }
 #else
     static void ensureDidMoveToNewDocumentWasCalled(Document&);
 #endif
 
 private:
-    void updateTreeScope(Node*) const;
-    void moveTreeToNewScope(Node*) const;
-    void moveTreeToNewDocument(Node*, Document* oldDocument, Document* newDocument) const;
-    void moveNodeToNewDocument(Node*, Document* oldDocument, Document* newDocument) const;
+    void updateTreeScope(Node&) const;
+    void moveTreeToNewScope(Node&) const;
+    void moveTreeToNewDocument(Node&, Document& oldDocument, Document* newDocument) const;
+    void moveNodeToNewDocument(Node&, Document& oldDocument, Document* newDocument) const;
 
-    Node* m_toAdopt;
-    TreeScope* m_newScope;
-    TreeScope* m_oldScope;
+    Node& m_toAdopt;
+    TreeScope& m_newScope;
+    TreeScope& m_oldScope;
 };
 
-// FIXME: Should take |TreeScope&| instead of |TreeScope*|.
-inline TreeScopeAdopter::TreeScopeAdopter(Node* toAdopt, TreeScope* newScope)
+inline TreeScopeAdopter::TreeScopeAdopter(Node& toAdopt, TreeScope& newScope)
     : m_toAdopt(toAdopt)
     , m_newScope(newScope)
-    , m_oldScope(&toAdopt->treeScope())
+    , m_oldScope(toAdopt.treeScope())
 {
-    ASSERT(newScope);
 }
 
 }
 
 #endif