405181 - nsIDOMNSRange.isPointInRange probably shouldn't throw WRONG_DOCUMENT_ERR

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Alex Vincent [:WeirdAl]]

I don't think I was counting on isPointInRange throwing NS_ERROR_DOM_WRONG_DOCUMENT_ERR when I wrote the patch for bug 359046.  The check-in comment indicates otherwise...

Jonas, can we just eat that exception and return false?

Comment 1

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Yeah, that seems fine to me.

Comment 2

[Alex Vincent [:WeirdAl]]

Attachment: patch

Comment 3

[Reed Loden [:reed]]

Checking in content/base/src/nsRange.cpp;
/cvsroot/mozilla/content/base/src/nsRange.cpp,v  <--  nsRange.cpp
new revision: 1.225; previous revision: 1.224
done

Comment 4

[Jeff Walden [:Waldo]]

Alex, can you write a Mochitest for this?