Add code to load Hangout Services as a component extension.

Add code to load Hangout Services as a component extension.

Rename the extension from "Hangouts Services" to "Hangout Services".

Add public key and add ID to appropriate whitelists.

TBR=jochen@chromium.org
BUG=291271
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=230693

[Jói, 2013-10-23 17:45:02]

Victoria, this makes the extension load as a component extension.

Will find appropriate owners after you've taken a quick look.

Cheers,
Jói

[Jói, 2013-10-23 18:32:10]

nkostylev: Could you review the changes to chrome/browser/resources? This is
just a rename plus adding a public key to the component extension you reviewed
earlier today.

finnur: Are you an appropriate reviewer for the extension bits here? Or do I
need someone else for the ComponentLoader change?

Cheers,
Jói

[Nikita (slow), 2013-10-23 21:07:40]

lgtm

[Jói, 2013-10-23 22:00:04]

+yoz@ instead of finnur@ to optimize for time zone :)

Yoyo, if you have time it'd be great if you could review this today.

Cheers,
Jói

[Yoyo Zhou, 2013-10-23 23:24:27]

LGTM

https://codereview.chromium.org/30833003/diff/210001/chrome/browser/extension...
File chrome/browser/extensions/component_loader.cc (right):

https://codereview.chromium.org/30833003/diff/210001/chrome/browser/extension...
chrome/browser/extensions/component_loader.cc:436:
AddHangoutServicesExtension();
Can this (and other such functions) be in an anonymous namespace instead?

[Jói, 2013-10-24 08:23:31]

TBR=jochen@chromium.org for browser_resources.grd change.

Yoyo, see below. If I misunderstood your suggestion and you had something
different in mind, I'm happy to fix in a follow-up.

Cheers,
Jói

https://codereview.chromium.org/30833003/diff/210001/chrome/browser/extension...
File chrome/browser/extensions/component_loader.cc (right):

https://codereview.chromium.org/30833003/diff/210001/chrome/browser/extension...
chrome/browser/extensions/component_loader.cc:436:
AddHangoutServicesExtension();
On 2013/10/23 23:24:27, Yoyo Zhou wrote:
> Can this (and other such functions) be in an anonymous namespace instead?

Without some larger changes, they need to be members, since they all call the
Add() member.

[commit-bot: I haz the power, 2013-10-24 09:16:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/joi@chromium.org/30833003/210001

[commit-bot: I haz the power, 2013-10-24 11:36:13]

Change committed as 230693

[jochen (gone - plz use gerrit), 2013-10-24 13:05:15]

lgtm