Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(432)

Unified Diff: ios/chrome/browser/web/chrome_web_client_unittest.mm

Issue 2580333003: Upstream Chrome on iOS source code [10/11]. (Closed)
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ios/chrome/browser/web/chrome_web_client.mm ('k') | ios/chrome/browser/web/chrome_web_test.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/chrome/browser/web/chrome_web_client_unittest.mm
diff --git a/ios/chrome/browser/web/chrome_web_client_unittest.mm b/ios/chrome/browser/web/chrome_web_client_unittest.mm
new file mode 100644
index 0000000000000000000000000000000000000000..b070f8fc029b6f2ff34d632867b613cf437e11b6
--- /dev/null
+++ b/ios/chrome/browser/web/chrome_web_client_unittest.mm
@@ -0,0 +1,87 @@
+// Copyright (c) 2014 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/chrome/browser/web/chrome_web_client.h"
+
+#include <UIKit/UIKit.h>
+
+#include <memory>
+
+#include "base/mac/scoped_nsobject.h"
+#include "base/memory/ptr_util.h"
+#include "base/strings/string_split.h"
+#include "base/strings/sys_string_conversions.h"
+#import "ios/web/public/test/js_test_util.h"
+#include "ios/web/public/test/scoped_testing_web_client.h"
+#include "ios/web/public/test/test_browser_state.h"
+#include "ios/web/public/test/web_test.h"
+#import "ios/web/public/web_view_creation_util.h"
+#include "testing/gtest/include/gtest/gtest.h"
+#include "testing/gtest_mac.h"
+
+namespace {
+
+// Test fixture for testing ChromeWebClient.
+typedef web::WebTest ChromeWebClientTest;
+
+TEST_F(ChromeWebClientTest, UserAgent) {
+ std::vector<std::string> pieces;
+
+ // Check if the pieces of the user agent string come in the correct order.
+ ChromeWebClient web_client;
+ std::string buffer = web_client.GetUserAgent(false);
+
+ pieces = base::SplitStringUsingSubstr(
+ buffer, "Mozilla/5.0 (", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL);
+ ASSERT_EQ(2u, pieces.size());
+ buffer = pieces[1];
+ EXPECT_EQ("", pieces[0]);
+
+ pieces = base::SplitStringUsingSubstr(
+ buffer, ") AppleWebKit/", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL);
+ ASSERT_EQ(2u, pieces.size());
+ buffer = pieces[1];
+ std::string os_str = pieces[0];
+
+ pieces =
+ base::SplitStringUsingSubstr(buffer, " (KHTML, like Gecko) ",
+ base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL);
+ ASSERT_EQ(2u, pieces.size());
+ buffer = pieces[1];
+ std::string webkit_version_str = pieces[0];
+
+ pieces = base::SplitStringUsingSubstr(
+ buffer, " Safari/", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL);
+ ASSERT_EQ(2u, pieces.size());
+ std::string product_str = pieces[0];
+ std::string safari_version_str = pieces[1];
+
+ // Not sure what can be done to better check the OS string, since it's highly
+ // platform-dependent.
+ EXPECT_FALSE(os_str.empty());
+
+ EXPECT_FALSE(webkit_version_str.empty());
+ EXPECT_FALSE(safari_version_str.empty());
+
+ EXPECT_EQ(0u, product_str.find("CriOS/"));
+}
+
+// Tests that ChromeWebClient provides print script and does not provide
+// windowOpenFix script for WKWebView.
+TEST_F(ChromeWebClientTest, WKWebViewEarlyPageScript) {
+ // Chrome scripts rely on __gCrWeb object presence.
+ web::TestBrowserState browser_state;
+ WKWebView* web_view = web::BuildWKWebView(CGRectZero, &browser_state);
+ web::ExecuteJavaScript(web_view, @"__gCrWeb = {};");
+
+ web::ScopedTestingWebClient web_client(base::MakeUnique<ChromeWebClient>());
+ NSString* script = web_client.Get()->GetEarlyPageScript();
+ web::ExecuteJavaScript(web_view, script);
+ EXPECT_NSEQ(@"object",
+ web::ExecuteJavaScript(web_view, @"typeof __gCrWeb.print"));
+ EXPECT_NSEQ(@"undefined", web::ExecuteJavaScript(
+ web_view, @"typeof __gCrWeb.windowOpenFix"));
+}
+
+} // namespace
« no previous file with comments | « ios/chrome/browser/web/chrome_web_client.mm ('k') | ios/chrome/browser/web/chrome_web_test.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698