[ios] Automatically fail any WebTest whose render process crashes.

ios/web/test/web_test_unittest.mm

Index: ios/web/test/web_test_unittest.mm
diff --git a/ios/web/test/web_test_unittest.mm b/ios/web/test/web_test_unittest.mm
new file mode 100644
index 0000000000000000000000000000000000000000..bd1ed4d2e27829ce35bc16e296fa6ed639cda235
--- /dev/null
+++ b/ios/web/test/web_test_unittest.mm
@@ -0,0 +1,50 @@
+// Copyright 2017 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.
+
+#import <WebKit/WebKit.h>
+
+#import "ios/web/public/test/fakes/test_web_view_content_view.h"
+#import "ios/web/test/web_test_with_web_controller.h"
+#import "ios/web/test/wk_web_view_crash_utils.h"
+#include "testing/gtest/include/gtest/gtest-spi.h"
+
+namespace {
+
+// Fixture to test that the WebTest fixture properly fails tests when the render
+// process crashes.
+class WebTestFixtureTest : public web::WebTestWithWebController {
+ protected:
+  void SetUp() override {
+    web::WebTestWithWebController::SetUp();
+    web_view_.reset([web::BuildTerminatedWKWebView() retain]);
+    base::scoped_nsobject<TestWebViewContentView> web_view_content_view(
+        [[TestWebViewContentView alloc]
+            initWithMockWebView:web_view_
+                     scrollView:[web_view_ scrollView]]);
+    [web_controller() injectWebViewContentView:web_view_content_view];
+  }
+
+  base::scoped_nsobject<WKWebView> web_view_;
+};
+
+// Tests that the WebTest fixture triggers a test failure when a render process
+// crashes during the test.
+TEST_F(WebTestFixtureTest, FailsOnRenderCrash) {
+  // EXPECT_FATAL_FAILURE() uses an inner class, which cannot access member
+  // variables or local variables.  Work around this by using a local static
+  // variable to hold the WKWebView pointer.
+  static WKWebView* web_view = web_view_;

[Eugene But (OOO till 7-30), 2017/02/01 18:41:45]

How about this comment?:
// EXPECT_FATAL_FAILURE() uses a local class which can not access non-static
variables of the enclosing function.

[rohitrao (ping after 24h), 2017/02/01 20:29:13]

Done.

+
+  EXPECT_FATAL_FAILURE(web::SimulateWKWebViewCrash(web_view),
+                       "Renderer process died unexpectedly");
+};
+
+// Tests that |SetIgnoreRenderProcessCrashesDuringTesting()| properly ignores
+// intentional render process crashes.
+TEST_F(WebTestFixtureTest, SucceedsOnRenderCrash) {
+  SetIgnoreRenderProcessCrashesDuringTesting(true);
+  web::SimulateWKWebViewCrash(web_view_);
+};
+
+}  // namespace