[DevTools] remove separate event for FailedToParseScriptSource

Everywhere on the frontend side, we use both events and process them
in the same way.

R=lushnikov@chromium.org

Bug: none
Change-Id: I3ab9b196921ff9e7e8eac86e9217b0869f0c2313
Reviewed-on: https://chromium-review.googlesource.com/1163838
Commit-Queue: Andrey Lushnikov <lushnikov@chromium.org>
Reviewed-by: Andrey Lushnikov <lushnikov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580974}

third_party/WebKit/LayoutTests/http/tests/devtools/sources/debugger/script-failed-to-parse-expected.txt

-Tests that FailedToParseScriptSource event is raised after compile script with syntax error.
+Tests that ParsedScriptSource event is raised after compile script with syntax error.
 
 
 Running: testScriptParsedEvent

third_party/WebKit/LayoutTests/http/tests/devtools/sources/debugger/script-failed-to-parse.js

@@ -4,7 +4,7 @@
 
 (async function() {
   TestRunner.addResult(
-      `Tests that FailedToParseScriptSource event is raised after compile script with syntax error.\n`);
+      `Tests that ParsedScriptSource event is raised after compile script with syntax error.\n`);
   await TestRunner.loadModule('sources_test_runner');
   await TestRunner.showPanel('sources');
   await TestRunner.evaluateInPagePromise(`
@@ -20,7 +20,7 @@
   SourcesTestRunner.runDebuggerTestSuite([
     function testScriptParsedEvent(next) {
       TestRunner.debuggerModel.addEventListener(
-          SDK.DebuggerModel.Events.FailedToParseScriptSource, scriptFailedToParse);
+          SDK.DebuggerModel.Events.ParsedScriptSource, scriptFailedToParse);
       TestRunner.evaluateInPage('addScript(\'resources/script-failed-to-parse.js\')');
 
       function scriptFailedToParse(event) {
@@ -28,7 +28,7 @@
         if (script.sourceURL.indexOf('script-failed-to-parse.js') !== -1) {
           TestRunner.addResult('Event with script-failed-to-parse.js received');
           TestRunner.debuggerModel.removeEventListener(
-              SDK.DebuggerModel.Events.FailedToParseScriptSource, scriptFailedToParse);
+              SDK.DebuggerModel.Events.ParsedScriptSource, scriptFailedToParse);
           next();
         }
       }

third_party/blink/renderer/devtools/front_end/bindings/DefaultScriptMapping.js

@@ -46,8 +46,6 @@ Bindings.DefaultScriptMapping = class {
     this._eventListeners = [
       debuggerModel.addEventListener(SDK.DebuggerModel.Events.GlobalObjectCleared, this._debuggerReset, this),
       debuggerModel.addEventListener(SDK.DebuggerModel.Events.ParsedScriptSource, this._parsedScriptSource, this),
-      debuggerModel.addEventListener(
-          SDK.DebuggerModel.Events.FailedToParseScriptSource, this._parsedScriptSource, this),
       debuggerModel.addEventListener(
           SDK.DebuggerModel.Events.DiscardedAnonymousScriptSource, this._discardedScriptSource, this)
     ];

third_party/blink/renderer/devtools/front_end/bindings/PresentationConsoleMessageHelper.js

@@ -94,9 +94,6 @@ Bindings.PresentationConsoleMessageHelper = class {
     // TODO(dgozman): setImmediate because we race with DebuggerWorkspaceBinding on ParsedScriptSource event delivery.
     debuggerModel.addEventListener(
         SDK.DebuggerModel.Events.ParsedScriptSource, event => setImmediate(this._parsedScriptSource.bind(this, event)));
-    debuggerModel.addEventListener(
-        SDK.DebuggerModel.Events.FailedToParseScriptSource,
-        event => setImmediate(this._parsedScriptSource.bind(this, event)));
     debuggerModel.addEventListener(SDK.DebuggerModel.Events.GlobalObjectCleared, this._debuggerReset, this);
 
     this._locationPool = new Bindings.LiveLocationPool();

third_party/blink/renderer/devtools/front_end/bindings/ResourceScriptMapping.js

@@ -52,8 +52,6 @@ Bindings.ResourceScriptMapping = class {
     const runtimeModel = debuggerModel.runtimeModel();
     this._eventListeners = [
       this._debuggerModel.addEventListener(SDK.DebuggerModel.Events.ParsedScriptSource, this._parsedScriptSource, this),
-      this._debuggerModel.addEventListener(
-          SDK.DebuggerModel.Events.FailedToParseScriptSource, this._parsedScriptSource, this),
       this._debuggerModel.addEventListener(
           SDK.DebuggerModel.Events.GlobalObjectCleared, this._globalObjectCleared, this),
       runtimeModel.addEventListener(

third_party/blink/renderer/devtools/front_end/sdk/DebuggerModel.js

@@ -571,10 +571,7 @@ SDK.DebuggerModel = class extends SDK.SDKModel {
         this, scriptId, sourceURL, startLine, startColumn, endLine, endColumn, executionContextId,
         this._internString(hash), isContentScript, isLiveEdit, sourceMapURL, hasSourceURLComment, length);
     this._registerScript(script);
-    if (!hasSyntaxError)
-      this.dispatchEventToListeners(SDK.DebuggerModel.Events.ParsedScriptSource, script);
-    else
-      this.dispatchEventToListeners(SDK.DebuggerModel.Events.FailedToParseScriptSource, script);
+    this.dispatchEventToListeners(SDK.DebuggerModel.Events.ParsedScriptSource, script);
 
     const sourceMapId =
         SDK.DebuggerModel._sourceMapId(script.executionContextId, script.sourceURL, script.sourceMapURL);