52341 - nsScrollPortView::AddRef() and Release() generate WARNINGs

Status: RESOLVED WORKSFORME

(Core :: Web Painting, defect, P4)

Description

[Randell Jesup [:jesup] (needinfo me)]

FreeBSD 4.1 20000910xx

nsScrollPortView::AddRef() and Release() generate WARNINGs on this page.  Either
they shouldn't be getting called (a bug somewhere else), or maybe they shouldn't
be generating warnings.  Excessive warnings for things that are actually fine
makes it hard to find real problems, of course.

Backtrace attached.  The call that started this was nsDOMEvent::GetPageX()

Minor unless this is flagging a real bug elsewhere.

Comment 1

[Randell Jesup [:jesup] (needinfo me)]

Attachment: backtrace

Comment 2

[Kevin McCluskey (gone)]

Views are not refcounted. The calling code should not try to refcount the view.

This bug has been marked "future" because the original netscape engineer working
on this is over-burdened. If you feel this is an error, that you or another
known resource will be working on this bug,or if it blocks your work in some way
-- please attach your concern to the bug for reconsideration.

Comment 3

[Kevin McCluskey (gone)]

Build moving all existing future-P3 bugs to future-P4.

Comment 4

[Mike]

Attachment: Changed text of warning message to identify function called in error

Comment 5

[Kevin McCluskey (gone)]

Attachment: fix scroll warning by removing refcounting of view in nsDOMEvent.cpp

Comment 6

[Timothy Nikkel (:tnikkel)]

We don't have nsScrollPortView's anymore and the code in nsDOMEvent.cpp isn't there anymore.