[DevTools] Do not expose agents on Target

third_party/WebKit/Source/devtools/front_end/sdk/Script.js

 /*
  * Copyright (C) 2008 Apple 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.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 12 matching lines @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 /**
  * @implements {Common.ContentProvider}
  * @unrestricted
  */
 SDK.Script = class {
   /**
    * @param {!SDK.DebuggerModel} debuggerModel
+   * @param {!Protocol.DebuggerAgent} debuggerAgent
    * @param {string} scriptId
    * @param {string} sourceURL
    * @param {number} startLine
    * @param {number} startColumn
    * @param {number} endLine
    * @param {number} endColumn
    * @param {!Protocol.Runtime.ExecutionContextId} executionContextId
    * @param {string} hash
    * @param {boolean} isContentScript
    * @param {boolean} isLiveEdit
    * @param {string|undefined} sourceMapURL
    * @param {boolean} hasSourceURL
    * @param {number} length
    */
   constructor(
-      debuggerModel, scriptId, sourceURL, startLine, startColumn, endLine, endColumn, executionContextId, hash,
-      isContentScript, isLiveEdit, sourceMapURL, hasSourceURL, length) {
+      debuggerModel, debuggerAgent, scriptId, sourceURL, startLine, startColumn, endLine, endColumn, executionContextId,
+      hash, isContentScript, isLiveEdit, sourceMapURL, hasSourceURL, length) {
     this.debuggerModel = debuggerModel;
+    this._debuggerAgent = debuggerAgent;
     this.scriptId = scriptId;
     this.sourceURL = sourceURL;
     this.lineOffset = startLine;
     this.columnOffset = startColumn;
     this.endLine = endLine;
     this.endColumn = endColumn;
 
     this.executionContextId = executionContextId;
     this.hash = hash;
     this._isContentScript = isContentScript;
@@ skipping 67 matching lines @@
    * @return {!Promise<?string>}
    */
   requestContent() {
     if (this._source)
       return Promise.resolve(this._source);
     if (!this.scriptId)
       return Promise.resolve(/** @type {?string} */ (''));
 
     var callback;
     var promise = new Promise(fulfill => callback = fulfill);
-    this.debuggerModel.target().debuggerAgent().getScriptSource(this.scriptId, didGetScriptSource.bind(this));
+    this._debuggerAgent.getScriptSource(this.scriptId, didGetScriptSource.bind(this));
     return promise;
 
     /**
      * @this {SDK.Script}
      * @param {?Protocol.Error} error
      * @param {string} source
      */
     function didGetScriptSource(error, source) {
       this._source = error ? '' : SDK.Script._trimSourceURLComment(source);
       if (this._originalSource === null)
@@ skipping 36 matching lines @@
       for (var i = 0; i < searchMatches.length; ++i) {
         var searchMatch =
             new Common.ContentProvider.SearchMatch(searchMatches[i].lineNumber, searchMatches[i].lineContent);
         result.push(searchMatch);
       }
       callback(result || []);
     }
 
     if (this.scriptId) {
       // Script failed to parse.
-      this.debuggerModel.target().debuggerAgent().searchInContent(
-          this.scriptId, query, caseSensitive, isRegex, innerCallback);
+      this._debuggerAgent.searchInContent(this.scriptId, query, caseSensitive, isRegex, innerCallback);
     } else {
       callback([]);
     }
   }
 
   /**
    * @param {string} source
    * @return {string}
    */
   _appendSourceURLCommentIfNeeded(source) {
@@ skipping 21 matching lines @@
       var needsStepIn = !!stackChanged;
       callback(error, exceptionDetails, callFrames, asyncStackTrace, needsStepIn);
     }
 
     newSource = SDK.Script._trimSourceURLComment(newSource);
     // We append correct sourceURL to script for consistency only. It's not actually needed for things to work correctly.
     newSource = this._appendSourceURLCommentIfNeeded(newSource);
 
     if (this.scriptId) {
       this.requestContent().then(
-          () => this.debuggerModel.target().debuggerAgent().setScriptSource(
-              this.scriptId, newSource, undefined, didEditScriptSource.bind(this)));
+          () =>
+              this._debuggerAgent.setScriptSource(this.scriptId, newSource, undefined, didEditScriptSource.bind(this)));
     } else {
       callback('Script failed to parse');
     }
   }
 
   /**
    * @param {number} lineNumber
    * @param {number=} columnNumber
    * @return {!SDK.DebuggerModel.Location}
    */
@@ skipping 29 matching lines @@
    */
   setBlackboxedRanges(positions) {
     return new Promise(setBlackboxedRanges.bind(this));
 
     /**
      * @param {function(?)} fulfill
      * @param {function(*)} reject
      * @this {SDK.Script}
      */
     function setBlackboxedRanges(fulfill, reject) {
-      this.debuggerModel.target().debuggerAgent().setBlackboxedRanges(this.scriptId, positions, callback);
+      this._debuggerAgent.setBlackboxedRanges(this.scriptId, positions, callback);
       /**
        * @param {?Protocol.Error} error
        */
       function callback(error) {
         if (error)
           console.error(error);
         fulfill(!error);
       }
     }
   }
 };
 
 SDK.Script.sourceURLRegex = /^[\040\t]*\/\/[@#] sourceURL=\s*(\S*?)\s*$/m;