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

Unified Diff: extensions/browser/extension_icon_image_unittest.cc

Issue 925783002: Split ValueSerializer into separate Serializer and Deserializer classes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed cpplint warnings. Created 5 years, 10 months 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: extensions/browser/extension_icon_image_unittest.cc
diff --git a/extensions/browser/extension_icon_image_unittest.cc b/extensions/browser/extension_icon_image_unittest.cc
index 24d0ac2e4a4c46ad925cacb3dea48f3c95c1d283..3fb4e691b711b001f44de60ee3f7fea1eeac34e3 100644
--- a/extensions/browser/extension_icon_image_unittest.cc
+++ b/extensions/browser/extension_icon_image_unittest.cc
@@ -150,10 +150,11 @@ class ExtensionIconImageTest : public ExtensionsTest,
test_file = test_file.AppendASCII(name);
int error_code = 0;
std::string error;
- JSONFileValueSerializer serializer(test_file.AppendASCII("manifest.json"));
+ JSONFileValueDeserializer deserializer(
+ test_file.AppendASCII("manifest.json"));
scoped_ptr<base::DictionaryValue> valid_value(
- static_cast<base::DictionaryValue*>(serializer.Deserialize(&error_code,
- &error)));
+ static_cast<base::DictionaryValue*>(
+ deserializer.Deserialize(&error_code, &error)));
EXPECT_EQ(0, error_code) << error;
if (error_code != 0)
return NULL;
« no previous file with comments | « extensions/browser/api/printer_provider/printer_provider_apitest.cc ('k') | extensions/browser/image_loader_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698