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

Unified Diff: chrome/common/pref_service_unittest.cc

Issue 13169: Add error messages to JSONReader and friends. This required a bit of refactor... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 12 years 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: chrome/common/pref_service_unittest.cc
===================================================================
--- chrome/common/pref_service_unittest.cc (revision 6390)
+++ chrome/common/pref_service_unittest.cc (working copy)
@@ -155,19 +155,20 @@
Value* transient_value;
{
JSONStringValueSerializer serializer(transient);
- ASSERT_TRUE(serializer.Deserialize(&transient_value));
+ ASSERT_TRUE(serializer.Deserialize(&transient_value, NULL));
}
prefs.transient()->Set(transient_string, transient_value);
Value* both_transient_value;
{
JSONStringValueSerializer serializer(transient);
- ASSERT_TRUE(serializer.Deserialize(&both_transient_value));
+ ASSERT_TRUE(serializer.Deserialize(&both_transient_value, NULL));
}
prefs.transient()->Set(L"both", both_transient_value);
// Register test prefs
- const wchar_t* kTypes[] = { L"neither.", L"persistent.", L"transient.", L"both." };
+ const wchar_t* kTypes[] =
+ { L"neither.", L"persistent.", L"transient.", L"both." };
for (size_t i = 0; i < arraysize(kTypes); ++i) {
wchar_t temp[1024];
wcscpy_s(temp, kTypes[i]);

Powered by Google App Engine
This is Rietveld 408576698