Star0
63
代码
代码
Issues247
Pull Requests42
流水线
讨论
Wiki
分析
Star0
63
Yyongsheng.zhu@intel.comFix the failed case ValueStoreTest.DotsInKeyNamesWithDicts for Android
8b0f9abb创建于 2012年8月23日历史提交
文件最后提交记录最后更新时间
json_file_value_serializer.cc
Break two classes defined in json_value_serializer.cc, .h into separate files. This will allow the use of JSONStringValueSerializer in the NaCl sandbox. BUG=none TEST=none Review URL: https://chromiumcodereview.appspot.com/9465030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@123701 0039d316-1c4b-4281-b951-d872f2087c98
14 年前
json_file_value_serializer.h
Remove the rest of #pragma once in one big CL. For context see this thread: https://groups.google.com/a/chromium.org/forum/?fromgroups#!topic/chromium-dev/RMcVNGjB4II TBR=thakis,pkasting,jam git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146163 0039d316-1c4b-4281-b951-d872f2087c98
13 年前
json_parser.cc
Fix the failed case ValueStoreTest.DotsInKeyNamesWithDicts for Android Json parser uses a stack-based input copy string and the parsed result 'Value' references the internal pointer of the input copy string. To make the references valid, 'Value' has one member std::string to swap its content with the input copy string. On other platforms, the member std::string can get the internal pointer of the input copy string. However, on Android, std::string just swaps the content of two strings instead of swapping their internal pointers. This makes the pointer reference of value invalid. Fix this by forcing options |= JSON_DETACHABLE_CHILDREN for Android so disable this optimization. BUG=139434 TEST=run_tests.py -s unit_tests Review URL: https://chromiumcodereview.appspot.com/10831322 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152944 0039d316-1c4b-4281-b951-d872f2087c98
13 年前
json_parser.h
Remove the rest of #pragma once in one big CL. For context see this thread: https://groups.google.com/a/chromium.org/forum/?fromgroups#!topic/chromium-dev/RMcVNGjB4II TBR=thakis,pkasting,jam git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146163 0039d316-1c4b-4281-b951-d872f2087c98
13 年前
json_parser_unittest.cc
Fix a bug in json parser where a 4 byte unicode character caused us to read an unterminated string from the stack. BUG=128737 Review URL: https://chromiumcodereview.appspot.com/10458002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139634 0039d316-1c4b-4281-b951-d872f2087c98
13 年前
json_reader.cc
Update JSONReader to take base::StringPiece instead of std::string I also went through the source tree and fixed any now-unnecessary conversions from base::StringPiece to std::string BUG=114785 TEST=Manual Review URL: https://chromiumcodereview.appspot.com/10003001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@140177 0039d316-1c4b-4281-b951-d872f2087c98
13 年前
json_reader.h
Remove the rest of #pragma once in one big CL. For context see this thread: https://groups.google.com/a/chromium.org/forum/?fromgroups#!topic/chromium-dev/RMcVNGjB4II TBR=thakis,pkasting,jam git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146163 0039d316-1c4b-4281-b951-d872f2087c98
13 年前
json_reader_unittest.cc
Rewrite base::JSONReader to be 35-40% faster, depending on the input string. This change does the following: * Parses the input string and generates the object representation in O(n) time. * Optimizes string decoding by using StringPiece where possible, which also introduces the JSON_DETACHABLE_CHILDREN parser option. * Makes JSONReader a simpler interface by hiding the parser details in an internal JSONParser class. BUG=49212,111581,121469 TEST=Hopefully covered by all test suites. New tests added for edge cases. Review URL: https://chromiumcodereview.appspot.com/10035042 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137430 0039d316-1c4b-4281-b951-d872f2087c98
13 年前
json_string_value_serializer.cc
Modify the base::JSONReader interface to take a set of options rather than a boolean flag. This is just prep work for a large CL that rewrites the JSON parser. Splitting out this interface change will make it easier to review the other CL. BUG=121469 TEST=No functional change. Review URL: https://chromiumcodereview.appspot.com/9960077 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@131680 0039d316-1c4b-4281-b951-d872f2087c98
13 年前
json_string_value_serializer.h
Remove the rest of #pragma once in one big CL. For context see this thread: https://groups.google.com/a/chromium.org/forum/?fromgroups#!topic/chromium-dev/RMcVNGjB4II TBR=thakis,pkasting,jam git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146163 0039d316-1c4b-4281-b951-d872f2087c98
13 年前
json_value_converter.h
Correct const accessors in base/values.(h|cc), Part II (ListValue) For problem description and other info please see the BUG page. This is for ListValue. BUG=138946 TEST=N/A (no fix & no new feature) TBR=jar,zelidrag,scottbyer,mpcomplete,darin,achuith,sky,estade,atwilson,grt,thakis,jamesr,hans,sadrul,pastarmovj Review URL: https://chromiumcodereview.appspot.com/10837044 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149819 0039d316-1c4b-4281-b951-d872f2087c98
13 年前
json_value_converter_unittest.cc
base: Fix minor warnings reported by cppcheck. Found with: $ cppcheck --enable=all -j4 --force base 2> base-errors R=jar@chromium.org NOTRY=True Review URL: https://chromiumcodereview.appspot.com/10827112 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150059 0039d316-1c4b-4281-b951-d872f2087c98
13 年前
json_value_serializer_unittest.cc
Break two classes defined in json_value_serializer.cc, .h into separate files. This will allow the use of JSONStringValueSerializer in the NaCl sandbox. BUG=none TEST=none Review URL: https://chromiumcodereview.appspot.com/9465030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@123701 0039d316-1c4b-4281-b951-d872f2087c98
14 年前
json_writer.cc
Correct const accessors in base/values.(h|cc), Part II (ListValue) For problem description and other info please see the BUG page. This is for ListValue. BUG=138946 TEST=N/A (no fix & no new feature) TBR=jar,zelidrag,scottbyer,mpcomplete,darin,achuith,sky,estade,atwilson,grt,thakis,jamesr,hans,sadrul,pastarmovj Review URL: https://chromiumcodereview.appspot.com/10837044 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149819 0039d316-1c4b-4281-b951-d872f2087c98
13 年前
json_writer.h
Remove the rest of #pragma once in one big CL. For context see this thread: https://groups.google.com/a/chromium.org/forum/?fromgroups#!topic/chromium-dev/RMcVNGjB4II TBR=thakis,pkasting,jam git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146163 0039d316-1c4b-4281-b951-d872f2087c98
13 年前
json_writer_unittest.cc
JSONWriter cleanup: integrate pretty print into write options. BUG= TEST=base_unittests TBR=abodenha@chromium.org,ajwong@chromium.org,chocobo@chromium.org,mnissler@chromium.org,akalin@chromium.org,brettw@chromium.org,arv@chromium.org Review URL: http://codereview.chromium.org/9590002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@127080 0039d316-1c4b-4281-b951-d872f2087c98
14 年前
string_escape.cc
iwyu: Include stringprintf.h where appropriate, part 1. BUG=82098 TEST=none R=thakis@chromium.org Review URL: http://codereview.chromium.org/6997006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84754 0039d316-1c4b-4281-b951-d872f2087c98
14 年前
string_escape.h
Remove the rest of #pragma once in one big CL. For context see this thread: https://groups.google.com/a/chromium.org/forum/?fromgroups#!topic/chromium-dev/RMcVNGjB4II TBR=thakis,pkasting,jam git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146163 0039d316-1c4b-4281-b951-d872f2087c98
13 年前
string_escape_unittest.cc
Improve the underlying escaping function JsonDoubleQuoteT to escape < and > characters BY DEFAULT to prevent script execution. BUG=40147 TEST=StringEscapeTest.* Review URL: http://codereview.chromium.org/1512013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43695 0039d316-1c4b-4281-b951-d872f2087c98
15 年前