[DevTools] Removed __defineGetter__ from InjectedScriptSource.js

third_party/WebKit/Source/platform/v8_inspector/InjectedScriptSource.js

 /*
  * Copyright (C) 2007 Apple Inc.  All rights reserved.
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 1537 matching lines @@
  * @param {?JavaScriptCallFrame} callFrame
  */
 function CommandLineAPI(commandLineAPIImpl, callFrame)
 {
     /**
      * @param {string} member
      * @return {boolean}
      */
     function inScopeVariables(member)
     {
-        if (!callFrame)
-            return (member in inspectedGlobalObject);
-
         var scopeChain = callFrame.scopeChain;
         for (var i = 0; i < scopeChain.length; ++i) {
             if (member in scopeChain[i])
                 return true;
         }
         return false;
     }
 
     /**
      * @param {string} name The name of the method for which a toString method should be generated.
@@ skipping 12 matching lines @@
                 funcSyntax = funcSyntax.replace(/\bopt_(\w+)\b/g, "[$1]");
                 funcArgsSyntax = funcSyntax.trim();
             } catch (e) {
             }
             return "function " + name + "(" + funcArgsSyntax + ") { [Command Line API] }";
         };
     }
 
     for (var i = 0; i < CommandLineAPI.members_.length; ++i) {
         var member = CommandLineAPI.members_[i];
-        if (inScopeVariables(member))
+        if (callFrame && inScopeVariables(member))
             continue;
 
         this[member] = bind(commandLineAPIImpl[member], commandLineAPIImpl);
         this[member].toString = customToStringMethod(member);
     }
 
     for (var i = 0; i < 5; ++i) {
         var member = "$" + i;
-        if (inScopeVariables(member))
+        if (callFrame && inScopeVariables(member))
             continue;
 
-        this.__defineGetter__("$" + i, bind(commandLineAPIImpl._inspectedObject, commandLineAPIImpl, i));
+        this[member] = bind(commandLineAPIImpl._inspectedObject, commandLineAPIImpl, i);
     }
 
     this.$_ = injectedScript._lastResult;
 
     this.__proto__ = null;
 }
 
 // NOTE: Please keep the list of API methods below synchronized to that in WebInspector.RuntimeModel
 // and V8InjectedScriptHost!
 // NOTE: Argument names of these methods will be printed in the console, so use pretty names!
@@ skipping 278 matching lines @@
      */
     _logEvent: function(event)
     {
         inspectedGlobalObject.console.log(event.type, event);
     }
 }
 
 injectedScript._commandLineAPIImpl = new CommandLineAPIImpl();
 return injectedScript;
 })