[Extensions API] Remove inline enums

chrome/test/data/extensions/api_test/messaging/externally_connectable/sites/assertions.js

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 (function() {
 
 // We are going to kill all of the builtins, so hold onto the ones we need.
 var defineGetter = Object.prototype.__defineGetter__;
 var defineSetter = Object.prototype.__defineSetter__;
 var Error = window.Error;
 var forEach = Array.prototype.forEach;
 var push = Array.prototype.push;
 var hasOwnProperty = Object.prototype.hasOwnProperty;
 var getOwnPropertyNames = Object.getOwnPropertyNames;
 var stringify = JSON.stringify;
 
 // Kill all of the builtins functions to give us a fairly high confidence that
 // the environment our bindings run in can't interfere with our code.
 // These are taken from the ECMAScript spec.
 var builtinTypes = [
   Object, Function, Array, String, Boolean, Number, Math, Date, RegExp, JSON,
 ];
 
 function clobber(obj, name, qualifiedName) {
   // Clobbering constructors would break everything.
-  // Clobbering toString is annoying.
+  // Clobbering toString or substr is annoying.
   // Clobbering __proto__ breaks in ways that grep can't find.
   // Clobbering function name will break because
   // SafeBuiltins does not support getters yet. See crbug.com/463526.
   // Clobbering Function.call would make it impossible to implement these tests.
   // Clobbering Object.valueOf breaks v8.
   if (name == 'constructor' ||
       name == 'toString' ||
+      name == 'substr' ||

[Devlin, 2015/04/03 00:00:56]

Without these, we hit String.substr and Object.hasOwnProperty in the bindings
when creating the enums
(https://code.google.com/p/chromium/codesearch#chromium/src/extensions/rendere...).
 This strikes me as probably wrong... but this patch is just making that code
path trigger, not causing the bindings to execute anywhere differently. 
Thoughts?

[Devlin, 2015/04/03 19:38:50]

Resolved.

+      name == 'hasOwnProperty' ||
       name == '__proto__' ||
       name == 'name' && typeof obj == 'function' ||
       qualifiedName == 'Function.call' ||
       qualifiedName == 'Object.valueOf') {
     return;
   }
   if (typeof obj[name] == 'function') {
     obj[name] = function() {
       throw new Error('Clobbered ' + qualifiedName + ' function');
     };
@@ skipping 250 matching lines @@
     if (!message)
       message = kMessage;
 
     chrome.runtime.sendMessage(extensionId, message,
         {'includeTlsChannelId': includeTlsChannelId},
         checkTlsChannelIdResponse);
   }
 };
 
 }());