Revert of Enable overriding interface binders for any services running in current process.

Revert of Enable overriding interface binders for any services running in current process. (patchset #5 id:130001 of https://codereview.chromium.org/2859343003/ )

Reason for revert:
VibrationTest.Vibrate times out in browser_side_navigation_content_browsertests on Linux.

See https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.linux%2FLinux_Tests__dbg__1_%2F63093%2F%2B%2Frecipes%2Fsteps%2Fbrowser_side_navigation_content_browsertests%2F0%2Fstdout

Similar issues were found on many different platforms.

Original issue's description:
> Enable overriding interface binders for any services running in current process.
> 
> This CL
>   - holds a 'service name <--> binder registry' map inside
>     service manager client library, at run time it acts as a global
>     variable in any process linking with service manager client library.
>   - lets ServiceContext::OnBindInterface() use the global map to
>     intercept requests before dispatching them to target Services, that
>     is to say, such interception may happen for any Services running in
>     current process.
>   - exposes {SetGlobalBinder,ClearGlobalBinders}ForTesting to set/clear
>     binders in the global map.
>   - adds a VibrationTest browser test to show the usage of the new infra
>     described above.
> 
> BUG=717377, 717378
> TEST=content_browsertests
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Review-Url: https://codereview.chromium.org/2859343003
> Cr-Commit-Position: refs/heads/master@{#471216}
> Committed: https://chromium.googlesource.com/chromium/src/+/d25e37df0f08a754fbc11041451a5ce7ca7d043d

TBR=rockot@chromium.org,blundell@chromium.org,jam@chromium.org,leon.han@intel.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=717377, 717378
Review-Url: https://codereview.chromium.org/2882613002
Cr-Commit-Position: refs/heads/master@{#471246}
Committed: https://chromium.googlesource.com/chromium/src/+/7795bff0b81a0cd0a8f840357545f591d0cdf6b0

[henrika (OOO until Aug 14), 2017-05-12 07:59:54]

The CQ bit was checked by henrika@chromium.org

[henrika (OOO until Aug 14), 2017-05-12 07:59:55]

Created Revert of Enable overriding interface binders for any services running
in current process.

[commit-bot: I haz the power, 2017-05-12 08:00:10]

CQ is trying da patch.

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

[henrika (OOO until Aug 14), 2017-05-12 08:00:34]

[ RUN      ] VibrationTest.Vibrate
[7018:7018:0511/221018.273968:24891087465:WARNING:audio_manager.cc(293)]
Multiple instances of AudioManager detected
[7018:7018:0511/221018.274260:24891087628:WARNING:audio_manager.cc(254)]
Multiple instances of AudioManager detected
Xlib:  extension "RANDR" missing on display ":99".
BrowserTestBase received signal: Terminated. Backtrace:
#0 0x7f40877eb89b base::debug::StackTrace::StackTrace()
#1 0x7f40877ea59c base::debug::StackTrace::StackTrace()
#2 0x000001402597 content::(anonymous namespace)::DumpStackTraceSignalHandler()
#3 0x7f407ad76cb0 <unknown>
#4 0x7f407ae3084d __poll
#5 0x7f4079513fe4 <unknown>
#6 0x7f40795140ec g_main_context_iteration
#7 0x7f4087896eaf base::MessagePumpGlib::Run()
#8 0x7f4087883982 base::MessageLoop::RunHandler()
#9 0x7f408792b18a base::RunLoop::Run()
#10 0x000000a56544 content::(anonymous
namespace)::VibrationTest_Vibrate_Test::RunTestOnMainThread()
#11 0x00000140226a content::BrowserTestBase::ProxyRunTestOnMainThreadLoop()
#12 0x000000630725
_ZN4base8internal13FunctorTraitsIMN7content21CrossSiteTransferTestEFvvEvE6InvokeIPS3_JEEEvS5_OT_DpOT0_
#13 0x000000630641
_ZN4base8internal12InvokeHelperILb0EvE8MakeItSoIRKMN7content21CrossSiteTransferTestEFvvEJPS5_EEEvOT_DpOT0_
#14 0x000001404097
_ZN4base8internal7InvokerINS0_9BindStateIMN7content15BrowserTestBaseEFvvEJNS0_17UnretainedWrapperIS4_EEEEEFvvEE7RunImplIRKS6_RKSt5tupleIJS8_EEJLm0EEEEvOT_OT0_NS_13IndexSequenceIJXspT1_EEEE
#15 0x000001403fdc
_ZN4base8internal7InvokerINS0_9BindStateIMN7content15BrowserTestBaseEFvvEJNS0_17UnretainedWrapperIS4_EEEEEFvvEE3RunEPNS0_13BindStateBaseE
#16 0x00000061135d
_ZNKR4base8CallbackIFvvELNS_8internal8CopyModeE1ELNS2_10RepeatModeE1EE3RunEv
#17 0x000001570a00 content::ShellBrowserMainParts::PreMainMessageLoopRun()
#18 0x7f4089596cc1 content::BrowserMainLoop::PreMainMessageLoopRun()
#19 0x7f40887c8805
_ZN4base8internal13FunctorTraitsIMN7content13URLLoaderImplEFvvEvE6InvokeIPS3_JEEEvS5_OT_DpOT0_
#20 0x7f40895a06a1
_ZN4base8internal12InvokeHelperILb0EiE8MakeItSoIRKMN7content15BrowserMainLoopEFivEJPS5_EEEiOT_DpOT0_
#21 0x7f40895a0647
_ZN4base8internal7InvokerINS0_9BindStateIMN7content15BrowserMainLoopEFivEJNS0_17UnretainedWrapperIS4_EEEEEFivEE7RunImplIRKS6_RKSt5tupleIJS8_EEJLm0EEEEiOT_OT0_NS_13IndexSequenceIJXspT1_EEEE
#22 0x7f40895a058c
_ZN4base8internal7InvokerINS0_9BindStateIMN7content15BrowserMainLoopEFivEJNS0_17UnretainedWrapperIS4_EEEEEFivEE3RunEPNS0_13BindStateBaseE
#23 0x7f40887984cd
_ZNKR4base8CallbackIFvvELNS_8internal8CopyModeE1ELNS2_10RepeatModeE1EE3RunEv
#24 0x7f408a1c5c1b content::StartupTaskRunner::RunAllTasksNow()
#25 0x7f40895947c0 content::BrowserMainLoop::CreateStartupTasks()
#26 0x7f40895a38b7 content::BrowserMainRunnerImpl::Initialize()
#27 0x00000151dab4 ShellBrowserMain()
#28 0x0000015023c5 content::ShellMainDelegate::RunProcess()
#29 0x7f408aed300b content::RunNamedProcessTypeMain()
#30 0x7f408aed538d content::ContentMainRunnerImpl::Run()
#31 0x7f408aed1afa
content::ContentServiceManagerMainDelegate::RunEmbedderProcess()
#32 0x7f407806d501 service_manager::Main()
#33 0x7f408aed2aaf content::ContentMain()
#34 0x000001401f5e content::BrowserTestBase::SetUp()
#35 0x0000013ee63d content::ContentBrowserTest::SetUp()
#36 0x000000de8e7e testing::internal::HandleSehExceptionsInMethodIfSupported<>()
#37 0x000000dd9ee2 testing::internal::HandleExceptionsInMethodIfSupported<>()
#38 0x000000dce944 testing::Test::Run()
#39 0x000000dcf14d testing::TestInfo::Run()
#40 0x000000dcf6ef testing::TestCase::Run()
#41 0x000000dd4a5c testing::internal::UnitTestImpl::RunAllTests()
#42 0x000000ded00e testing::internal::HandleSehExceptionsInMethodIfSupported<>()
#43 0x000000ddb752 testing::internal::HandleExceptionsInMethodIfSupported<>()
#44 0x000000dd46ff testing::UnitTest::Run()
#45 0x0000014b5f81 RUN_ALL_TESTS()
#46 0x0000014b4c72 base::TestSuite::Run()
#47 0x0000013facd0 content::ContentTestLauncherDelegate::RunTestSuite()
#48 0x00000145789f content::LaunchTests()
#49 0x0000013fab18 main
#50 0x7f407ad61f45 __libc_start_main
#51 0x0000005a6b54 <unknown>
[1306/1893] VibrationTest.Vibrate (TIMED OUT)

[commit-bot: I haz the power, 2017-05-12 08:00:39]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1494575994925980, "parent_rev":
"e1f4f5e2d1cc9c40e2ab79491606e46d2bc86a90", "commit_rev":
"7795bff0b81a0cd0a8f840357545f591d0cdf6b0"}

[commit-bot: I haz the power, 2017-05-12 08:00:52]

Description was changed from

==========
Revert of Enable overriding interface binders for any services running in
current process. (patchset #5 id:130001 of
https://codereview.chromium.org/2859343003/ )

Reason for revert:
VibrationTest.Vibrate times out in browser_side_navigation_content_browsertests
on Linux.

See
https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.linux%2FLinux_T...

Original issue's description:
> Enable overriding interface binders for any services running in current
process.
> 
> This CL
>   - holds a 'service name <--> binder registry' map inside
>     service manager client library, at run time it acts as a global
>     variable in any process linking with service manager client library.
>   - lets ServiceContext::OnBindInterface() use the global map to
>     intercept requests before dispatching them to target Services, that
>     is to say, such interception may happen for any Services running in
>     current process.
>   - exposes {SetGlobalBinder,ClearGlobalBinders}ForTesting to set/clear
>     binders in the global map.
>   - adds a VibrationTest browser test to show the usage of the new infra
>     described above.
> 
> BUG=717377, 717378
> TEST=content_browsertests
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Review-Url: https://codereview.chromium.org/2859343003
> Cr-Commit-Position: refs/heads/master@{#471216}
> Committed:
https://chromium.googlesource.com/chromium/src/+/d25e37df0f08a754fbc11041451a...

TBR=rockot@chromium.org,blundell@chromium.org,jam@chromium.org,leon.han@intel...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=717377, 717378
==========

to

==========
Revert of Enable overriding interface binders for any services running in
current process. (patchset #5 id:130001 of
https://codereview.chromium.org/2859343003/ )

Reason for revert:
VibrationTest.Vibrate times out in browser_side_navigation_content_browsertests
on Linux.

See
https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.linux%2FLinux_T...

Original issue's description:
> Enable overriding interface binders for any services running in current
process.
> 
> This CL
>   - holds a 'service name <--> binder registry' map inside
>     service manager client library, at run time it acts as a global
>     variable in any process linking with service manager client library.
>   - lets ServiceContext::OnBindInterface() use the global map to
>     intercept requests before dispatching them to target Services, that
>     is to say, such interception may happen for any Services running in
>     current process.
>   - exposes {SetGlobalBinder,ClearGlobalBinders}ForTesting to set/clear
>     binders in the global map.
>   - adds a VibrationTest browser test to show the usage of the new infra
>     described above.
> 
> BUG=717377, 717378
> TEST=content_browsertests
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Review-Url: https://codereview.chromium.org/2859343003
> Cr-Commit-Position: refs/heads/master@{#471216}
> Committed:
https://chromium.googlesource.com/chromium/src/+/d25e37df0f08a754fbc11041451a...

TBR=rockot@chromium.org,blundell@chromium.org,jam@chromium.org,leon.han@intel...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=717377, 717378

Review-Url: https://codereview.chromium.org/2882613002
Cr-Commit-Position: refs/heads/master@{#471246}
Committed:
https://chromium.googlesource.com/chromium/src/+/7795bff0b81a0cd0a8f840357545...
==========

[commit-bot: I haz the power, 2017-05-12 08:00:53]

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

[henrika (OOO until Aug 14), 2017-05-12 08:08:47]

Description was changed from

==========
Revert of Enable overriding interface binders for any services running in
current process. (patchset #5 id:130001 of
https://codereview.chromium.org/2859343003/ )

Reason for revert:
VibrationTest.Vibrate times out in browser_side_navigation_content_browsertests
on Linux.

See
https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.linux%2FLinux_T...

Original issue's description:
> Enable overriding interface binders for any services running in current
process.
> 
> This CL
>   - holds a 'service name <--> binder registry' map inside
>     service manager client library, at run time it acts as a global
>     variable in any process linking with service manager client library.
>   - lets ServiceContext::OnBindInterface() use the global map to
>     intercept requests before dispatching them to target Services, that
>     is to say, such interception may happen for any Services running in
>     current process.
>   - exposes {SetGlobalBinder,ClearGlobalBinders}ForTesting to set/clear
>     binders in the global map.
>   - adds a VibrationTest browser test to show the usage of the new infra
>     described above.
> 
> BUG=717377, 717378
> TEST=content_browsertests
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Review-Url: https://codereview.chromium.org/2859343003
> Cr-Commit-Position: refs/heads/master@{#471216}
> Committed:
https://chromium.googlesource.com/chromium/src/+/d25e37df0f08a754fbc11041451a...

TBR=rockot@chromium.org,blundell@chromium.org,jam@chromium.org,leon.han@intel...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=717377, 717378

Review-Url: https://codereview.chromium.org/2882613002
Cr-Commit-Position: refs/heads/master@{#471246}
Committed:
https://chromium.googlesource.com/chromium/src/+/7795bff0b81a0cd0a8f840357545...
==========

to

==========
Revert of Enable overriding interface binders for any services running in
current process. (patchset #5 id:130001 of
https://codereview.chromium.org/2859343003/ )

Reason for revert:
VibrationTest.Vibrate times out in browser_side_navigation_content_browsertests
on Linux.

See
https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.linux%2FLinux_T...

Similar issues were found on many different platforms.

Original issue's description:
> Enable overriding interface binders for any services running in current
process.
> 
> This CL
>   - holds a 'service name <--> binder registry' map inside
>     service manager client library, at run time it acts as a global
>     variable in any process linking with service manager client library.
>   - lets ServiceContext::OnBindInterface() use the global map to
>     intercept requests before dispatching them to target Services, that
>     is to say, such interception may happen for any Services running in
>     current process.
>   - exposes {SetGlobalBinder,ClearGlobalBinders}ForTesting to set/clear
>     binders in the global map.
>   - adds a VibrationTest browser test to show the usage of the new infra
>     described above.
> 
> BUG=717377, 717378
> TEST=content_browsertests
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Review-Url: https://codereview.chromium.org/2859343003
> Cr-Commit-Position: refs/heads/master@{#471216}
> Committed:
https://chromium.googlesource.com/chromium/src/+/d25e37df0f08a754fbc11041451a...

TBR=rockot@chromium.org,blundell@chromium.org,jam@chromium.org,leon.han@intel...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=717377, 717378

Review-Url: https://codereview.chromium.org/2882613002
Cr-Commit-Position: refs/heads/master@{#471246}
Committed:
https://chromium.googlesource.com/chromium/src/+/7795bff0b81a0cd0a8f840357545...
==========