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