Moved guest_view_registry to GuestViewManager and made it an instance map

Moved guest_view_registry to GuestViewManager and made it an instance map

This change allows for the change towards moving GuestViewManager to components
and implementing an extensions specific GuestViewManager that installs extensions-specific
guest types.

BUG=444869
Committed: https://crrev.com/21d4d15a81b030f522fef29a0429f08a70220f68
Cr-Commit-Position: refs/heads/master@{#325919}

[Fady Samuel, 2015-04-16 23:15:14]

fsamuel@chromium.org changed reviewers:
+ lazyboy@chromium.org

[Fady Samuel, 2015-04-16 23:15:14]

GuestViewBase now has no dependencies on extensions.

GuestViewManager will next be refactored and a new
ExtensionsGuestViewManagerDelegate will handle GuestView type registration, and
permissions.

[Fady Samuel, 2015-04-20 15:59:29]

fsamuel@chromium.org changed reviewers:
+ lfg@chromium.org
- lazyboy@chromium.org

[Fady Samuel, 2015-04-20 15:59:30]

fsamuel@chromium.org changed reviewers:
+ lfg@chromium.org
- lazyboy@chromium.org

[Fady Samuel, 2015-04-20 16:00:01]

fsamuel@chromium.org changed reviewers:
+ lazyboy@chromium.org
- lfg@chromium.org

[Fady Samuel, 2015-04-20 16:00:01]

OOps, assigning back to Istiaque

[lazyboy, 2015-04-20 19:08:57]

https://chromiumcodereview.appspot.com/1096623002/diff/1/extensions/browser/g...
File extensions/browser/guest_view/guest_view.h (right):

https://chromiumcodereview.appspot.com/1096623002/diff/1/extensions/browser/g...
extensions/browser/guest_view/guest_view.h:8: #include "base/bind.h"
remove?

https://chromiumcodereview.appspot.com/1096623002/diff/1/extensions/browser/g...
File extensions/browser/guest_view/guest_view_manager.h (right):

https://chromiumcodereview.appspot.com/1096623002/diff/1/extensions/browser/g...
extensions/browser/guest_view/guest_view_manager.h:115: // Create a guest of the
provided |view_type|.
Creates

[Fady Samuel, 2015-04-20 20:35:23]

Addressed comments. PTAL

https://codereview.chromium.org/1096623002/diff/1/extensions/browser/guest_vi...
File extensions/browser/guest_view/guest_view.h (right):

https://codereview.chromium.org/1096623002/diff/1/extensions/browser/guest_vi...
extensions/browser/guest_view/guest_view.h:8: #include "base/bind.h"
On 2015/04/20 19:08:57, lazyboy wrote:
> remove?

Done.

https://codereview.chromium.org/1096623002/diff/1/extensions/browser/guest_vi...
File extensions/browser/guest_view/guest_view_manager.h (right):

https://codereview.chromium.org/1096623002/diff/1/extensions/browser/guest_vi...
extensions/browser/guest_view/guest_view_manager.h:115: // Create a guest of the
provided |view_type|.
On 2015/04/20 19:08:57, lazyboy wrote:
> Creates

Done.

[lazyboy, 2015-04-20 20:39:50]

lgtm

[Fady Samuel, 2015-04-20 20:40:17]

The CQ bit was checked by fsamuel@chromium.org

[commit-bot: I haz the power, 2015-04-20 20:41:02]

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

[commit-bot: I haz the power, 2015-04-20 22:06:08]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-04-20 22:07:34]

Patchset 2 (id:??) landed as
https://crrev.com/21d4d15a81b030f522fef29a0429f08a70220f68
Cr-Commit-Position: refs/heads/master@{#325919}