Add getSelf method to management API to use without the "management" permission.

Add getSelf method to management API to use without the "management" permission.

BUG=392215
Committed: https://crrev.com/5780ffac611477a24274a7cdeb16bfde4b672640
Cr-Commit-Position: refs/heads/master@{#292660}

[Daniel Nishi, 2014-08-28 20:55:41]

dhnishi@chromium.org changed reviewers:
+ isherman@chromium.org, scheib@chromium.org

[Daniel Nishi, 2014-08-28 20:55:41]

isherman@chromium.org: Please review histogram changes.

scheib@chromium.org: Please review changes in chrome/browser/extensions/*,
chrome/common/extensions/*, and chrome/test/*.

https://codereview.chromium.org/508263003/diff/1/chrome/common/extensions/api...
File chrome/common/extensions/api/_api_features.json (right):

https://codereview.chromium.org/508263003/diff/1/chrome/common/extensions/api...
chrome/common/extensions/api/_api_features.json:549: "channel": "stable",
I wasn't entirely sure if this was the appropriate way to document the
no-permission function in the _api_features.json file. Is this correct?

[Daniel Nishi, 2014-08-28 20:56:32]

scheib@chromium.org: Please review changes in chrome/browser/extensions/*,
chrome/common/extensions/*, and chrome/test/*.

isherman@chromium.org: Please review changes in histograms.

[scheib, 2014-08-28 21:32:35]

lgtm with additional doc request.

https://codereview.chromium.org/508263003/diff/1/chrome/common/extensions/api...
File chrome/common/extensions/api/_api_features.json (right):

https://codereview.chromium.org/508263003/diff/1/chrome/common/extensions/api...
chrome/common/extensions/api/_api_features.json:549: "channel": "stable",
On 2014/08/28 20:55:40, Daniel Nishi wrote:
> I wasn't entirely sure if this was the appropriate way to document the
> no-permission function in the _api_features.json file. Is this correct?

https://developer.chrome.com/extensions/management documents the exception to
requiring the permission in multiple places, you've gotten one of them in this
patch but should add getSelf to the list in the overview as well in
chrome/common/extensions/docs/templates/intros/management.html.

[Daniel Nishi, 2014-08-28 22:04:17]

https://codereview.chromium.org/508263003/diff/1/chrome/common/extensions/api...
File chrome/common/extensions/api/_api_features.json (right):

https://codereview.chromium.org/508263003/diff/1/chrome/common/extensions/api...
chrome/common/extensions/api/_api_features.json:549: "channel": "stable",
On 2014/08/28 21:32:35, scheib wrote:
> On 2014/08/28 20:55:40, Daniel Nishi wrote:
> > I wasn't entirely sure if this was the appropriate way to document the
> > no-permission function in the _api_features.json file. Is this correct?
> 
> https://developer.chrome.com/extensions/management documents the exception to
> requiring the permission in multiple places, you've gotten one of them in this
> patch but should add getSelf to the list in the overview as well in
> chrome/common/extensions/docs/templates/intros/management.html.

Done.

[Ilya Sherman, 2014-08-28 22:40:23]

histograms lgtm

[Daniel Nishi, 2014-08-28 22:56:15]

The CQ bit was checked by dhnishi@chromium.org

[commit-bot: I haz the power, 2014-08-28 22:57:05]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/dhnishi@chromium.org/508263003/20001

[commit-bot: I haz the power, 2014-08-28 23:58:51]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_chromium_x64_rel_swarming on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[commit-bot: I haz the power, 2014-08-29 01:32:15]

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

[commit-bot: I haz the power, 2014-08-29 01:32:16]

Try jobs failed on following builders:
  win_chromium_x64_rel_swarming on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Daniel Nishi, 2014-08-29 16:20:06]

The CQ bit was checked by dhnishi@chromium.org

[commit-bot: I haz the power, 2014-08-29 16:21:10]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/dhnishi@chromium.org/508263003/40001

[commit-bot: I haz the power, 2014-08-29 17:14:43]

Committed patchset #3 (id:40001) as a77f7ccf01dee0ace8aef2dbfd596f0b5383ea7f

[commit-bot: I haz the power, 2014-09-10 03:09:44]

Patchset 3 (id:??) landed as
https://crrev.com/5780ffac611477a24274a7cdeb16bfde4b672640
Cr-Commit-Position: refs/heads/master@{#292660}