rename nsIFrame::GetPresContext to PresContext

RESOLVED FIXED

Status

()

Core
General
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: (dormant account), Assigned: (dormant account))

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

11 years ago
Created attachment 260180 [details] [diff] [review]
Automatic patch produced in debug configuration on linux/osx + manual step for windows
Component: General → General
Product: Firefox → Core
QA Contact: general → general
Hardware: PC → All
Checked in.
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
taras, I was wondering whether we could fix up situations like:

  foo->GetPresShell()->Foo(barArg,
                           bazArg);

so they don't end up like:

  foo->PresShell()->Foo(barArg,
                           bazArg);

and instead up as:

  foo->PresShell()->Foo(barArg,
                        bazArg);

?
(Assignee)

Comment 3

11 years ago
Boris,
Right now I'm pushing the limits of what oink can do without producing extremely ugly source code.
I think what you want is a good pretty printer for C++, with proper indentation. That's already on the TODO list, but there are a few higher priority items before that.
Sure.  It's not urgent; I just ended up doing a lot of indent-fixing when I had to touch some presshell functions recently, so I was wondering whether we could have avoided it easily.  If not, then so be it.
You need to log in before you can comment on or make changes to this bug.