Fix devtools bugs Fix a spurious layout test failure

Source/devtools/front_end/sdk/RuntimeModel.js

 /*
  * Copyright (C) 2012 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:
  *
  *     * 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
@@ skipping 257 matching lines @@
      * @param {boolean} includeCommandLineAPI
      * @param {boolean} doNotPauseOnExceptionsAndMuteConsole
      * @param {boolean} returnByValue
      * @param {boolean} generatePreview
      * @param {function(?WebInspector.RemoteObject, boolean, ?RuntimeAgent.RemoteObject=, ?DebuggerAgent.ExceptionDetails=)} callback
      */
     evaluate: function(expression, objectGroup, includeCommandLineAPI, doNotPauseOnExceptionsAndMuteConsole, returnByValue, generatePreview, callback)
     {
         // Dart adds the isEvaluateOnCallFrameMode option to make debugging multiple Dart libraries more usable.
         // FIXME: It will be moved to separate ExecutionContext.
-        if (this.debuggerModel.selectedCallFrame() && this._debuggerModel.isEvaluateOnCallFrameMode()) {
+        if (this.debuggerModel.selectedCallFrame() && this.debuggerModel.isEvaluateOnCallFrameMode()) {
             this.debuggerModel.evaluateOnSelectedCallFrame(expression, objectGroup, includeCommandLineAPI, doNotPauseOnExceptionsAndMuteConsole, returnByValue, generatePreview, callback);
             return;
         }
         this._evaluateGlobal.apply(this, arguments);
     },
 
     /**
      * @param {string} objectGroup
      * @param {boolean} returnByValue
      * @param {boolean} generatePreview
@@ skipping 41 matching lines @@
         }
         this.target().runtimeAgent().evaluate(expression, objectGroup, includeCommandLineAPI, doNotPauseOnExceptionsAndMuteConsole, this.id, returnByValue, generatePreview, evalCallback.bind(this));
     },
 
     /**
      * @param {string} expression
      * @param {function(?Protocol.Error, !Array.<!string>)} callback
      */
     getCompletions: function(expression, callback)
     {
-        var selectedCallFrame = WebInspector.debuggerModel.selectedCallFrame();
+        var selectedCallFrame = this.debuggerModel.selectedCallFrame();
 
         // Dart/Modules specific tweak: don't use the call frame if we are
         // we have explicitly selected an execution context for the console
         // that is part of a different library than the current call frame.
 
-        if (selectedCallFrame && this._debuggerModel.isEvaluateOnCallFrameMode()) {
+        if (selectedCallFrame && this.debuggerModel.isEvaluateOnCallFrameMode()) {
             selectedCallFrame.getCompletions(expression, callback);
             return;
         }
 
         this.target().runtimeAgent().getCompletions(expression, this.id, callback);
     },
 
     /**
      * @param {string} expressionString
      * @param {string} prefix
      * @param {boolean} force
      * @param {function(!Array.<string>, number=)} completionsReadyCallback
      */
     completionsForExpression: function(expressionString, prefix, force, completionsReadyCallback)
     {
-        if ((this._debuggerModel.selectedCallFrame() && this._debuggerModel.selectedCallFrame().script.language == "Dart") ||
-            (!this._debuggerModel.selectedCallFrame() && this.language == "Dart")) {
+        if ((this.debuggerModel.selectedCallFrame() && this.debuggerModel.selectedCallFrame().script.language == "Dart") ||
+            (!this.debuggerModel.selectedCallFrame() && this.language == "Dart")) {
             return this.completionsForExpressionDart(expressionString, prefix, force, completionsReadyCallback);
         }
 
         var lastIndex = expressionString.length - 1;
 
         var dotNotation = (expressionString[lastIndex] === ".");
         var bracketNotation = (expressionString[lastIndex] === "[");
 
         if (dotNotation || bracketNotation)
             expressionString = expressionString.substr(0, lastIndex);
@@ skipping 164 matching lines @@
         if (bracketNotation) {
             if (prefix.length && prefix[0] === "'")
                 var quoteUsed = "'";
             else
                 var quoteUsed = "\"";
         }
 
         var results = [];
 
         if (!expressionString) {
-            var selectedCallFrame = WebInspector.debuggerModel.selectedCallFrame();
+            var selectedCallFrame = this.debuggerModel.selectedCallFrame();
             var dartContext = false;
 
             if (selectedCallFrame) {
                 dartContext = selectedCallFrame.script && selectedCallFrame.script.language == "Dart";
             } else {
                 dartContext = this.language == "Dart";
             }
 
             if (dartContext) {
                 // Note we omit get, set, final, class, import, and library as those
@@ skipping 100 matching lines @@
     /**
      * @return {!RuntimeAgent.RemoteObjectId}
      */
      objectId: function()
      {
         return this._objectId;
      },
 
     __proto__: WebInspector.SDKObject.prototype
 }