PaymentApp: Factor out functions to serialize/deserialize manifest data.

PaymentApp: Factor out functions to serialize/deserialize manifest data.

The functions will be used by other classes as well. For example, when we
query all manifest data, it can be used.
This CL doesn't change existing behavior.

BUG=661608
TEST=existing unittests

[zino, 2016-12-05 18:28:31]

Description was changed from

==========
PaymentApp: Factor out functions to serialize/deserialize manifest data.

The functions will be used by other classes as well. For example, when we
query all manifest data, it can be used.
This CL doesn't change existing behavior.

BUG=661608
TEST=existing unittests
==========

to

==========
PaymentApp: Factor out functions to serialize/deserialize manifest data.

The functions will be used by other classes as well. For example, when we
query all manifest data, it can be used.
This CL doesn't change existing behavior.

BUG=661608
TEST=existing unittests
==========

[zino, 2016-12-05 18:28:32]

jinho.bang@samsung.com changed reviewers:
+ avi@chromium.org, rouslan@chromium.org

[zino, 2016-12-05 18:28:42]

PTAL

[zino, 2016-12-05 18:29:37]

The CQ bit was checked by jinho.bang@samsung.com to run a CQ dry run

[commit-bot: I haz the power, 2016-12-05 18:30:00]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[please use gerrit instead, 2016-12-05 20:33:31]

I'm not a big fan of refactoring for something that may or may not happen in the
future. The rule of thumb is to submit only CLs that fix bugs or add features.
Factoring out function X() should happen only when X() is called from multiple
places.

If this is a small part of a larger patch that you're submitting separately to
make reviewers' lives easier, then please make it clear. One way to do that
would be to upload all patches with links to each other in the description.
Another way is to write up a design doc with all steps described in there.

[commit-bot: I haz the power, 2016-12-05 21:23:34]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-05 21:23:35]

Dry run: This issue passed the CQ dry run.