DevTools: autoresume execution on parse errors.

webkit/glue/devtools/js/tests.js

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 
 /**
  * @fileoverview This file contains small testing framework along with the
  * test suite for the frontend. These tests are a part of the continues build
  * and are executed by the devtools_sanity_unittest.cc as a part of the
  * Interactive UI Test suite.
@@ skipping 649 matching lines @@
                 test.releaseControl();
               });
         });
   }
 
   this.takeControl();
 };
 
 
 /**
+ * Tests that inspected page doesn't hang on reload if it contains a syntax
+ * error and DevTools window is open.
+ */
+TestSuite.prototype.testAutoContinueOnSyntaxError = function() {
+  this.showPanel('scripts');
+  var test = this;
+
+  function checkScriptsList() {
+    var scriptSelect = document.getElementById('scripts-files');
+    var options = scriptSelect.options;
+    // There should be only console API source (see
+    // InjectedScript._ensureCommandLineAPIInstalled) since the page script
+    // contains a syntax error.
+    for (var i = 0 ; i < options.length; i++) {
+      if (options[i].text.search('script_syntax_error.html$') != -1) {
+        test.fail('Script with syntax error should not be in the list of ' +
+                  'parsed scripts.');
+      }
+    }
+  }
+
+  this.addSniffer(devtools.DebuggerAgent.prototype, 'handleScriptsResponse_',
+      function(msg) {
+        checkScriptsList();
+
+        // Reload inspected page.
+        test.evaluateInConsole_(
+            'window.location.reload(true);',
+            function(resultText) {
+              test.assertEquals('undefined', resultText,
+                                'Unexpected result of reload().');
+              waitForExceptionEvent();
+            });
+      });
+
+  function waitForExceptionEvent() {
+    var exceptionCount = 0;
+    test.addSniffer(
+        devtools.DebuggerAgent.prototype,
+        'handleExceptionEvent_',
+        function(msg) {
+          exceptionCount++;
+          test.assertEquals(1, exceptionCount, 'Too many exceptions.');
+          test.assertEquals(undefined, msg.getBody().script,
+                            'Unexpected exception: ' + JSON.stringify(msg));
+          test.releaseControl();
+        });
+
+    // Check that the script is not paused on parse error.
+    test.addSniffer(
+        WebInspector,
+        'pausedScript',
+        function(callFrames) {
+          test.fail('Script execution should not pause on syntax error.');
+        });
+  }
+
+  this.takeControl();
+};
+
+
+/**
  * Tests 'Pause' button will pause debugger when a snippet is evaluated.
  */
 TestSuite.prototype.testPauseInEval = function() {
   this.showPanel('scripts');
 
   var test = this;
 
   var pauseButton = document.getElementById('scripts-pause');
   pauseButton.click();
 
@@ skipping 15 matching lines @@
   this.keyIdentifier = key;
 };
 TestSuite.KeyEvent.prototype.preventDefault = function() {};
 TestSuite.KeyEvent.prototype.stopPropagation = function() {};
 
 
 /**
  * Tests console eval.
  */
 TestSuite.prototype.testConsoleEval = function() {
-  WebInspector.console.visible = true;
-  WebInspector.console.prompt.text = '123';
-  WebInspector.console.promptElement.handleKeyEvent(
-      new TestSuite.KeyEvent('Enter'));
-
   var test = this;
-  this.addSniffer(WebInspector.ConsoleView.prototype, 'addMessage',
-      function(commandResult) {
-        test.assertEquals('123', commandResult.toMessageElement().textContent);
+  this.evaluateInConsole_('123',
+      function(resultText) {
+        test.assertEquals('123', resultText);
         test.releaseControl();
       });
 
   this.takeControl();
 };
 
 
 /**
  * Tests console log.
  */
@@ skipping 100 matching lines @@
 /**
  * Run specified test on a fresh instance of the test suite.
  * @param {string} name Name of a test method from TestSuite class.
  */
 uiTests.runTest = function(name) {
   new TestSuite().runTest(name);
 };
 
 
 }