Closed Bug 421662 Opened 12 years ago Closed 12 years ago
Rectangle* to Client Rect*
by mutual agreement, we're renaming TextRectangle(List) to ClientRectangle(List). This won't really affect Web apps, which don't use these interface names, but our implementation names should match the spec to minimize future confusion. The best time to rename files and change the identifier names is now.
make that ClientRect and ClientRectList.
Summary: Rename TextRectangle* to ClientRectangle* → Rename TextRectangle* to ClientRect*
If there are no constants on these interfaces I wouldn't expect their names to show up at all, so not a blocker. Though I suppose someone could muck around with the prototype in which case it would show up. Also seems like basically zero risk, so patch appreciated.
12 years ago
Comment on attachment 308542 [details] [diff] [review] trivial patch Very low risk renaming patch for maximum standards compliance goodness
Attachment #308542 - Flags: approval1.9?
Comment on attachment 308542 [details] [diff] [review] trivial patch a1.9+=damons
Attachment #308542 - Flags: approval1.9? → approval1.9+
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
It looks like the removal of files wasn't checked in? Following files are still in the tree but don't seem to be built: dom/public/idl/base/nsIDOMTextRectangle.idl dom/public/idl/base/nsIDOMTextRectangleList.idl content/html/content/src/nsTextRectangle.h content/html/content/src/nsTextRectangle.cpp
You need to log in before you can comment on or make changes to this bug.