1077393 - Rename ScalarValueString to USVString

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Anne (:annevk)]

heycam picked a different name from me when he uplifted my proposal into IDL. I'm assuming this is it now.

Comment 1

[Akshendra Pratap Singh]

This bug is all about changing the name from ScalarValueString to USVString at all places.

Comment 2

[Boris Zbarsky [:bzbarsky]]

What information are you looking for?

Comment 3

[Akshendra Pratap Singh]

Sorry I messed up the needinfo, I asked about that in comment 1.
I wanted to know is this bug all about, renaming the occurances of ScalarValueString to USVString?

Comment 4

[Boris Zbarsky [:bzbarsky]]

Yes, exactly.  Comment 1 wasn't a question.... ;)

Comment 5

[Andrea Marchesini [:baku]]

Attachment: svs.patch

https://tbpl.mozilla.org/?tree=Try&rev=2ecadd751073

Just a bit more than: %s/ScalarValueString/USVString/g

Comment 6

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8526054 [details] [diff] [review]
svs.patch

>diff --git a/dom/bindings/test/TestBindingHeader.h b/dom/bindings/test/TestBindingHeader.h
>--- a/dom/bindings/test/TestBindingHeader.h
>+++ b/dom/bindings/test/TestBindingHeader.h
>@@ -475,17 +475,17 @@ public:
> 
>   // ByteString types
>   void PassByteString(const nsCString&);
>   void PassNullableByteString(const nsCString&);
>   void PassOptionalByteString(const Optional<nsCString>&);
>   void PassOptionalNullableByteString(const Optional<nsCString>&);
>   void PassVariadicByteString(const Sequence<nsCString>&);
> 
>-  // ScalarValueString types
>+  // USVString types
>   void PassSVS(const nsAString&);
s/SVS/USV/
tai
s/SVS/USVS/


>-  void PassUnionWithSVS(const ScalarValueStringOrLong&);
>+  void PassUnionWithSVS(const USVStringOrLong&);
ditto

>-  // ScalarValueString types
>-  void passSVS(ScalarValueString arg);
>-  void passNullableSVS(ScalarValueString? arg);
>-  void passOptionalSVS(optional ScalarValueString arg);
>-  void passOptionalSVSWithDefaultValue(optional ScalarValueString arg = "abc");
>-  void passOptionalNullableSVS(optional ScalarValueString? arg);
>-  void passOptionalNullableSVSWithDefaultValue(optional ScalarValueString? arg = null);
>-  void passVariadicSVS(ScalarValueString... arg);
>-  ScalarValueString receiveSVS();
>+  // USVString types
>+  void passSVS(USVString arg);
>+  void passNullableSVS(USVString? arg);
>+  void passOptionalSVS(optional USVString arg);
>+  void passOptionalSVSWithDefaultValue(optional USVString arg = "abc");
>+  void passOptionalNullableSVS(optional USVString? arg);
>+  void passOptionalNullableSVSWithDefaultValue(optional USVString? arg = null);
>+  void passVariadicSVS(USVString... arg);
>+  USVString receiveSVS();
And here


