commit d7de09a486899ae13a4b664947c8b79270fb9240
author bauerb <bauerb@chromium.org> Thu Jul 02 16:20:44 2015
committer Commit bot <commit-bot@chromium.org> Thu Jul 02 16:21:13 2015
tree 4eb222431b0af63d8a7a87f6e6f2d17f6919854e
parent 4e132285cadc8875bd8bf5d0078f325562fdd620

Disallow escaped invalid Unicode characters in JSONParser.

Without this CL, the JSONParser will DCHECK when trying to construct a
StringValue while parsing these characters (see
JSONParserTest.DecodeUnicodeNonCharacter). In a Release build, the parsed
StringValue will contain malformed UTF-8, which could lead to further bugs down
the line.

Review URL: https://codereview.chromium.org/1214993004

Cr-Commit-Position: refs/heads/master@{#337229}

chrome/browser/safe_json_parser_browsertest.cc

diff --git a/chrome/browser/safe_json_parser_browsertest.cc b/chrome/browser/safe_json_parser_browsertest.cc
index 119ecbc..4472a24 100644
--- a/chrome/browser/safe_json_parser_browsertest.cc
+++ b/chrome/browser/safe_json_parser_browsertest.cc
@@ -103,4 +103,7 @@
   TestParse(std::string());
   TestParse("☃");
   TestParse("\"☃\"");
+  TestParse("\"\\ufdd0\"");
+  TestParse("\"\\ufffe\"");
+  TestParse("\"\\ud83f\\udffe\"");
 }