Closed Bug 129909 Opened 18 years ago Closed 18 years ago

Rename nsGfxTextControlFrame2.cpp

Categories

(Core :: Layout: Form Controls, defect)

Product:
Core
Component:
Layout: Form Controls
Platform:
x86
All
Type:
defect
Priority:
Not set

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: john, Assigned: john)

Details

(Whiteboard: [FIX])

Attachments

(1 file)

Patch
275.55 KB, patch
rods
: review+
bzbarsky
: superreview+
asa
: approval+
Details | Diff | Splinter Review 
Remove nsGfxTextControlFrame.cpp and rename nsGfxTextControlFrame2.cpp to
nsGfxTextControlFrame.cpp.  No need for this darn 2 stuff now that there aren't
2 of them.

Heck, I don't know what the Gfx is there for, but maybe we can remove that while
we're at it.
QA Contact: madhur → tpreston
Status: NEW → ASSIGNED
Attached patch Patch — — Splinter Review 
This patch renames nsIGfxTextControlFrame.h -> nsITextControlFrame.h and
nsGfxTextControlFrame2.h/cpp -> nsTextControlFrame.h/cpp.
Summary: remove nsGfxTextControlFrame.cpp → Rename nsGfxTextControlFrame2.cpp
Whiteboard: [FIX]
Comment on attachment 92053 [details] [diff] [review]
Patch

Fix a couple of lines that have this:
+//==== NSIGFXTEXTCONTROLFRAME2
Attachment #92053 - Flags: review+
oops, fix those lines and r=rods
Comment on attachment 92053 [details] [diff] [review]
Patch

sr=bzbarsky, but please have magic happen in the repository rather than losing
the change history for those files, ok?
Attachment #92053 - Flags: superreview+
Comment on attachment 92053 [details] [diff] [review]
Patch

a=asa (on behalf of drivers) for checkin to 1.1
Attachment #92053 - Flags: approval+
Checked in.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.