From 54a99450c633d64842217b10fa6841b2a3e7eb77 Mon Sep 17 00:00:00 2001 From: prashhir Date: Thu, 5 Mar 2015 01:30:57 -0800 Subject: Split ValueSerializer into separate Serializer and Deserializer classes. This CL removes the Deserializer functionality from ValueSerializer, and puts into separate class ValueDeserializer, so that class responsibility is maintained. BUG=455453 TBR=brettw@chromium.org Review URL: https://codereview.chromium.org/925783002 Cr-Commit-Position: refs/heads/master@{#319239} --- extensions/utility/unpacker.cc | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'extensions/utility') diff --git a/extensions/utility/unpacker.cc b/extensions/utility/unpacker.cc index d244ba5..c61faa2 100644 --- a/extensions/utility/unpacker.cc +++ b/extensions/utility/unpacker.cc @@ -118,9 +118,9 @@ base::DictionaryValue* Unpacker::ReadManifest() { return NULL; } - JSONFileValueSerializer serializer(manifest_path); + JSONFileValueDeserializer deserializer(manifest_path); std::string error; - scoped_ptr root(serializer.Deserialize(NULL, &error)); + scoped_ptr root(deserializer.Deserialize(NULL, &error)); if (!root.get()) { SetError(error); return NULL; @@ -270,9 +270,9 @@ bool Unpacker::AddDecodedImage(const base::FilePath& path) { bool Unpacker::ReadMessageCatalog(const base::FilePath& message_path) { std::string error; - JSONFileValueSerializer serializer(message_path); + JSONFileValueDeserializer deserializer(message_path); scoped_ptr root(static_cast( - serializer.Deserialize(NULL, &error))); + deserializer.Deserialize(NULL, &error))); if (!root.get()) { base::string16 messages_file = message_path.LossyDisplayName(); if (error.empty()) { -- cgit v1.1