[Intl] create new instances when new.target is undefined

src/js/i18n.js

 // Copyright 2013 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // ECMAScript 402 API implementation.
 
 /**
  * Intl object is a single object that has some named properties,
  * all of which are constructors.
  */
@@ skipping 929 matching lines @@
   return collator;
 }
 
 
 /**
  * Constructs Intl.Collator object given optional locales and options
  * parameters.
  *
  * @constructor
  */
-%AddNamedProperty(Intl, 'Collator', function() {
-    var locales = %_Arguments(0);
-    var options = %_Arguments(1);
+function Collator() {
+  var locales = %_Arguments(0);
+  var options = %_Arguments(1);
 
-    if (!this || this === Intl) {
-      // Constructor is called as a function.
-      return new Intl.Collator(locales, options);
-    }
+  if (IS_UNDEFINED(new.target)) return new Collator(locales, options);
 
-    return initializeCollator(TO_OBJECT(this), locales, options);
-  },
-  DONT_ENUM
-);
+  return initializeCollator(this, locales, options);
+}
+
+%AddNamedProperty(Intl, 'Collator', Collator, DONT_ENUM);
 
 
 /**
  * Collator resolvedOptions method.
  */
 %AddNamedProperty(Intl.Collator.prototype, 'resolvedOptions', function() {
     if (%_IsConstructCall()) {
       throw MakeTypeError(kOrdinaryFunctionCalledAsConstructor);
     }
 
@@ skipping 209 matching lines @@
   return numberFormat;
 }
 
 
 /**
  * Constructs Intl.NumberFormat object given optional locales and options
  * parameters.
  *
  * @constructor
  */
-%AddNamedProperty(Intl, 'NumberFormat', function() {
-    var locales = %_Arguments(0);
-    var options = %_Arguments(1);
+function NumberFormat() {
+  var locales = %_Arguments(0);
+  var options = %_Arguments(1);
 
-    if (!this || this === Intl) {
-      // Constructor is called as a function.
-      return new Intl.NumberFormat(locales, options);
-    }
+  if (IS_UNDEFINED(new.target)) return new NumberFormat(locales, options);
 
-    return initializeNumberFormat(TO_OBJECT(this), locales, options);
-  },
-  DONT_ENUM
-);
+  return initializeNumberFormat(this, locales, options);
+}
+%AddNamedProperty(Intl, 'NumberFormat', NumberFormat, DONT_ENUM);
 
 
 /**
  * NumberFormat resolvedOptions method.
  */
 %AddNamedProperty(Intl.NumberFormat.prototype, 'resolvedOptions', function() {
     if (%_IsConstructCall()) {
       throw MakeTypeError(kOrdinaryFunctionCalledAsConstructor);
     }
 
@@ skipping 369 matching lines @@
   return dateFormat;
 }
 
 
 /**
  * Constructs Intl.DateTimeFormat object given optional locales and options
  * parameters.
  *
  * @constructor
  */
-%AddNamedProperty(Intl, 'DateTimeFormat', function() {
-    var locales = %_Arguments(0);
-    var options = %_Arguments(1);
+function DateTimeFormat() {
+  var locales = %_Arguments(0);
+  var options = %_Arguments(1);
 
-    if (!this || this === Intl) {
-      // Constructor is called as a function.
-      return new Intl.DateTimeFormat(locales, options);
-    }
+  if (IS_UNDEFINED(new.target)) return new DateTimeFormat(locales, options);
 
-    return initializeDateTimeFormat(TO_OBJECT(this), locales, options);
-  },
-  DONT_ENUM
-);
+  return initializeDateTimeFormat(this, locales, options);
+}
+%AddNamedProperty(Intl, 'DateTimeFormat', DateTimeFormat, DONT_ENUM);
 
 
 /**
  * DateTimeFormat resolvedOptions method.
  */
 %AddNamedProperty(Intl.DateTimeFormat.prototype, 'resolvedOptions', function() {
     if (%_IsConstructCall()) {
       throw MakeTypeError(kOrdinaryFunctionCalledAsConstructor);
     }
 
@@ skipping 496 matching lines @@
     }
 
     var locales = %_Arguments(0);
     var options = %_Arguments(1);
     return toLocaleDateTime(
         this, locales, options, 'time', 'time', 'dateformattime');
   }
 );
 
 })