commit 0ba935147805c722a5320fa203ef5a600cff3f90
author sky@google.com <sky@google.com@0039d316-1c4b-4281-b951-d872f2087c98> Thu Oct 02 02:26:06 2008 +0900
committer Qijiang Fan <fqj@google.com> Thu Jun 04 09:13:49 2020 +0900
tree 632c9b6b1f230eae79d0a40128c17b3447ffd6aa
parent d4eb87f2d5f0eb3cdd12ec803330f079d8beaa49

Fixes bug where JSON reader errored on reading vertical tabs (\v). The
writer properly escapes a vertical tab, but the readed instead errored
on the vertical this. This resulted in lost bookmarks:(

BUG=3031
TEST=covered by unit tests

Review URL: http://codereview.chromium.org/5628

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@2759 0039d316-1c4b-4281-b951-d872f2087c98


CrOS-Libchrome-Original-Commit: d46d6f3b80099ce0c630185a2e86b649bba49254

base/json_reader_unittest.cc

diff --git a/base/json_reader_unittest.cc b/base/json_reader_unittest.cc
index c2d6a42..4339be1 100644
--- a/base/json_reader_unittest.cc
+++ b/base/json_reader_unittest.cc
@@ -214,13 +214,13 @@
 
   // Test basic string escapes
   root = NULL;
-  ASSERT_TRUE(JSONReader::JsonToValue("\" \\\"\\\\\\/\\b\\f\\n\\r\\t\"", &root,
+  ASSERT_TRUE(JSONReader::JsonToValue("\" \\\"\\\\\\/\\b\\f\\n\\r\\t\\v\"", &root,
                                       false, false));
   ASSERT_TRUE(root);
   ASSERT_TRUE(root->IsType(Value::TYPE_STRING));
   str_val.clear();
   ASSERT_TRUE(root->GetAsString(&str_val));
-  ASSERT_EQ(L" \"\\/\b\f\n\r\t", str_val);
+  ASSERT_EQ(L" \"\\/\b\f\n\r\t\v", str_val);
   delete root;
 
   // Test hex and unicode escapes including the null character.