Adds ModuleRegistryObserver interface

Adds ModuleRegistryObserver interface

This will be used to know when a module has been added so that I can
download needed modules.

BUG=none
TEST=none
R=abarth@chromium.org

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

[sky, 2014-03-05 15:24:55]

I'm also adding RegisterGlobalsObject, which is used to register 'define' on an
object.

[abarth-chromium, 2014-03-06 00:32:17]

lgtm

https://codereview.chromium.org/187653004/diff/20001/gin/modules/module_regis...
File gin/modules/module_registry.cc (right):

https://codereview.chromium.org/187653004/diff/20001/gin/modules/module_regis...
gin/modules/module_registry.cc:119: GetDefineTemplate(isolate)->GetFunction());
Please be aware that this can execute arbitrary script in the page.

https://codereview.chromium.org/187653004/diff/20001/gin/modules/module_regis...
File gin/modules/module_registry.h (right):

https://codereview.chromium.org/187653004/diff/20001/gin/modules/module_regis...
gin/modules/module_registry.h:12: #include <vector>
Why did you need to add this include?

https://codereview.chromium.org/187653004/diff/20001/gin/modules/module_regis...
gin/modules/module_registry.h:49: v8::Handle<v8::Object> obj);
This isn't really registering any globals.  It's installing the globals... 
Maybe:

InstallGlobals

?

[sky, 2014-03-06 00:55:01]

https://codereview.chromium.org/187653004/diff/20001/gin/modules/module_regis...
File gin/modules/module_registry.cc (right):

https://codereview.chromium.org/187653004/diff/20001/gin/modules/module_regis...
gin/modules/module_registry.cc:119: GetDefineTemplate(isolate)->GetFunction());
On 2014/03/06 00:32:17, abarth wrote:
> Please be aware that this can execute arbitrary script in the page.

I added a warning.

https://codereview.chromium.org/187653004/diff/20001/gin/modules/module_regis...
File gin/modules/module_registry.h (right):

https://codereview.chromium.org/187653004/diff/20001/gin/modules/module_regis...
gin/modules/module_registry.h:12: #include <vector>
On 2014/03/06 00:32:17, abarth wrote:
> Why did you need to add this include?

Good point, nuked.

https://codereview.chromium.org/187653004/diff/20001/gin/modules/module_regis...
gin/modules/module_registry.h:49: v8::Handle<v8::Object> obj);
On 2014/03/06 00:32:17, abarth wrote:
> This isn't really registering any globals.  It's installing the globals... 
> Maybe:
> 
> InstallGlobals
> 
> ?

Done.

[sky, 2014-03-06 00:55:06]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2014-03-06 00:55:37]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sky@chromium.org/187653004/60001

[commit-bot: I haz the power, 2014-03-06 02:53:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sky@chromium.org/187653004/60001

[commit-bot: I haz the power, 2014-03-06 03:17:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sky@chromium.org/187653004/60001

[commit-bot: I haz the power, 2014-03-06 10:28:16]

Change committed as 255291