Refactor web shell integration tests utilities.

ios/web/shell/test/navigation_test_util.h

Index: ios/web/shell/test/navigation_test_util.h
diff --git a/ios/web/shell/test/navigation_test_util.h b/ios/web/shell/test/navigation_test_util.h
deleted file mode 100644
index b46e9893a994338eb7b2975c46620ba968f4e911..0000000000000000000000000000000000000000
--- a/ios/web/shell/test/navigation_test_util.h
+++ /dev/null
@@ -1,20 +0,0 @@
-// 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.
-
-#ifndef IOS_WEB_SHELL_TEST_NAVIGATION_TEST_UTIL_H_
-#define IOS_WEB_SHELL_TEST_NAVIGATION_TEST_UTIL_H_
-
-#include "url/gurl.h"
-
-namespace web {
-namespace navigation_test_util {
-
-// Loads |url| in current webState with transition of type
-// ui::PAGE_TRANSITION_TYPED.
-void LoadUrl(GURL url);

[Eugene But (OOO till 7-30), 2016/05/05 21:43:41]

Maybe you should keep this method and use |void LoadUrl(web::WebState*, const
GURL&);| in the implementation?
Passing web state to LoadUrl seems to be unnecessary burden for a test writer.

[baxley, 2016/05/05 22:03:25]

Mind if I save it for a follow up CL?

[Eugene But (OOO till 7-30), 2016/05/05 22:22:44]

Sure.

-
-}  // namespace navigation_test_util
-}  // namespace web
-
-#endif  // IOS_WEB_SHELL_TEST_NAVIGATION_TEST_UTIL_H_