Remove duplicate ToObject implementaion from i18n.

src/i18n.js

 // Copyright 2013 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 284 matching lines @@
  */
 function supportedLocalesOf(service, locales, options) {
   if (IS_NULL(service.match(GetServiceRE()))) {
     throw new $Error('Internal error, wrong service type: ' + service);
   }
 
   // Provide defaults if matcher was not specified.
   if (options === undefined) {
     options = {};
   } else {
-    options = toObject(options);
+    options = ToObject(options);
   }
 
   var matcher = options.localeMatcher;
   if (matcher !== undefined) {
     matcher = $String(matcher);
     if (matcher !== 'lookup' && matcher !== 'best fit') {
       throw new $RangeError('Illegal value for localeMatcher:' + matcher);
     }
   } else {
     matcher = 'best fit';
@@ skipping 213 matching lines @@
       // There is a value that's too long, or that doesn't have a key.
       return {};
     }
   }
 
   return extensionMap;
 }
 
 
 /**
- * Converts parameter to an Object if possible.
- */
-function toObject(value) {
-  if (IS_NULL_OR_UNDEFINED(value)) {
-    throw new $TypeError('Value cannot be converted to an Object.');
-  }
-
-  return $Object(value);
-}
-
-
-/**
  * Populates internalOptions object with boolean key-value pairs
  * from extensionMap and options.
  * Returns filtered extension (number and date format constructors use
  * Unicode extensions for passing parameters to ICU).
  * It's used for extension-option pairs only, e.g. kn-normalization, but not
  * for 'sensitivity' since it doesn't have extension equivalent.
  * Extensions like nu and ca don't have options equivalent, so we place
  * undefined in the map.property to denote that.
  */
 function setOptions(inOptions, extensionMap, keyValues, getOption, outOptions) {
@@ skipping 208 matching lines @@
   if (locales === undefined) {
     // Constructor is called without arguments.
     seen = [];
   } else {
     // We allow single string localeID.
     if (typeof locales === 'string') {
       seen.push(canonicalizeLanguageTag(locales));
       return freezeArray(seen);
     }
 
-    var o = toObject(locales);
+    var o = ToObject(locales);
     // Converts it to UInt32 (>>> is shr on 32bit integers).
     var len = o.length >>> 0;
 
     for (var k = 0; k < len; k++) {
       if (k in o) {
         var value = o[k];
 
         var tag = canonicalizeLanguageTag(value);
 
         if (seen.indexOf(tag) === -1) {
@@ skipping 191 matching lines @@
  */
 %SetProperty(Intl, 'Collator', function() {
     var locales = %_Arguments(0);
     var options = %_Arguments(1);
 
     if (!this || this === Intl) {
       // Constructor is called as a function.
       return new Intl.Collator(locales, options);
     }
 
-    return initializeCollator(toObject(this), locales, options);
+    return initializeCollator(ToObject(this), locales, options);
   },
   DONT_ENUM
 );
 
 
 /**
  * Collator resolvedOptions method.
  */
 %SetProperty(Intl.Collator.prototype, 'resolvedOptions', function() {
     if (%_IsConstructCall()) {
@@ skipping 206 matching lines @@
  */
 %SetProperty(Intl, 'NumberFormat', function() {
     var locales = %_Arguments(0);
     var options = %_Arguments(1);
 
     if (!this || this === Intl) {
       // Constructor is called as a function.
       return new Intl.NumberFormat(locales, options);
     }
 
-    return initializeNumberFormat(toObject(this), locales, options);
+    return initializeNumberFormat(ToObject(this), locales, options);
   },
   DONT_ENUM
 );
 
 
 /**
  * NumberFormat resolvedOptions method.
  */
 %SetProperty(Intl.NumberFormat.prototype, 'resolvedOptions', function() {
     if (%_IsConstructCall()) {
@@ skipping 232 matching lines @@
 }
 
 
 /**
  * Returns options with at least default values in it.
  */
 function toDateTimeOptions(options, required, defaults) {
   if (options === undefined) {
     options = null;
   } else {
-    options = toObject(options);
+    options = ToObject(options);
   }
 
   options = $Object.apply(this, [options]);
 
   var needsDefault = true;
   if ((required === 'date' || required === 'any') &&
       (options.weekday !== undefined || options.year !== undefined ||
        options.month !== undefined || options.day !== undefined)) {
     needsDefault = false;
   }
@@ skipping 121 matching lines @@
  */
 %SetProperty(Intl, 'DateTimeFormat', function() {
     var locales = %_Arguments(0);
     var options = %_Arguments(1);
 
     if (!this || this === Intl) {
       // Constructor is called as a function.
       return new Intl.DateTimeFormat(locales, options);
     }
 
-    return initializeDateTimeFormat(toObject(this), locales, options);
+    return initializeDateTimeFormat(ToObject(this), locales, options);
   },
   DONT_ENUM
 );
 
 
 /**
  * DateTimeFormat resolvedOptions method.
  */
 %SetProperty(Intl.DateTimeFormat.prototype, 'resolvedOptions', function() {
     if (%_IsConstructCall()) {
@@ skipping 185 matching lines @@
  */
 %SetProperty(Intl, 'v8BreakIterator', function() {
     var locales = %_Arguments(0);
     var options = %_Arguments(1);
 
     if (!this || this === Intl) {
       // Constructor is called as a function.
       return new Intl.v8BreakIterator(locales, options);
     }
 
-    return initializeBreakIterator(toObject(this), locales, options);
+    return initializeBreakIterator(ToObject(this), locales, options);
   },
   DONT_ENUM
 );
 
 
 /**
  * BreakIterator resolvedOptions method.
  */
 %SetProperty(Intl.v8BreakIterator.prototype, 'resolvedOptions', function() {
     if (%_IsConstructCall()) {
@@ skipping 307 matching lines @@
   writable: true,
   configurable: true,
   enumerable: false
 });
 %FunctionSetName($Date.prototype.toLocaleTimeString, 'toLocaleTimeString');
 %FunctionRemovePrototype($Date.prototype.toLocaleTimeString);
 %SetNativeFlag($Date.prototype.toLocaleTimeString);
 
 return Intl;
 }())});