Closed
Bug 916520
Opened 11 years ago
Closed 11 years ago
Rename nsClientRect(List) to mozilla::dom::DOMRect(List)
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
mozilla27
People
(Reporter: roc, Assigned: roc)
References
Details
Attachments
(1 file, 1 obsolete file)
49.36 KB,
patch
|
khuey
:
review+
|
Details | Diff | Splinter Review |
I think we've reached a consensus to rename ClientRect to DOMRect, but I'm still waiting for that to be officially recorded. Anyway, I have a patch.
Assignee | ||
Comment 1•11 years ago
|
||
This patch also moves the implementation from content/html/content/src to content/base/src.
Attachment #804978 -
Flags: review?(Ms2ger)
Assignee | ||
Comment 2•11 years ago
|
||
Attachment #807035 -
Flags: review?(Ms2ger)
Assignee | ||
Comment 3•11 years ago
|
||
(In reply to Robert O'Callahan (:roc) (Mozilla Corporation) from comment #0) > I think we've reached a consensus to rename ClientRect to DOMRect, but I'm > still waiting for that to be officially recorded. This has happened now.
Assignee | ||
Comment 4•11 years ago
|
||
Comment on attachment 807035 [details] [diff] [review] Part 2: Make DOMRectList an ArrayClass bz already landed this.
Attachment #807035 -
Attachment is obsolete: true
Attachment #807035 -
Flags: review?(Ms2ger)
Assignee | ||
Updated•11 years ago
|
Attachment #804978 -
Flags: review?(Ms2ger) → review?(khuey)
Comment on attachment 804978 [details] [diff] [review] fix Review of attachment 804978 [details] [diff] [review]: ----------------------------------------------------------------- r=me with nits picked ::: content/base/src/DOMRect.cpp @@ +58,3 @@ > > > NS_IMETHODIMP while you're here can you remove the extra whitespace after NS_IMETHODIMP? @@ +63,5 @@ > *aLength = Length(); > return NS_OK; > } > > NS_IMETHODIMP here too. ::: content/base/src/DOMRect.h @@ +151,5 @@ > nsCOMPtr<nsISupports> mParent; > }; > > +} > +} nit: please add // namespace foo comments here. ::: dom/webidl/moz.build @@ +47,5 @@ > 'ChannelSplitterNode.webidl', > 'CharacterData.webidl', > 'ChildNode.webidl', > + 'DOMRect.webidl', > + 'DOMRectList.webidl', Please keep this list in alphabetical order.
Attachment #804978 -
Flags: review?(khuey) → review+
Assignee | ||
Comment 6•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/8b94cd262ef5
Comment 7•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/8b94cd262ef5
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•