>@@ -573,17 +573,17 @@ interface TestInterface {
>   void passUnion26((sequence<sequence<ImageData?>> or long) arg);
>   void passUnion27(optional (sequence<DOMString> or EventInit) arg);
>   void passUnion28(optional (EventInit or sequence<DOMString>) arg);
>   void passUnionWithCallback((EventHandler or long) arg);
>   void passUnionWithByteString((ByteString or long) arg);
>   void passUnionWithMozMap((MozMap<DOMString> or DOMString) arg);
>   void passUnionWithMozMapAndSequence((MozMap<DOMString> or sequence<DOMString>) arg);
>   void passUnionWithSequenceAndMozMap((sequence<DOMString> or MozMap<DOMString>) arg);
>-  void passUnionWithSVS((ScalarValueString or long) arg);
>+  void passUnionWithSVS((USVString or long) arg);
and here


>-  // ScalarValueString types
>-  void passSVS(ScalarValueString arg);
>-  void passNullableSVS(ScalarValueString? arg);
>-  void passOptionalSVS(optional ScalarValueString arg);
>-  void passOptionalSVSWithDefaultValue(optional ScalarValueString arg = "abc");
>-  void passOptionalNullableSVS(optional ScalarValueString? arg);
>-  void passOptionalNullableSVSWithDefaultValue(optional ScalarValueString? arg = null);
>-  void passVariadicSVS(ScalarValueString... arg);
>-  ScalarValueString receiveSVS();
>+  // USVString types
>+  void passSVS(USVString arg);
>+  void passNullableSVS(USVString? arg);
>+  void passOptionalSVS(optional USVString arg);
>+  void passOptionalSVSWithDefaultValue(optional USVString arg = "abc");
>+  void passOptionalNullableSVS(optional USVString? arg);
>+  void passOptionalNullableSVSWithDefaultValue(optional USVString? arg = null);
>+  void passVariadicSVS(USVString... arg);
>+  USVString receiveSVS();
and here

>   void passOptionalNullableEnumWithDefaultValue(optional TestEnum? arg = null);
>@@ -438,17 +438,17 @@ interface TestExampleInterface {
>   void passUnion26((sequence<sequence<ImageData?>> or long) arg);
>   void passUnion27(optional (sequence<DOMString> or EventInit) arg);
>   void passUnion28(optional (EventInit or sequence<DOMString>) arg);
>   void passUnionWithCallback((EventHandler or long) arg);
>   void passUnionWithByteString((ByteString or long) arg);
>   void passUnionWithMozMap((MozMap<DOMString> or DOMString) arg);
>   void passUnionWithMozMapAndSequence((MozMap<DOMString> or sequence<DOMString>) arg);
>   void passUnionWithSequenceAndMozMap((sequence<DOMString> or MozMap<DOMString>) arg);
>-  void passUnionWithSVS((ScalarValueString or long) arg);
>+  void passUnionWithSVS((USVString or long) arg);
ditto

>-  // ScalarValueString types
>-  void passSVS(ScalarValueString arg);
>-  void passNullableSVS(ScalarValueString? arg);
>-  void passOptionalSVS(optional ScalarValueString arg);
>-  void passOptionalSVSWithDefaultValue(optional ScalarValueString arg = "abc");
>-  void passOptionalNullableSVS(optional ScalarValueString? arg);
>-  void passOptionalNullableSVSWithDefaultValue(optional ScalarValueString? arg = null);
>-  void passVariadicSVS(ScalarValueString... arg);
>-  ScalarValueString receiveSVS();
>+  // USVString types
>+  void passSVS(USVString arg);
>+  void passNullableSVS(USVString? arg);
>+  void passOptionalSVS(optional USVString arg);
>+  void passOptionalSVSWithDefaultValue(optional USVString arg = "abc");
>+  void passOptionalNullableSVS(optional USVString? arg);
>+  void passOptionalNullableSVSWithDefaultValue(optional USVString? arg = null);
>+  void passVariadicSVS(USVString... arg);
>+  USVString receiveSVS();
ditto

>@@ -451,17 +451,17 @@ interface TestJSImplInterface {
>   void passUnion26((sequence<sequence<ImageData?>> or long) arg);
>   void passUnion27(optional (sequence<DOMString> or EventInit) arg);
>   void passUnion28(optional (EventInit or sequence<DOMString>) arg);
>   void passUnionWithCallback((EventHandler or long) arg);
>   void passUnionWithByteString((ByteString or long) arg);
>   void passUnionWithMozMap((MozMap<DOMString> or DOMString) arg);
>   void passUnionWithMozMapAndSequence((MozMap<DOMString> or sequence<DOMString>) arg);
>   void passUnionWithSequenceAndMozMap((sequence<DOMString> or MozMap<DOMString>) arg);
>-  void passUnionWithSVS((ScalarValueString or long) arg);
>+  void passUnionWithSVS((USVString or long) arg);
ditto



(I assume s/SVS/USVS/gsm for the relevant files would do the trick)

Comment 7

[Andrea Marchesini [:baku]]

Attachment: svs.patch

Comment 8

[Carsten Book [:Tomcat]]

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

Comment 9

[Carsten Book [:Tomcat]]

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

Comment 10

[Chris Mills [:cmills]]

USVString documented here - could perhaps do with a quick tech review

https://developer.mozilla.org/en-US/docs/Web/API/USVString