[ios] Speed up egtest failures by skipping _recordFailure.

_recordFailure internally spends a very long time symbolicating
on iOS13. Skipping this seems to be safe.

Bug:991338

Change-Id: I74c2872cd82de0840a6bcdd0f799703b5eb9bdb0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1725137Reviewed-by: Justin Cohen <justincohen@chromium.org>
Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Commit-Queue: Justin Cohen <justincohen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#684557}

ios/chrome/test/earl_grey/chrome_test_case.mm

@@ -104,6 +104,12 @@ UIDeviceOrientation GetCurrentDeviceOrientation() {
 
 }  // namespace
 
+// Category for overriding private methods on XCTestCase. See crbug.com/991338
+// for more information.
+@interface XCTestCase (Private)
+- (void)_recordFailure:(id)arg1;
+@end
+
 #if defined(CHROME_EARL_GREY_2)
 GREY_STUB_CLASS_IN_APP_MAIN_QUEUE(ChromeTestCaseAppInterface)
 #endif
@@ -180,6 +186,16 @@ GREY_STUB_CLASS_IN_APP_MAIN_QUEUE(ChromeTestCaseAppInterface)
   }
 }
 
+- (void)_recordFailure:(id)arg1 {
+  if (@available(iOS 13, *)) {
+    // _recordFailure internally spends a very long time symbolicating
+    // on iOS13. Skipping this seems to be safe. See crbug.com/991338
+    // for more information.
+  } else {
+    [super _recordFailure:arg1];
+  }
+}
+
 #if defined(CHROME_EARL_GREY_1)
 + (void)setUp {
   [super setUp];