nsSliderFrame::GetContentOf hides inherited method

RESOLVED FIXED in Future

Status

()

Core
Layout
P3
normal
RESOLVED FIXED
18 years ago
15 years ago

People

(Reporter: Brade, Assigned: Marc Loiselle)

Tracking

Trunk
Future
PowerPC
Mac System 8.5
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [good first bug])

Attachments

(1 attachment, 2 obsolete attachments)

(Reporter)

Description

18 years ago
on the Macintosh tinderbox Warning page, I see these warnings which should be 
investigated:

nsSliderFrame::GetContentOf(nsIBox *, nsIContent **)' hides inherited virtual 
function 'nsBoxFrame::GetContentOf(nsIContent **)
evaughan
Assignee: pinkerton → evaughan

Comment 2

18 years ago
targeting
Status: NEW → ASSIGNED
Target Milestone: --- → M20

Comment 3

18 years ago
moving all evaughan non-nsbeta3+ bugs to Future milestone, per trudelle
Target Milestone: M20 → Future

Updated

16 years ago
Blocks: 179028
Just renaming the method would make the warning go away (and make things
clearer, imo).
Whiteboard: [good first bug]
(Assignee)

Comment 5

15 years ago
Created attachment 124149 [details] [diff] [review]
Removes "GetContentOf is hidden" warning

Rename nsSliderFrame::GetContentOf to nsSliderFrame::GetContentOfScrollbar. 
The method is private is only used by nsSliderFrame.cpp.
(Assignee)

Updated

15 years ago
Attachment #124149 - Flags: superreview?(bryner)
Attachment #124149 - Flags: review?(dbaron)
Comment on attachment 124149 [details] [diff] [review]
Removes "GetContentOf is hidden" warning

I'd prefer "GetContentForBox" since that's what the function does.
Attachment #124149 - Flags: superreview?(bryner) → superreview-
(Assignee)

Comment 7

15 years ago
Created attachment 124245 [details] [diff] [review]
Removes "GetContentOf is hidden" warning v2

GetContentOfBox() instead of GetContentOf() to fix compiler warning
Attachment #124149 - Attachment is obsolete: true
(Assignee)

Updated

15 years ago
Attachment #124245 - Flags: superreview?(bryner)
Attachment #124245 - Flags: review?(dbaron)
(Assignee)

Updated

15 years ago
Attachment #124149 - Flags: review?(dbaron)
Comment on attachment 124245 [details] [diff] [review]
Removes "GetContentOf is hidden" warning v2

I don't see any reason for this to be a (non-static) member function -- it just
means you're passing around an extra |this| pointer that you don't need. 
However, why should it be a member at all?  How about:

static already_AddRefed<nsIContent>
ContentOfBox(nsIBox *aBox)
{
  nsIFrame *frame;
  aBox->GetFrame(&frame);
  nsIContent *content;
  frame->GetContent(&content);
  return content;
}
(Assignee)

Comment 9

15 years ago
Created attachment 124313 [details] [diff] [review]
Removes "GetContentOf is hidden" warning v3

Add static GetContentOfBox() function.	Remove nsSliderFrame::GetContentOf().
Attachment #124245 - Attachment is obsolete: true
(Assignee)

Updated

15 years ago
Attachment #124313 - Flags: superreview?(bryner)
Attachment #124313 - Flags: review?(dbaron)
(Assignee)

Updated

15 years ago
Attachment #124245 - Flags: superreview?(bryner)
Attachment #124245 - Flags: review?(dbaron)
Attachment #124313 - Flags: review?(dbaron) → review+
Attachment #124313 - Flags: superreview?(bryner) → superreview+
(Assignee)

Comment 10

15 years ago
dbaron, Can you check-in for me?
.
Assignee: eric → mloiselle
Status: ASSIGNED → NEW
Fix checked in to trunk, 2003-06-01 19:39 -0700.
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.