1262399 - remove Impl suffixes from nsSupports* implementations

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[Nathan Froyd [:froydnj]]

Having the Impl suffix isn't really necessary, and if we start creating
instances of these classes directly, it's also rather ugly.  Let's get
rid of them.

Comment 1

[Nathan Froyd [:froydnj]]

Attachment: remove Impl suffixes from nsSupports* implementations

Comment 2

[Andrew McCreight [:mccr8]]

Comment on attachment 8738505 [details] [diff] [review]
remove Impl suffixes from nsSupports* implementations

Review of attachment 8738505 [details] [diff] [review]:
-----------------------------------------------------------------

::: xpcom/ds/nsStringEnumerator.cpp
@@ +122,1 @@
>      if (!stringImpl) {

Redundant OOM checks here and below, if you want to remove that.

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/ceb2fd9beec7

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/ceb2fd9beec7