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

ios/web/public/test/web_test.mm

 // Copyright 2016 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.
 
 #include "ios/web/public/test/web_test.h"
 
 #include "base/memory/ptr_util.h"
 #include "ios/web/public/active_state_manager.h"
+#include "ios/web/public/web_state/global_web_state_observer.h"
 #import "ios/web/public/test/fakes/test_web_client.h"
 
+namespace {

[Eugene But (OOO till 7-30), 2017/02/01 16:43:04]

Drop this?

[rohitrao (ping after 24h), 2017/02/01 18:22:59]

Done.

+
+}  // namespace
+
 namespace web {
 
-WebTest::WebTest() : web_client_(base::WrapUnique(new TestWebClient)) {}
+class RendererCrashObserver : public GlobalWebStateObserver {
+ public:
+  RendererCrashObserver() = default;
+  ~RendererCrashObserver() override = default;
+
+  void RenderProcessGone(WebState* web_state) override {

[Eugene But (OOO till 7-30), 2017/02/01 16:43:04]

Optional nit: Drop |web_state| argument

[rohitrao (ping after 24h), 2017/02/01 18:22:59]

GlobalWebStateObserver needs to pass this, so that individual observers can know
which WebState crashed.

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

Sorry, what I meant was:
|void RenderProcessGone(WebState*) override {|

It's not important, so please ignore it :)

+    FAIL() << "Renderer process died unexpectedly during the test";
+  }
+};
+
+WebTest::WebTest()
+    : web_client_(base::WrapUnique(new TestWebClient)) {

[Eugene But (OOO till 7-30), 2017/02/01 16:43:04]

Do we need web client? WebTestSuite already sets it:
https://cs.chromium.org/chromium/src/ios/web/test/web_test_suite.mm?type=cs&q...

[rohitrao (ping after 24h), 2017/02/01 18:22:59]

Some tests call GetWebClient(), so leaving this in for now.

+  renderer_crash_observer_ = base::MakeUnique<RendererCrashObserver>();
+}
 
 WebTest::~WebTest() {}
 
 void WebTest::SetUp() {
   PlatformTest::SetUp();
   BrowserState::GetActiveStateManager(&browser_state_)->SetActive(true);
 }
 
 void WebTest::TearDown() {
   BrowserState::GetActiveStateManager(&browser_state_)->SetActive(false);
   PlatformTest::TearDown();
 }
 
 TestWebClient* WebTest::GetWebClient() {
   return static_cast<TestWebClient*>(web_client_.Get());
 }
 
 BrowserState* WebTest::GetBrowserState() {
   return &browser_state_;
 }
 
+void WebTest::AllowRenderProcessCrashesDuringTesting(bool allow) {
+  if (allow) {

[Eugene But (OOO till 7-30), 2017/02/01 16:43:04]

Optional nit: Do you want to use ternary?
renderer_crash_observer_ = allow ? nullptr :
base::MakeUnique<RendererCrashObserver>();

[rohitrao (ping after 24h), 2017/02/01 18:22:59]

If you don't feel strongly, I'll keep this as-is.

+    renderer_crash_observer_ = nullptr;
+  } else {
+    renderer_crash_observer_ = base::MakeUnique<RendererCrashObserver>();
+  }
+}
+
 }  // namespace web