Remove matcher to wait for static HTML view.

ios/chrome/test/app/static_html_view_test_util.h

Index: ios/chrome/test/app/static_html_view_test_util.h
diff --git a/ios/chrome/test/app/static_html_view_test_util.h b/ios/chrome/test/app/static_html_view_test_util.h
new file mode 100644
index 0000000000000000000000000000000000000000..8e19659a3dbedb8dfbb13b13989e7050d463d9ad
--- /dev/null
+++ b/ios/chrome/test/app/static_html_view_test_util.h
@@ -0,0 +1,19 @@
+// 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.
+
+#ifndef IOS_CHROME_TEST_APP_STATIC_HTML_VIEW_TEST_UTIL_H_
+#define IOS_CHROME_TEST_APP_STATIC_HTML_VIEW_TEST_UTIL_H_
+
+#import "ios/web/public/web_state/web_state.h"
+
+namespace chrome_test_util {
+
+// Returns true if there is a static html view in the |web_state|, which
+// contains |text|. If the text is not present, or there is no static html view
+// it returns false.
+bool StaticHtmlViewContainingText(web::WebState* web_state, std::string text);

[baxley, 2017/03/31 20:59:22]

I know you had a concern about having some methods that return bool, some that
assert, some that wait, and some that don't.

I think it's reasonable to have ios/chrome/test/app methods that do no waiting
and just check a state. Then ChromeEarlGrey can do any waiting that is
necessary.

[Eugene But (OOO till 7-30), 2017/03/31 22:19:17]

I think this is fine. AFAIK, methods inside chrome_test_util are not suppose to
assert.

[baxley, 2017/04/03 18:10:41]

Yes, this comment was in response to an offline conversation about it being
confusing to have some that wait, some that don't, some that assert, some that
return bool.

I think that didn't have the context of this, where it fits fine.

+
+}  // namespace chrome_test_util
+
+#endif  // IOS_CHROME_TEST_APP_STATIC_HTML_VIEW_TEST_UTIL_H_