Fix uncaught exception bug from liveEditScriptSource

test/inspector/debugger/set-script-source-exception.js

+// Copyright 2017 the V8 project authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+print('Check that setScriptSource completes correctly when an exception is thrown.');
+
+Protocol.Debugger.enable();
+Protocol.Debugger.onScriptParsed(message => {runTests(message.params.scriptId)});

[kozy, 2017/01/25 23:35:07]

Protocol.Debugger.onceScriptParsed().then(message =>
runTests(message.params.scriptId));

[luoe, 2017/01/25 23:48:54]

Done.

+InspectorTest.addScript(`function foo() {}`);

[kozy, 2017/01/25 23:35:07]

No reason to use `` for single line source, please use '.

[luoe, 2017/01/25 23:48:54]

Done.

+
+function runTests(scriptId) {

[kozy, 2017/01/25 23:35:07]

Let's rewrite it a little:

InspectorTest.runTestSuite([
  function testIncorrectScriptId(next) {
    Protocol.Debugger.setScriptSource({ scriptId: '-1', scriptSource: '0')
      .then(InspectorTest.logMessage) // it's protocol test so let's just dump
protocol output without any formatting
      .then(next);
  },

  function testSourceWithSyntaxError(next) {
    Protocol.Debugger.onceScriptParsed()
      .then(message => Protocol.Debugger.setScriptSource({ scriptId:
message.params.scriptId, scriptSource: 'a # b'})
      .then(InspectorTest.logMessage)
      .then(next);
    InspectorTest.addScript('function foo() {}'); // will issue scriptParsed
event and resolve Protocol.Debugger.onceScriptParsed promise
  }

]);

[luoe, 2017/01/25 23:48:54]

Done.

+  Protocol.Debugger.setScriptSource({scriptId: scriptId, scriptSource: 'a # b'})
+      .then(logExceptionOrError)
+      .then(() => Protocol.Debugger.setScriptSource({scriptId: '-1', scriptSource: '0'}))
+      .then(logExceptionOrError)
+      .then(InspectorTest.completeTest);
+}
+
+function logExceptionOrError(message) {
+  if (message.result && message.result.exceptionDetails) {

[kozy, 2017/01/25 23:35:07]

Since it's protocol test we prefer to just dump protocol output without any
formatting.

[luoe, 2017/01/25 23:48:54]

Done.

+    InspectorTest.log('Error with exception:');
+    InspectorTest.logMessage(message.result.exceptionDetails);
+  } else if (message.error) {
+    InspectorTest.log('Error with message:', message.error.message);
+  }
+}