Remove matcher to wait for static HTML view.

ios/chrome/test/earl_grey/chrome_earl_grey.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.
 
 #import "ios/chrome/test/earl_grey/chrome_earl_grey.h"
 
 #import <Foundation/Foundation.h>
 #import <WebKit/WebKit.h>
 
 #include "base/mac/foundation_util.h"
 #include "base/strings/sys_string_conversions.h"
+#import "base/test/ios/wait_util.h"
+#include "components/strings/grit/components_strings.h"
+#import "ios/chrome/browser/ui/static_content/static_html_view_controller.h"
 #import "ios/chrome/test/app/chrome_test_util.h"
 #import "ios/chrome/test/app/history_test_util.h"
 #include "ios/chrome/test/app/navigation_test_util.h"
+#import "ios/chrome/test/app/static_html_view_test_util.h"
 #import "ios/testing/wait_util.h"
 #import "ios/web/public/test/earl_grey/js_test_util.h"
 #import "ios/web/public/test/web_view_interaction_test_util.h"
 #import "ios/web/public/web_state/js/crw_js_injection_receiver.h"
 #import "ios/web/public/web_state/web_state.h"
+#include "ui/base/l10n/l10n_util.h"
 
 #if !defined(__has_feature) || !__has_feature(objc_arc)
 #error "This file requires ARC support."
 #endif
 
 namespace chrome_test_util {
 
 id ExecuteJavaScript(NSString* javascript,
                      NSError* __unsafe_unretained* out_error) {
   __block bool did_complete = false;
@@ skipping 88 matching lines @@
 }
 
 + (void)tapWebViewElementWithID:(NSString*)elementID {
   BOOL success =
       web::test::TapWebViewElementWithId(chrome_test_util::GetCurrentWebState(),
                                          base::SysNSStringToUTF8(elementID));
   GREYAssertTrue(success, @"Failed to tap web view element with ID: %@",
                  elementID);
 }
 
++ (void)waitForErrorPage {
+  NSString* const kInternetDisconnectedError =

[lpromero, 2017/04/19 11:35:10]

The error is not about internet disconnection, but about page being unavailable.

[baxley, 2017/04/19 14:39:49]

Thanks for pointing out the inconsistency. Do you think the API make sense, but
should maybe have a more clear comment/method name, and definitely a better
variable name here? Or do you think it should be doing something different?

[lpromero, 2017/04/19 14:44:52]

The variable name might be sufficient, but since the error being waited upon is
specific, maybe a more specific method name could be worth it? I didn't feel the
need for it, I only thought of raising the issue for the variable.

+      l10n_util::GetNSString(IDS_ERRORPAGES_HEADING_NOT_AVAILABLE);
+  [self waitForStaticHTMLViewContainingText:kInternetDisconnectedError];
+}
+
++ (void)waitForStaticHTMLViewContainingText:(NSString*)text {
+  GREYCondition* condition = [GREYCondition
+      conditionWithName:@"Wait for static HTML text."
+                  block:^BOOL {
+                    return chrome_test_util::StaticHtmlViewContainingText(
+                        chrome_test_util::GetCurrentWebState(),
+                        base::SysNSStringToUTF8(text));
+                  }];
+  GREYAssert([condition waitWithTimeout:testing::kWaitForUIElementTimeout],
+             @"Failed to find static html view containing %@", text);
+}
+
++ (void)waitForStaticHTMLViewNotContainingText:(NSString*)text {
+  GREYCondition* condition = [GREYCondition
+      conditionWithName:@"Wait for absence of static HTML text."
+                  block:^BOOL {
+                    return !chrome_test_util::StaticHtmlViewContainingText(
+                        chrome_test_util::GetCurrentWebState(),
+                        base::SysNSStringToUTF8(text));
+                  }];
+  GREYAssert([condition waitWithTimeout:testing::kWaitForUIElementTimeout],
+             @"Failed, there was a static html view containing %@", text);
+}
+
 @end