Bug 864727 part 6. Make all the WrapNode methods take a handle for the scope object. r=ms2ger

Author: bzbarsky

content/base/public/nsINode.h

@@ -387,7 +387,7 @@ class nsINode : public mozilla::dom::EventTarget
    * does some additional checks and fix-up that's common to all nodes. WrapNode
    * should just call the DOM binding's Wrap function.
    */
-  virtual JSObject* WrapNode(JSContext *aCx, JSObject *aScope)
+  virtual JSObject* WrapNode(JSContext *aCx, JS::Handle<JSObject*> aScope)
   {
     MOZ_ASSERT(!IsDOMBinding(), "Someone forgot to override WrapNode");
     return nullptr;

content/base/src/Comment.cpp

@@ -61,7 +61,7 @@ Comment::List(FILE* out, int32_t aIndent) const
 #endif
 
 JSObject*
-Comment::WrapNode(JSContext *aCx, JSObject *aScope)
+Comment::WrapNode(JSContext *aCx, JS::Handle<JSObject*> aScope)
 {
   return CommentBinding::Wrap(aCx, aScope, this);
 }

content/base/src/Comment.h

@@ -67,7 +67,8 @@ class Comment : public nsGenericDOMDataNode,
 #endif
 
 protected:
-  virtual JSObject* WrapNode(JSContext *aCx, JSObject *aScope) MOZ_OVERRIDE;
+  virtual JSObject* WrapNode(JSContext *aCx,
+                             JS::Handle<JSObject*> aScope) MOZ_OVERRIDE;
 };
 
 } // namespace dom

content/base/src/DocumentFragment.cpp

@@ -22,7 +22,7 @@ namespace mozilla {
 namespace dom {
 
 JSObject*
-DocumentFragment::WrapNode(JSContext *aCx, JSObject *aScope)
+DocumentFragment::WrapNode(JSContext *aCx, JS::Handle<JSObject*> aScope)
 {
   return DocumentFragmentBinding::Wrap(aCx, aScope, this);
 }

content/base/src/DocumentFragment.h

@@ -70,7 +70,8 @@ class DocumentFragment : public FragmentOrElement,
   {
   }
 
-  virtual JSObject* WrapNode(JSContext *aCx, JSObject *aScope) MOZ_OVERRIDE;
+  virtual JSObject* WrapNode(JSContext *aCx,
+                             JS::Handle<JSObject*> aScope) MOZ_OVERRIDE;
 
   // nsIContent
   virtual already_AddRefed<nsINodeInfo>

content/base/src/DocumentType.cpp

@@ -60,7 +60,7 @@ namespace mozilla {
 namespace dom {
 
 JSObject*
-DocumentType::WrapNode(JSContext *cx, JSObject *scope)
+DocumentType::WrapNode(JSContext *cx, JS::Handle<JSObject*> scope)
 {
   return DocumentTypeBinding::Wrap(cx, scope, this);
 }

content/base/src/DocumentType.h

@@ -76,7 +76,8 @@ class DocumentType : public DocumentTypeForward
   virtual nsIDOMNode* AsDOMNode() { return this; }
 
 protected:
-  virtual JSObject* WrapNode(JSContext *cx, JSObject *scope) MOZ_OVERRIDE;
+  virtual JSObject* WrapNode(JSContext *cx,
+                             JS::Handle<JSObject*> scope) MOZ_OVERRIDE;
 
   nsString mPublicId;
   nsString mSystemId;

content/base/src/nsTextNode.cpp

@@ -98,7 +98,7 @@ NS_IMPL_ISUPPORTS_INHERITED3(nsTextNode, nsGenericDOMDataNode, nsIDOMNode,
                              nsIDOMText, nsIDOMCharacterData)
 
 JSObject*
-nsTextNode::WrapNode(JSContext *aCx, JSObject *aScope)
+nsTextNode::WrapNode(JSContext *aCx, JS::Handle<JSObject*> aScope)
 {
   return TextBinding::Wrap(aCx, aScope, this);
 }

content/base/src/nsTextNode.h

@@ -80,7 +80,8 @@ class nsTextNode : public mozilla::dom::Text,
 #endif
 
 protected:
-  virtual JSObject* WrapNode(JSContext *aCx, JSObject *aScope) MOZ_OVERRIDE;
+  virtual JSObject* WrapNode(JSContext *aCx,
+                             JS::Handle<JSObject*> aScope) MOZ_OVERRIDE;
 };
 
 #endif // nsTextNode_h

content/html/content/public/HTMLAudioElement.h

@@ -66,7 +66,8 @@ class HTMLAudioElement : public HTMLMediaElement,
   uint64_t MozCurrentSampleOffset(ErrorResult& aRv);
 
 protected:
-  virtual JSObject* WrapNode(JSContext* aCx, JSObject* aScope) MOZ_OVERRIDE;
+  virtual JSObject* WrapNode(JSContext* aCx,
+                             JS::Handle<JSObject*> aScope) MOZ_OVERRIDE;
 };
 
 } // namespace dom