Whitelist extensions for oauth2 auto_approve.

Whitelist extensions for oauth2 auto_approve.

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

[Joao da Silva, 2014-08-04 19:59:36]

PTAL, thanks!

[not at google - send to devlin, 2014-08-04 20:09:25]

lgtm

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

https://codereview.chromium.org/441613005/diff/1/chrome/common/extensions/api...
chrome/common/extensions/api/_manifest_features.json:294:
"A4577D8C2AF4CF26F40CBCA83FFA4251D6F6C8F8",
I expect the trailing comma will break

[Joao da Silva, 2014-08-04 20:15:24]

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

https://codereview.chromium.org/441613005/diff/1/chrome/common/extensions/api...
chrome/common/extensions/api/_manifest_features.json:294:
"A4577D8C2AF4CF26F40CBCA83FFA4251D6F6C8F8",
On 2014/08/04 20:09:24, kalman wrote:
> I expect the trailing comma will break

So this JSON-esque format was modified to parse comments but not trailing commas
:-) Done!

BTW, there is an annoying presubmit error because these files can't be parsed as
vanilla JSON.

[not at google - send to devlin, 2014-08-04 20:16:39]

On 2014/08/04 20:15:24, Joao da Silva wrote:
>
https://codereview.chromium.org/441613005/diff/1/chrome/common/extensions/api...
> File chrome/common/extensions/api/_manifest_features.json (right):
> 
>
https://codereview.chromium.org/441613005/diff/1/chrome/common/extensions/api...
> chrome/common/extensions/api/_manifest_features.json:294:
> "A4577D8C2AF4CF26F40CBCA83FFA4251D6F6C8F8",
> On 2014/08/04 20:09:24, kalman wrote:
> > I expect the trailing comma will break
> 
> So this JSON-esque format was modified to parse comments but not trailing
commas
> :-) Done!
> 
> BTW, there is an annoying presubmit error because these files can't be parsed
as
> vanilla JSON.

hm, there shouldn't be. we fixed that.

[Joao da Silva, 2014-08-04 20:20:23]

> > BTW, there is an annoying presubmit error because these files can't be
parsed
> as
> > vanilla JSON.
> 
> hm, there shouldn't be. we fixed that.

You're right, it's gone once the trailing comma is removed :-)

[Joao da Silva, 2014-08-06 16:40:33]

The CQ bit was checked by joaodasilva@chromium.org

[commit-bot: I haz the power, 2014-08-06 16:42:15]

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

[commit-bot: I haz the power, 2014-08-06 19:55:47]

Change committed as 287835