Cleanup: Remove deprecated base::Value methods from chrome/common. Use base::Value too.

Cleanup: Remove deprecated base::Value methods from chrome/common. Use base::Value too.

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

[Yoyo Zhou, 2013-02-13 23:17:30]

LGTM

[commit-bot: I haz the power, 2013-02-14 00:59:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/thestig@chromium.org/12207167/1

[commit-bot: I haz the power, 2013-02-14 01:23:52]

Retried try job too often on linux_chromeos for step(s) aura_unittests,
base_unittests, browser_tests, cacheinvalidation_unittests, check_deps,
chromeos_unittests, components_unittests, content_browsertests,
content_unittests, crypto_unittests, dbus_unittests, device_unittests,
gpu_unittests, interactive_ui_tests, ipc_tests, jingle_unittests,
media_unittests, net_unittests, ppapi_unittests, printing_unittests,
sandbox_linux_unittests, sql_unittests, sync_unit_tests, unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...

[Yoyo Zhou, 2013-02-14 02:41:50]

On 2013/02/14 01:23:52, I haz the power (commit-bot) wrote:
> Retried try job too often on linux_chromeos for step(s) aura_unittests,
> base_unittests, browser_tests, cacheinvalidation_unittests, check_deps,
> chromeos_unittests, components_unittests, content_browsertests,
> content_unittests, crypto_unittests, dbus_unittests, device_unittests,
> gpu_unittests, interactive_ui_tests, ipc_tests, jingle_unittests,
> media_unittests, net_unittests, ppapi_unittests, printing_unittests,
> sandbox_linux_unittests, sql_unittests, sync_unit_tests, unit_tests
>
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...

I guess I was expecting to see chrome/common/extensions/manifest.cc in this CL
too, but I don't.

[Lei Zhang, 2013-02-14 02:49:35]

On 2013/02/14 02:41:50, Yoyo Zhou wrote:
> I guess I was expecting to see chrome/common/extensions/manifest.cc in this CL
> too, but I don't.

I'm mostly trying to get rid of deprecated Value::CreateFooValue functions.

[Yoyo Zhou, 2013-02-14 02:50:55]

On 2013/02/14 02:49:35, Lei Zhang wrote:
> On 2013/02/14 02:41:50, Yoyo Zhou wrote:
> > I guess I was expecting to see chrome/common/extensions/manifest.cc in this
CL
> > too, but I don't.
> 
> I'm mostly trying to get rid of deprecated Value::CreateFooValue functions.

Ok, no worries. I've got it open in another CL so I'll switch it from Value to
base::Value.

[commit-bot: I haz the power, 2013-02-14 05:35:37]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/thestig@chromium.org/12207167/1

[commit-bot: I haz the power, 2013-02-14 07:32:47]

Change committed as 182406