Revert of media: Use StrongBindingSet for InterfaceFactoryImpl in MediaService

Revert of media: Use StrongBindingSet for InterfaceFactoryImpl in MediaService (patchset #1 id:1 of https://codereview.chromium.org/2884163002/ )

Reason for revert:
Layout tests failure ; mojo/module-loading(-manual-deps-loading).html

BUG=723461
Original issue's description:
> media: Use StrongBindingSet for InterfaceFactoryImpl in MediaService
> 
> Previously InterfaceFactoryImpl instances are using SrongBinding such
> that they will only be destroyed on connection error. However, we could
> hit an issue when MediaService is being destructed, the MojoMediaClient
> is already destroyed, and an InterfaceFactoryImpl still tries to access
> the MojoMediaClient.
> 
> This CL ensures that when MediaService is being destructed, all
> InterfaceFactoryImpl will also be destroyed to avoid the issue.
> 
> BUG=721965
> 
> Review-Url: https://codereview.chromium.org/2884163002
> Cr-Commit-Position: refs/heads/master@{#472261}
> Committed: https://chromium.googlesource.com/chromium/src/+/5223bd13f8a97e293361a3b9b2cc31101ba597da

TBR=alokp@chromium.org,xhwang@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=721965
Review-Url: https://codereview.chromium.org/2892563002
Cr-Commit-Position: refs/heads/master@{#472405}
Committed: https://chromium.googlesource.com/chromium/src/+/0ca49acbd1483c0e82be1a546168c5740231d908

[kolos1, 2017-05-17 09:58:37]

The CQ bit was checked by kolos@chromium.org

[kolos1, 2017-05-17 09:58:37]

Created Revert of media: Use StrongBindingSet for InterfaceFactoryImpl in
MediaService

[commit-bot: I haz the power, 2017-05-17 09:59:18]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-17 09:59:58]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495015117391450, "parent_rev":
"36e3810d369c449a7f6659addd6697f7e81132a4", "commit_rev":
"0ca49acbd1483c0e82be1a546168c5740231d908"}

[commit-bot: I haz the power, 2017-05-17 10:00:14]

Description was changed from

==========
Revert of media: Use StrongBindingSet for InterfaceFactoryImpl in MediaService
(patchset #1 id:1 of https://codereview.chromium.org/2884163002/ )

Reason for revert:
Layout tests failure ; mojo/module-loading(-manual-deps-loading).html

BUG=723461

Original issue's description:
> media: Use StrongBindingSet for InterfaceFactoryImpl in MediaService
> 
> Previously InterfaceFactoryImpl instances are using SrongBinding such
> that they will only be destroyed on connection error. However, we could
> hit an issue when MediaService is being destructed, the MojoMediaClient
> is already destroyed, and an InterfaceFactoryImpl still tries to access
> the MojoMediaClient.
> 
> This CL ensures that when MediaService is being destructed, all
> InterfaceFactoryImpl will also be destroyed to avoid the issue.
> 
> BUG=721965
> 
> Review-Url: https://codereview.chromium.org/2884163002
> Cr-Commit-Position: refs/heads/master@{#472261}
> Committed:
https://chromium.googlesource.com/chromium/src/+/5223bd13f8a97e293361a3b9b2cc...

TBR=alokp@chromium.org,xhwang@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=721965
==========

to

==========
Revert of media: Use StrongBindingSet for InterfaceFactoryImpl in MediaService
(patchset #1 id:1 of https://codereview.chromium.org/2884163002/ )

Reason for revert:
Layout tests failure ; mojo/module-loading(-manual-deps-loading).html

BUG=723461

Original issue's description:
> media: Use StrongBindingSet for InterfaceFactoryImpl in MediaService
> 
> Previously InterfaceFactoryImpl instances are using SrongBinding such
> that they will only be destroyed on connection error. However, we could
> hit an issue when MediaService is being destructed, the MojoMediaClient
> is already destroyed, and an InterfaceFactoryImpl still tries to access
> the MojoMediaClient.
> 
> This CL ensures that when MediaService is being destructed, all
> InterfaceFactoryImpl will also be destroyed to avoid the issue.
> 
> BUG=721965
> 
> Review-Url: https://codereview.chromium.org/2884163002
> Cr-Commit-Position: refs/heads/master@{#472261}
> Committed:
https://chromium.googlesource.com/chromium/src/+/5223bd13f8a97e293361a3b9b2cc...

TBR=alokp@chromium.org,xhwang@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=721965

Review-Url: https://codereview.chromium.org/2892563002
Cr-Commit-Position: refs/heads/master@{#472405}
Committed:
https://chromium.googlesource.com/chromium/src/+/0ca49acbd1483c0e82be1a546168...
==========

[commit-bot: I haz the power, 2017-05-17 10:00:15]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/0ca49acbd1483c0e82be1a546168...

[xhwang, 2017-05-17 16:19:19]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2887153002/ by xhwang@chromium.org.

The reason for reverting is: This was not the cause of the failure, which has
been fixed by reverting other CLs. See BUG for details..