Added gcdPrivate.getDeviceInfo function

Added gcdPrivate.getDeviceInfo function

GCD App needs to query devices for basic information.

BUG=brillo:839
Committed: https://crrev.com/cd4ff3af3ba883510a0fe1f802c35210c44247cb
Cr-Commit-Position: refs/heads/master@{#327343}

[Vitaly Buka (NO REVIEWS), 2015-04-27 23:21:55]

vitalybuka@chromium.org changed reviewers:
+ alekseys@chromium.org, kalman@chromium.org, mednik@chromium.org,
zentaro@chromium.org

[Vitaly Buka (NO REVIEWS), 2015-04-27 23:22:03]

PTAL

[not at google - send to devlin, 2015-04-27 23:40:35]

IDL lgtm

[Aleksey Shlyapnikov, 2015-04-27 23:54:31]

lgtm

[mednik, 2015-04-28 00:35:40]

https://codereview.chromium.org/1106183006/diff/1/chrome/common/extensions/ap...
File chrome/common/extensions/api/gcd_private.idl (right):

https://codereview.chromium.org/1106183006/diff/1/chrome/common/extensions/ap...
chrome/common/extensions/api/gcd_private.idl:89: // Called when the device info
is availible or on error.
Typo: available

https://codereview.chromium.org/1106183006/diff/1/chrome/common/extensions/ap...
chrome/common/extensions/api/gcd_private.idl:91: // |deviceInfo| : Content of
/privet/info response.
Do you need to include the pairingTypes in here? Or is that already being
populated in the /info response?

https://codereview.chromium.org/1106183006/diff/1/chrome/common/extensions/ap...
chrome/common/extensions/api/gcd_private.idl:103: PairingType[] pairingTypes);
Didn't you say you were going to remove the pairingTypes from here? Will that
happen in a different CL?

[Vitaly Buka (NO REVIEWS), 2015-04-28 00:35:56]

The CQ bit was checked by vitalybuka@chromium.org

[Vitaly Buka (NO REVIEWS), 2015-04-28 00:36:01]

The CQ bit was unchecked by vitalybuka@chromium.org

[Vitaly Buka (NO REVIEWS), 2015-04-28 00:37:52]

https://codereview.chromium.org/1106183006/diff/1/chrome/common/extensions/ap...
File chrome/common/extensions/api/gcd_private.idl (right):

https://codereview.chromium.org/1106183006/diff/1/chrome/common/extensions/ap...
chrome/common/extensions/api/gcd_private.idl:91: // |deviceInfo| : Content of
/privet/info response.
On 2015/04/28 00:35:40, mednik wrote:
> Do you need to include the pairingTypes in here? Or is that already being
> populated in the /info response?

it's inside of deviceInfo

https://codereview.chromium.org/1106183006/diff/1/chrome/common/extensions/ap...
chrome/common/extensions/api/gcd_private.idl:103: PairingType[] pairingTypes);
On 2015/04/28 00:35:40, mednik wrote:
> Didn't you say you were going to remove the pairingTypes from here? Will that
> happen in a different CL?

yes, in different CL when you update the app code.

[Vitaly Buka (NO REVIEWS), 2015-04-28 01:13:21]

https://codereview.chromium.org/1106183006/diff/1/chrome/common/extensions/ap...
File chrome/common/extensions/api/gcd_private.idl (right):

https://codereview.chromium.org/1106183006/diff/1/chrome/common/extensions/ap...
chrome/common/extensions/api/gcd_private.idl:89: // Called when the device info
is availible or on error.
On 2015/04/28 00:35:40, mednik wrote:
> Typo: available

Done.

[Vitaly Buka (NO REVIEWS), 2015-04-28 01:14:18]

vitalybuka@chromium.org changed reviewers:
+ asvitkine@google.com

[Vitaly Buka (NO REVIEWS), 2015-04-28 01:14:19]

+asvitkine for extensions\browser\extension_function_histogram_value.h

[Vitaly Buka (NO REVIEWS), 2015-04-28 01:15:55]

vitalybuka@chromium.org changed reviewers:
+ asvitkine@chromium.org
- asvitkine@google.com

[Vitaly Buka (NO REVIEWS), 2015-04-28 01:16:47]

+asvitkine for extensions\browser\extension_function_histogram_value.h

[Alexei Svitkine (slow), 2015-04-28 14:30:39]

lgtm

[Vitaly Buka (NO REVIEWS), 2015-04-28 18:10:00]

The CQ bit was checked by vitalybuka@chromium.org

[Vitaly Buka (NO REVIEWS), 2015-04-28 18:10:00]

The patchset sent to the CQ was uploaded after l-g-t-m from
alekseys@chromium.org, kalman@chromium.org
Link to the patchset: https://codereview.chromium.org/1106183006/#ps60001
(title: "Mon Apr 27 22:56:49 PDT 2015")

[commit-bot: I haz the power, 2015-04-28 18:11:07]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1106183006/60001

[commit-bot: I haz the power, 2015-04-28 19:00:58]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-04-28 19:01:54]

Patchset 4 (id:??) landed as
https://crrev.com/cd4ff3af3ba883510a0fe1f802c35210c44247cb
Cr-Commit-Position: refs/heads/master@{#327343}