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

Unified Diff: ios/web/web_state/js/core_js_unittest.mm

Issue 2807213003: Move stringify, form, navigation and scroll methods out of core.js. (Closed)
Patch Set: Created 3 years, 8 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
Index: ios/web/web_state/js/core_js_unittest.mm
diff --git a/ios/web/web_state/js/core_js_unittest.mm b/ios/web/web_state/js/core_js_unittest.mm
index 46e2703ce8e1f6d18fa7ba049afe60cb831508ff..4d7623ffca5e5fb0b638f4c064180a3d7213ff4e 100644
--- a/ios/web/web_state/js/core_js_unittest.mm
+++ b/ios/web/web_state/js/core_js_unittest.mm
@@ -194,69 +194,6 @@ TEST_F(CoreJsTest, HasPasswordFieldinFrame) {
<< base::SysNSStringToUTF8(data.pageContent);
}
-TEST_F(CoreJsTest, Stringify) {
- // TODO(jeanfrancoisg): Test whether __gCrWeb.stringify(undefined) correctly
- //returns undefined.
- TestScriptAndExpectedValue testData[] = {
- // Stringify a string that contains various characters that must
- // be escaped.
- {
- @"__gCrWeb.stringify('a\\u000a\\t\\b\\\\\\\"Z')",
- @"\"a\\n\\t\\b\\\\\\\"Z\""
- },
- // Stringify a number.
- {
- @"__gCrWeb.stringify(77.7)",
- @"77.7"
- },
- // Stringify an array.
- {
- @"__gCrWeb.stringify(['a','b'])",
- @"[\"a\",\"b\"]"
- },
- // Stringify an object.
- {
- @"__gCrWeb.stringify({'a':'b','c':'d'})",
- @"{\"a\":\"b\",\"c\":\"d\"}"
- },
- // Stringify a hierarchy of objects and arrays.
- {
- @"__gCrWeb.stringify([{'a':['b','c'],'d':'e'},'f'])",
- @"[{\"a\":[\"b\",\"c\"],\"d\":\"e\"},\"f\"]"
- },
- // Stringify null.
- {
- @"__gCrWeb.stringify(null)",
- @"null"
- },
- // Stringify an object with a toJSON function.
- {
- @"temp = [1,2];"
- "temp.toJSON = function (key) {return undefined};"
- "__gCrWeb.stringify(temp)",
- @"[1,2]"
- },
- // Stringify an object with a toJSON property that is not a function.
- {
- @"temp = [1,2];"
- "temp.toJSON = 42;"
- "__gCrWeb.stringify(temp)",
- @"[1,2]"
- },
- };
-
- for (size_t i = 0; i < arraysize(testData); i++) {
- TestScriptAndExpectedValue& data = testData[i];
- // Load a sample HTML page. As a side-effect, loading HTML via
- // |webController_| will also inject core.js.
- LoadHtml(@"<p>");
- id result = ExecuteJavaScript(data.test_script);
- EXPECT_NSEQ(data.expected_value, result)
- << " in test " << i << ": "
- << base::SysNSStringToUTF8(data.test_script);
- }
-}
-
// Tests the javascript of the url of the an image present in the DOM.
TEST_F(CoreJsTest, LinkOfImage) {
// A page with a large image surrounded by a link.

Powered by Google App Engine
This is Rietveld 408576698