137129 - GetPageX() and GetPageY() shouldn't use nsIScrollableView COMPtrs

Status: VERIFIED DUPLICATE of bug 166804

(Core :: DOM: Events, defect, P3)

Description

[kinmoz]

I've been seeing some pages that generate lots of warnings like this in the 
console:

WARNING: not supported for views, file y:\mozilla\view\src\nsScrollPortView.cpp,
 line 113

when moving the mouse around over the content area. http://my.netscape.com is an 
example of a page that can cause this warning to show up.

It seems to be caused by the fact that the nsDOMEvent.cpp implementations of 
GetPageX() and GetPageY() use nsCOMPtr<nsIScrollableView>. Views in the view 
system are not refcounted at all so you'll notice throughout the product people 
are using raw pointers (like nsIScrollableView*) when dealing with views and 
just not releasing them.

Should GetPageX() and GetPageY() follow suit?

Comment 1

[Vladimir Ermakov]

No idea how important this is, can someone please take a look?

Comment 2

[Boris Zbarsky [:bzbarsky]]

fixed for a while now...

*** This bug has been marked as a duplicate of 166804 ***

Comment 3

[Vladimir Ermakov]

verifying