From 0510edeebf76b8a9753acc06afde8b09a8f00a54 Mon Sep 17 00:00:00 2001 From: Florin Malita Date: Thu, 14 Jun 2018 18:41:39 +0000 Subject: Revert "[skjson] Size-constrained input API" This reverts commit fc792b8718cc30e9da62c9559b23c1baac3166bb. Reason for revert: New ASAN, Android failures. Original change's description: > [skjson] Size-constrained input API > > Pass an explicit input size instead of requiring a C string. > > Thanks to mtklein's clever trick, this has no measurable perf impact. > > Change-Id: I64f210a9f653a78b05ab6b58fa34479504aa35ff > Reviewed-on: https://skia-review.googlesource.com/134940 > Reviewed-by: Mike Klein > Commit-Queue: Florin Malita TBR=mtklein@google.com,fmalita@chromium.org Change-Id: Ic0b52398b1ce6f64c781debb858829cb64bbae58 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://skia-review.googlesource.com/135022 Reviewed-by: Florin Malita Commit-Queue: Florin Malita --- modules/skjson/src/SkJSONTest.cpp | 13 ++----------- 1 file changed, 2 insertions(+), 11 deletions(-) (limited to 'modules/skjson/src/SkJSONTest.cpp') diff --git a/modules/skjson/src/SkJSONTest.cpp b/modules/skjson/src/SkJSONTest.cpp index 91ae497cf0..6d4338cbbb 100644 --- a/modules/skjson/src/SkJSONTest.cpp +++ b/modules/skjson/src/SkJSONTest.cpp @@ -22,14 +22,8 @@ DEF_TEST(SkJSON_Parse, reporter) { { "" , nullptr }, { "[" , nullptr }, { "]" , nullptr }, - { "[[]" , nullptr }, - { "[]]" , nullptr }, - { "[]f" , nullptr }, { "{" , nullptr }, { "}" , nullptr }, - { "{{}" , nullptr }, - { "{}}" , nullptr }, - { "{}f" , nullptr }, { "{]" , nullptr }, { "[}" , nullptr }, { "1" , nullptr }, @@ -57,9 +51,7 @@ DEF_TEST(SkJSON_Parse, reporter) { { "{ \"k\" : null \"k\" : 1 }", nullptr }, - { "[]" , "[]" }, { " \n\r\t [ \n\r\t ] \n\r\t " , "[]" }, - { "[[]]" , "[[]]" }, { "[ null ]" , "[null]" }, { "[ true ]" , "[true]" }, { "[ false ]" , "[false]" }, @@ -74,7 +66,6 @@ DEF_TEST(SkJSON_Parse, reporter) { { "[ \"123456789\" ]" , "[\"123456789\"]" }, { "[ null , true, false,0,12.8 ]", "[null,true,false,0,12.8]" }, - { "{}" , "{}" }, { " \n\r\t { \n\r\t } \n\r\t " , "{}" }, { "{ \"k\" : null }" , "{\"k\":null}" }, { "{ \"k1\" : null, \"k2 \":0 }", "{\"k1\":null,\"k2 \":0}" }, @@ -98,7 +89,7 @@ DEF_TEST(SkJSON_Parse, reporter) { }; for (const auto& tst : g_tests) { - DOM dom(tst.in, strlen(tst.in)); + DOM dom(tst.in); const auto success = !dom.root().is(); REPORTER_ASSERT(reporter, success == (tst.out != nullptr)); if (!success) continue; @@ -162,7 +153,7 @@ DEF_TEST(SkJSON_DOM_visit, reporter) { \"k8\": { \"kk1\": 2, \"kk2\": false, \"kk1\": \"baz\" } \n\ }"; - DOM dom(json, strlen(json)); + DOM dom(json); const auto& jroot = dom.root().as(); REPORTER_ASSERT(reporter, jroot.is()); -- cgit v1.2.3