[DevTools] Do not report edited resources via Page.getResourceContent.

Source/devtools/front_end/sdk/DebuggerModel.js

 /*
  * Copyright (C) 2010 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 107 matching lines @@
      */
     debuggerEnabled: function()
     {
         return !!this._debuggerEnabled;
     },
 
     enableDebugger: function()
     {
         if (this._debuggerEnabled)
             return;
-        this._agent.enable();
-        this._debuggerEnabled = true;
+        this._agent.enable(this._wasEnabled.bind(this));
         if (this._hasStaleState) {
             this._globalObjectCleared();
             this._hasStaleState = false;
         }
         this._pauseOnExceptionStateChanged();

[yurys, 2015/07/16 16:30:35]

I'd expect all of these code to be executed after debugger was enabled. Now we
will send  DebuggerWasEnabled after modifying exception state which is weird I
suspect it may well break something.

[dgozman, 2015/07/17 14:39:16]

Done.

         this.asyncStackTracesStateChanged();
+    },
+
+    /**
+     * @param {?Protocol.Error} error
+     */
+    _wasEnabled: function(error)
+    {
+        if (error)
+            console.error(error);
+        this._debuggerEnabled = true;
         this.dispatchEventToListeners(WebInspector.DebuggerModel.Events.DebuggerWasEnabled);
     },
 
     disableDebugger: function()
     {
         if (!this._debuggerEnabled)
             return;
 
         this._hasStaleState = true;
         this._agent.disable();
@@ skipping 1308 matching lines @@
 /**
  * @param {?WebInspector.Target} target
  * @return {?WebInspector.DebuggerModel}
  */
 WebInspector.DebuggerModel.fromTarget = function(target)
 {
     if (!target || !target.hasJSContext())
         return null;
     return /** @type {?WebInspector.DebuggerModel} */ (target.model(WebInspector.DebuggerModel));
 }