[Sync] Add favicon sync experiment

[Sync] Add favicon sync experiment

The old tab sync favicon experiment is deprecated. This patch updates the
strings to the fact that we now have a separate datatype, and hooks this
up to a new sync field.

BUG=154886
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=188974

[Nicolas Zea, 2013-03-18 18:39:16]

PTAL

[rlarocque, 2013-03-18 18:48:11]

lgtm

https://codereview.chromium.org/12700016/diff/2001/sync/protocol/proto_value_...
File sync/protocol/proto_value_conversions.cc (right):

https://codereview.chromium.org/12700016/diff/2001/sync/protocol/proto_value_...
sync/protocol/proto_value_conversions.cc:111: proto.field().has_enabled()) {    
        \
nit: align slashes.

[Nicolas Zea, 2013-03-18 19:12:18]

Thanks for the quick review! Done, committing.

https://codereview.chromium.org/12700016/diff/2001/sync/protocol/proto_value_...
File sync/protocol/proto_value_conversions.cc (right):

https://codereview.chromium.org/12700016/diff/2001/sync/protocol/proto_value_...
sync/protocol/proto_value_conversions.cc:111: proto.field().has_enabled()) {    
        \
On 2013/03/18 18:48:11, rlarocque wrote:
> nit: align slashes.

Done.

[commit-bot: I haz the power, 2013-03-18 19:13:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/zea@chromium.org/12700016/6001

[commit-bot: I haz the power, 2013-03-19 01:26:31]

Retried try job too often on win_rel for step(s) sync_integration_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2013-03-19 02:50:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/zea@chromium.org/12700016/6001

[commit-bot: I haz the power, 2013-03-19 09:58:41]

Change committed as 188974