1409598 - Change nsIXPCScriptable::className and nsIClassInfo::{contractID,classDescription} from string to AUTF8String

Status: RESOLVED FIXED

(Core :: XPCOM, enhancement)

Description

[Nicholas Nethercote [inactive]]

This lets us replace moz_xstrdup() of string literals with AssignLiteral(),
among other improvements.

The use of nullptr is replaced with empty strings. This should be fine on the
JavaScript side because `null` and "" are both falsy.

Comment 1

[Nicholas Nethercote [inactive]]

Attachment: Change nsIXPCScriptable::className and nsIClassInfo::{contractID,classDescription} from string to AUTF8String

Comment 2

[Nicholas Nethercote [inactive]]

Attachment: Change nsIXPCScriptable::className and nsIClassInfo::{contractID,classDescription} from string to AUTF8String

This lets us replace moz_xstrdup() of string literals with AssignLiteral(),
among other improvements.

Comment 3

[Nathan Froyd [:froydnj]]

Comment on attachment 8919554 [details] [diff] [review]
Change nsIXPCScriptable::className and nsIClassInfo::{contractID,classDescription} from string to AUTF8String

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

Yay.

::: caps/nsScriptSecurityManager.cpp
@@ +160,5 @@
>  }
>  
>  // Helper class to get stuff from the ClassInfo and not waste extra time with
>  // virtual method calls for things it has already gotten
>  class ClassInfoData

This class seems like a vestige of some bygone era, where it used to do a lot more, but now it seems to just be adding unnecessary complexity.  We instantiate it in one place, AFAICT, in CanCreateWrapper, and we never use the caching the class provides, since we only call the relevant methods once.

Maybe we should remove it in a followup?

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c2d3e50e7a8e
Change nsIXPCScriptable::className and nsIClassInfo::{contractID,classDescription} from string to AUTF8String. r=froydnj.

Comment 5

[Nicholas Nethercote [inactive]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/c2d3e50e7a8ecc231b8d9c0b9ef1c436a1bb6e2e
Bug 1409598 - Change nsIXPCScriptable::className and nsIClassInfo::{contractID,classDescription} from string to AUTF8String. r=froydnj.

Comment 6

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

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

Comment 7

[Nicholas Nethercote [inactive]]

> This class seems like a vestige of some bygone era, where it used to do a
> lot more, but now it seems to just be adding unnecessary complexity.  We
> instantiate it in one place, AFAICT, in CanCreateWrapper, and we never use
> the caching the class provides, since we only call the relevant methods once.
> 
> Maybe we should remove it in a followup?

I did this in bug 1409960.