Closed Bug 137129 Opened 23 years ago Closed 22 years ago

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

Categories

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

defect

Tracking

()

VERIFIED DUPLICATE of bug 166804

People

(Reporter: kinmoz, Assigned: joki)

References

()

Details

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?
No idea how important this is, can someone please take a look?
Priority: -- → P3
fixed for a while now... *** This bug has been marked as a duplicate of 166804 ***
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → DUPLICATE
verifying
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.