Closed Bug 540443 Opened 10 years ago Closed 10 years ago
Change one argument of ns
Content Utils::Check Same Origin from ns IDOMNode to ns INode
Allows us to remove some QIs.
Attachment #422218 - Flags: review?(bzbarsky) → review+
// This QI is dumb, but this shouldn't be a critical operation - nsCOMPtr<nsIDOMNode> domRoot = do_QueryInterface(mRoot); - nsresult rv = nsContentUtils::CheckSameOrigin(domRoot, aCurrentNode); + nsresult rv = nsContentUtils::CheckSameOrigin(mRoot, aCurrentNode); I think the comment should be removed, too.
nsContentUtils.h: @@ -402,9 +402,9 @@ public: * a js-object that implements nsIDOMNode. * Never call this function with the first node provided by script, it * must always be known to be a 'real' node! this comment is outdated as well, isn't it?
http://hg.mozilla.org/mozilla-central/rev/582494927009 http://hg.mozilla.org/mozilla-central/rev/d632fb50f68d http://hg.mozilla.org/mozilla-central/rev/2caefaaa7d77
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.