Expect should still fail for non-asynchronous tests.

chrome/test/data/webui/assertions.js

Index: chrome/test/data/webui/assertions.js
diff --git a/chrome/test/data/webui/assertions.js b/chrome/test/data/webui/assertions.js
index f3e13f467772407c8366439bc15260a779b077e4..f8d7f19f88b192991ece2a62dac2c00b6068c95e 100644
--- a/chrome/test/data/webui/assertions.js
+++ b/chrome/test/data/webui/assertions.js
@@ -59,20 +59,41 @@ TEST_F('WebUIAssertionsTest', 'testConstructedMessage', function() {
 });
 
 /**
- * Async version of WebUIAssertionsTest.
+ * Failing version of WebUIAssertionsTest.
  * @extends WebUIAssertionsTest
  * @constructor
  */
-function WebUIAssertionsTestAsyncFail() {}
+function WebUIAssertionsTestFail() {}
 
-WebUIAssertionsTestAsyncFail.prototype = {
+WebUIAssertionsTestFail.prototype = {
   __proto__: WebUIAssertionsTest.prototype,
 
   /** @inheritDoc */
-  isAsync: true,
+  testShouldFail: true,
+};
+
+// Test that an assertion failure fails test.
+TEST_F('WebUIAssertionsTestFail', 'testAssertFailFails', function() {
+  assertNotReached();
+});
+
+// Test that an expect failure fails test.

[Sheridan Rawlins, 2011/10/28 20:51:20]

FYI, this is the <new> test, which failed before the fix in test_api.js

+TEST_F('WebUIAssertionsTestFail', 'testExpectFailFails', function() {
+  expectNotReached();
+});
+
+/**
+ * Async version of WebUIAssertionsTestFail.
+ * @extends WebUIAssertionsTest
+ * @constructor
+ */
+function WebUIAssertionsTestAsyncFail() {}
+
+WebUIAssertionsTestAsyncFail.prototype = {
+  __proto__: WebUIAssertionsTestFail.prototype,
 
   /** @inheritDoc */
-  testShouldFail: true,
+  isAsync: true,
 };
 
 // Test that an assertion failure doesn't hang forever.