[cros] Change Store/RetrievePolicy to use a GArray

[cros] Change Store/RetrievePolicy to use a GArray

It seems that sending protobufs as 'strings' works fine with the C
dbus-glib bindings, but not with the dbus-python bindings.  The
signatures of these DBus methods has changed, so change libcros to
keep pace.

BUG=13249
TEST=manual, using login_drive

Change-Id: Id00a2c8c1c5537bf0afbc5a3eeafb38ef1f3ac06

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=fb885f0

[Chris Masone, 2011-03-29 22:43:07]

Nothing uses Store/RetrivePolicy yet, so I feel OK changing the signature at
this point.

[stevenjb, 2011-03-30 14:27:11]

LGTM with nits

http://codereview.chromium.org/6772019/diff/1/chromeos_login.cc
File chromeos_login.cc (right):

http://codereview.chromium.org/6772019/diff/1/chromeos_login.cc#newcode528
chromeos_login.cc:528: DCHECK(delegate);
nit: DCHECK(prop)

http://codereview.chromium.org/6772019/diff/1/chromeos_login.cc#newcode534
chromeos_login.cc:534: //g_array_append_vals(policy, prop, strlen(prop));
Delete, or comment if you want to keep this for reference.

[Chris Masone, 2011-03-30 16:43:29]

http://codereview.chromium.org/6772019/diff/1/chromeos_login.cc
File chromeos_login.cc (right):

http://codereview.chromium.org/6772019/diff/1/chromeos_login.cc#newcode528
chromeos_login.cc:528: DCHECK(delegate);
On 2011/03/30 14:27:11, Steven Bennetts wrote:
> nit: DCHECK(prop)

Done.

http://codereview.chromium.org/6772019/diff/1/chromeos_login.cc#newcode534
chromeos_login.cc:534: //g_array_append_vals(policy, prop, strlen(prop));
On 2011/03/30 14:27:11, Steven Bennetts wrote:
> Delete, or comment if you want to keep this for reference.

Done.