Bug 1568175: Directly store result of 'available-locales' function in args.rval(). r=khyperia

Differential Revision: https://phabricator.services.mozilla.com/D38996

--HG--
extra : moz-landing-system : lando

js/src/builtin/intl/Collator.cpp

@@ -185,13 +185,8 @@ bool js::intl_Collator_availableLocales(JSContext* cx, unsigned argc,
   CallArgs args = CallArgsFromVp(argc, vp);
   MOZ_ASSERT(args.length() == 0);
 
-  RootedValue result(cx);
-  if (!GetAvailableLocales(cx, ucol_countAvailable, ucol_getAvailable,
-                           &result)) {
-    return false;
-  }
-  args.rval().set(result);
-  return true;
+  return GetAvailableLocales(cx, ucol_countAvailable, ucol_getAvailable,
+                             args.rval());
 }
 
 bool js::intl_availableCollations(JSContext* cx, unsigned argc, Value* vp) {

js/src/builtin/intl/DateTimeFormat.cpp

@@ -231,13 +231,8 @@ bool js::intl_DateTimeFormat_availableLocales(JSContext* cx, unsigned argc,
   CallArgs args = CallArgsFromVp(argc, vp);
   MOZ_ASSERT(args.length() == 0);
 
-  RootedValue result(cx);
-  if (!GetAvailableLocales(cx, udat_countAvailable, udat_getAvailable,
-                           &result)) {
-    return false;
-  }
-  args.rval().set(result);
-  return true;
+  return GetAvailableLocales(cx, udat_countAvailable, udat_getAvailable,
+                             args.rval());
 }
 
 static bool DefaultCalendar(JSContext* cx, const UniqueChars& locale,

js/src/builtin/intl/NumberFormat.cpp

@@ -214,13 +214,8 @@ bool js::intl_NumberFormat_availableLocales(JSContext* cx, unsigned argc,
   CallArgs args = CallArgsFromVp(argc, vp);
   MOZ_ASSERT(args.length() == 0);
 
-  RootedValue result(cx);
-  if (!GetAvailableLocales(cx, unum_countAvailable, unum_getAvailable,
-                           &result)) {
-    return false;
-  }
-  args.rval().set(result);
-  return true;
+  return GetAvailableLocales(cx, unum_countAvailable, unum_getAvailable,
+                             args.rval());
 }
 
 bool js::intl_numberingSystem(JSContext* cx, unsigned argc, Value* vp) {

js/src/builtin/intl/PluralRules.cpp

@@ -173,15 +173,10 @@ bool js::intl_PluralRules_availableLocales(JSContext* cx, unsigned argc,
   CallArgs args = CallArgsFromVp(argc, vp);
   MOZ_ASSERT(args.length() == 0);
 
-  RootedValue result(cx);
   // We're going to use ULocale availableLocales as per ICU recommendation:
   // https://ssl.icu-project.org/trac/ticket/12756
-  if (!GetAvailableLocales(cx, uloc_countAvailable, uloc_getAvailable,
-                           &result)) {
-    return false;
-  }
-  args.rval().set(result);
-  return true;
+  return GetAvailableLocales(cx, uloc_countAvailable, uloc_getAvailable,
+                             args.rval());
 }
 
 /**

js/src/builtin/intl/RelativeTimeFormat.cpp

@@ -185,15 +185,10 @@ bool js::intl_RelativeTimeFormat_availableLocales(JSContext* cx, unsigned argc,
   CallArgs args = CallArgsFromVp(argc, vp);
   MOZ_ASSERT(args.length() == 0);
 
-  RootedValue result(cx);
   // We're going to use ULocale availableLocales as per ICU recommendation:
   // https://ssl.icu-project.org/trac/ticket/12756
-  if (!GetAvailableLocales(cx, uloc_countAvailable, uloc_getAvailable,
-                           &result)) {
-    return false;
-  }
-  args.rval().set(result);
-  return true;
+  return GetAvailableLocales(cx, uloc_countAvailable, uloc_getAvailable,
+                             args.rval());
 }
 
 /**