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

RESOLVED WORKSFORME

Status

()

Core
Layout: View Rendering
P4
minor
RESOLVED WORKSFORME
17 years ago
6 years ago

People

(Reporter: jesup, Unassigned)

Tracking

Trunk
Future
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(3 attachments)

(Reporter)

Description

17 years ago
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.
(Reporter)

Comment 1

17 years ago
Created attachment 14528 [details]
backtrace
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.
Status: NEW → ASSIGNED
Target Milestone: --- → Future

Updated

16 years ago
Attachment #14528 - Attachment is patch: false
Build moving all existing future-P3 bugs to future-P4.
Priority: P3 → P4

Comment 4

16 years ago
Created attachment 71813 [details]
Changed text of warning message to identify function called in error
Created attachment 92308 [details] [diff] [review]
fix scroll warning by removing refcounting of view in nsDOMEvent.cpp
Blocks: 128201
Assignee: kmcclusk → nobody
Status: ASSIGNED → NEW
QA Contact: chrispetersen → layout.view-rendering
We don't have nsScrollPortView's anymore and the code in nsDOMEvent.cpp isn't there anymore.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.