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

Unified Diff: ios/web/web_state/context_menu_params_utils_unittest.mm

Issue 2640433002: Clean the new CRWContextMenuController (Closed)
Patch Set: feedback Created 3 years, 11 months 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/web/web_state/context_menu_params_utils.mm ('k') | ios/web/web_state/ui/crw_context_menu_controller.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/web/web_state/context_menu_params_utils_unittest.mm
diff --git a/ios/web/web_state/context_menu_params_utils_unittest.mm b/ios/web/web_state/context_menu_params_utils_unittest.mm
new file mode 100644
index 0000000000000000000000000000000000000000..d43a4b352adef52d9cd99365010b48262b6c9b3e
--- /dev/null
+++ b/ios/web/web_state/context_menu_params_utils_unittest.mm
@@ -0,0 +1,99 @@
+// 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.
+
+#import "ios/web/web_state/context_menu_params_utils.h"
+
+#include "base/strings/sys_string_conversions.h"
+#include "components/url_formatter/url_formatter.h"
+#include "ios/web/public/referrer_util.h"
+#import "ios/web/public/web_state/context_menu_params.h"
+#import "net/base/mac/url_conversions.h"
+#include "testing/gtest/include/gtest/gtest.h"
+#import "testing/gtest_mac.h"
+#include "testing/platform_test.h"
+
+namespace {
+// Text values for the tapped element triggering the context menu.
+const char* kLinkUrl = "http://link.url/";
+const char* kSrcUrl = "http://src.url/";
+const char* kTitle = "title";
+const char* kReferrerPolicy = "always";
+const char* kLinkText = "link text";
+const char* kJavaScriptLinkUrl = "javascript://src.url/";
+const char* kDataUrl = "data://foo.bar/";
+}
+
+// Test fixture for error translation testing.
+typedef PlatformTest ContextMenuParamsUtilsTest;
+
+// Tests the empty contructor.
+TEST_F(ContextMenuParamsUtilsTest, EmptyParams) {
+ web::ContextMenuParams params;
+ EXPECT_EQ(params.menu_title.get(), nil);
+ EXPECT_FALSE(params.link_url.is_valid());
+ EXPECT_FALSE(params.src_url.is_valid());
+ EXPECT_EQ(params.referrer_policy, web::ReferrerPolicyDefault);
+ EXPECT_EQ(params.view.get(), nil);
+ EXPECT_TRUE(CGPointEqualToPoint(params.location, CGPointZero));
+ EXPECT_EQ(params.link_text.get(), nil);
+}
+
+// Tests the the parsing of the element NSDictionary.
+TEST_F(ContextMenuParamsUtilsTest, DictionaryConstructorTest) {
+ web::ContextMenuParams params = web::ContextMenuParamsFromElementDictionary(@{
+ @"href" : @(kLinkUrl),
+ @"src" : @(kSrcUrl),
+ @"title" : @(kTitle),
+ @"referrerPolicy" : @(kReferrerPolicy),
+ @"innerText" : @(kLinkText),
+ });
+
+ EXPECT_NSEQ(params.menu_title.get(), @(kTitle));
+ EXPECT_EQ(params.link_url, GURL(kLinkUrl));
+ EXPECT_EQ(params.src_url, GURL(kSrcUrl));
+ EXPECT_NSEQ(params.link_text.get(), @(kLinkText));
+ EXPECT_EQ(params.referrer_policy,
+ web::ReferrerPolicyFromString(kReferrerPolicy));
+
+ EXPECT_EQ(params.view.get(), nil);
+ EXPECT_TRUE(CGPointEqualToPoint(params.location, CGPointZero));
+}
+
+// Tests title is set as the formatted URL there is no title.
+TEST_F(ContextMenuParamsUtilsTest, DictionaryConstructorTestNoTitle) {
+ web::ContextMenuParams params = web::ContextMenuParamsFromElementDictionary(@{
+ @"href" : @(kLinkUrl),
+ });
+ base::string16 urlText = url_formatter::FormatUrl(GURL(kLinkUrl));
+ NSString* title = base::SysUTF16ToNSString(urlText);
+
+ EXPECT_NSEQ(params.menu_title.get(), title);
+}
+
+// Tests title is set to "JavaScript" if there is no title and "href" links to
+// JavaScript URL.
+TEST_F(ContextMenuParamsUtilsTest, DictionaryConstructorTestJavascriptTitle) {
+ web::ContextMenuParams params = web::ContextMenuParamsFromElementDictionary(@{
+ @"href" : @(kJavaScriptLinkUrl),
+ });
+ EXPECT_NSEQ(params.menu_title.get(), @"JavaScript");
+}
+
+// Tests title is set to |src_url| if there is no title.
+TEST_F(ContextMenuParamsUtilsTest, DictionaryConstructorTestSrcTitle) {
+ web::ContextMenuParams params = web::ContextMenuParamsFromElementDictionary(@{
+ @"src" : @(kSrcUrl),
+ });
+ EXPECT_EQ(params.src_url, GURL(kSrcUrl));
+ EXPECT_NSEQ(params.menu_title.get(), @(kSrcUrl));
+}
+
+// Tests title is set to nil if there is no title and src is a data URL.
+TEST_F(ContextMenuParamsUtilsTest, DictionaryConstructorTestDataTitle) {
+ web::ContextMenuParams params = web::ContextMenuParamsFromElementDictionary(@{
+ @"src" : @(kDataUrl),
+ });
+ EXPECT_EQ(params.src_url, GURL(kDataUrl));
+ EXPECT_NSEQ(params.menu_title.get(), nil);
+}
« no previous file with comments | « ios/web/web_state/context_menu_params_utils.mm ('k') | ios/web/web_state/ui/crw_context_menu_controller.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698