Remove uses of shell::Loader from content

Remove uses of shell::Loader from content

Replaces existing uses with the same SCF-based interface
used for other shell connections.

Adds a manifest for mojo:media (packaged in exe:content_browser)
so the shell knows where to route its connections.

BUG=601884
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
TBR=xhwang@chromium.org for addition of the app manifest

Committed: https://crrev.com/3cc6df2e0b129c29e4e9e02ca290d13ad32b4ae1
Cr-Commit-Position: refs/heads/master@{#387708}

[Ken Rockot(use gerrit already), 2016-04-15 18:24:50]

Description was changed from

==========
Remove uses of shell::Loader from content

Replaces existing uses with the same SCF-based interface
used for other shell connections.

BUG=601884
==========

to

==========
Remove uses of shell::Loader from content

Replaces existing uses with the same SCF-based interface
used for other shell connections.

BUG=601884
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

[Ken Rockot(use gerrit already), 2016-04-15 18:26:26]

Description was changed from

==========
Remove uses of shell::Loader from content

Replaces existing uses with the same SCF-based interface
used for other shell connections.

BUG=601884
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Remove uses of shell::Loader from content

Replaces existing uses with the same SCF-based interface
used for other shell connections.

Adds a manifest for mojo:media (packaged in exe:content_browser)
so the shell knows where to route its connections.

BUG=601884
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

[Ken Rockot(use gerrit already), 2016-04-15 18:30:15]

rockot@chromium.org changed reviewers:
+ ben@chromium.org

[Ken Rockot(use gerrit already), 2016-04-15 18:30:16]

BackgroundShell still uses Loader, but I'll kill that use (and Loader itself) in
a follow-up patch.

https://codereview.chromium.org/1889253002/diff/1/content/browser/frame_host/...
File content/browser/frame_host/frame_mojo_shell.cc (right):

https://codereview.chromium.org/1889253002/diff/1/content/browser/frame_host/...
content/browser/frame_host/frame_mojo_shell.cc:61: shell::mojom::kRootUserID,
target->name,
I realize we don't want to use "root" for anything in practice, but this gets
the requests routed to the right place (the global shell connection) for now.
FrameMojoShell is going to go away and shouldn't be used for anything new.

[Ben Goodger (Google), 2016-04-15 19:41:40]

lgtm

[Ken Rockot(use gerrit already), 2016-04-15 20:16:28]

Description was changed from

==========
Remove uses of shell::Loader from content

Replaces existing uses with the same SCF-based interface
used for other shell connections.

Adds a manifest for mojo:media (packaged in exe:content_browser)
so the shell knows where to route its connections.

BUG=601884
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Remove uses of shell::Loader from content

Replaces existing uses with the same SCF-based interface
used for other shell connections.

Adds a manifest for mojo:media (packaged in exe:content_browser)
so the shell knows where to route its connections.

BUG=601884
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
TBR=xhwang@chromium.org for addition of the app manifest
==========

[Ken Rockot(use gerrit already), 2016-04-15 20:38:35]

The CQ bit was checked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2016-04-15 20:38:35]

The patchset sent to the CQ was uploaded after l-g-t-m from ben@chromium.org
Link to the patchset: https://codereview.chromium.org/1889253002/#ps40001
(title: " ")

[commit-bot: I haz the power, 2016-04-15 20:38:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1889253002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1889253002/40001

[commit-bot: I haz the power, 2016-04-15 21:46:31]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-04-15 21:48:22]

Description was changed from

==========
Remove uses of shell::Loader from content

Replaces existing uses with the same SCF-based interface
used for other shell connections.

Adds a manifest for mojo:media (packaged in exe:content_browser)
so the shell knows where to route its connections.

BUG=601884
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
TBR=xhwang@chromium.org for addition of the app manifest
==========

to

==========
Remove uses of shell::Loader from content

Replaces existing uses with the same SCF-based interface
used for other shell connections.

Adds a manifest for mojo:media (packaged in exe:content_browser)
so the shell knows where to route its connections.

BUG=601884
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
TBR=xhwang@chromium.org for addition of the app manifest

Committed: https://crrev.com/3cc6df2e0b129c29e4e9e02ca290d13ad32b4ae1
Cr-Commit-Position: refs/heads/master@{#387708}
==========

[commit-bot: I haz the power, 2016-04-15 21:48:23]

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