Revert of services/ui: Avoid thread hopping in the gpu process.

Revert of services/ui: Avoid thread hopping in the gpu process. (patchset #3 id:40001 of https://codereview.chromium.org/2329043002/ )

Reason for revert:
mus_ws_unittests on linux debug builders are crashing:

[11191:11245:0913/111107:210172940:FATAL:lock_impl_posix.cc(65)] Check failed: rv == 0 (22 vs. 0). Invalid argument
#0 0x7f806072969e base::debug::StackTrace::StackTrace()
#1 0x7f806079332f logging::LogMessage::~LogMessage()
#2 0x7f806089106d base::internal::LockImpl::Lock()
#3 0x7f80606f9d23 base::Lock::Acquire()
#4 0x7f80606f99c3 base::AutoLock::AutoLock()
#5 0x7f8060857364 base::SequenceCheckerImpl::CalledOnValidSequence()
#6 0x000000660eba base::Thread::task_runner()
#7 0x0000006606b2 ui::GpuServiceInternal::Initialize()
#8 0x0000006a14f0 ui::mojom::GpuServiceInternalStub::AcceptWithResponder()
...

Original issue's description:
> services/ui: Avoid thread hopping in the gpu process.
>
> Bind the GpuServiceInternal request on the gpu thread, instead of in the
> main thread. This means it is possible to service a request from the host
> without thread hopping between the main and gpu threads.
>
> BUG=643746
>
> Committed: https://crrev.com/9894bae5c1e1f03ab2b14f7b2325296587002a58
> Cr-Commit-Position: refs/heads/master@{#418275}

TBR=fsamuel@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=643746
Committed: https://crrev.com/d1cca1e62f682bb2c84a9272c603c4d3aa40047b
Cr-Commit-Position: refs/heads/master@{#418344}

[sadrul, 2016-09-13 19:49:47]

The CQ bit was checked by sadrul@chromium.org

[sadrul, 2016-09-13 19:49:48]

Created Revert of services/ui: Avoid thread hopping in the gpu process.

[commit-bot: I haz the power, 2016-09-13 19:50:35]

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, 2016-09-13 19:51:54]

Description was changed from

==========
Revert of services/ui: Avoid thread hopping in the gpu process. (patchset #3
id:40001 of https://codereview.chromium.org/2329043002/ )

Reason for revert:
mus_ws_unittests on linux debug builders are crashing:

[11191:11245:0913/111107:210172940:FATAL:lock_impl_posix.cc(65)] Check failed:
rv == 0 (22 vs. 0). Invalid argument
#0 0x7f806072969e base::debug::StackTrace::StackTrace()
#1 0x7f806079332f logging::LogMessage::~LogMessage()
#2 0x7f806089106d base::internal::LockImpl::Lock()
#3 0x7f80606f9d23 base::Lock::Acquire()
#4 0x7f80606f99c3 base::AutoLock::AutoLock()
#5 0x7f8060857364 base::SequenceCheckerImpl::CalledOnValidSequence()
#6 0x000000660eba base::Thread::task_runner()
#7 0x0000006606b2 ui::GpuServiceInternal::Initialize()
#8 0x0000006a14f0 ui::mojom::GpuServiceInternalStub::AcceptWithResponder()
...

Original issue's description:
> services/ui: Avoid thread hopping in the gpu process.
> 
> Bind the GpuServiceInternal request on the gpu thread, instead of in the
> main thread. This means it is possible to service a request from the host
> without thread hopping between the main and gpu threads.
> 
> BUG=643746
> 
> Committed: https://crrev.com/9894bae5c1e1f03ab2b14f7b2325296587002a58
> Cr-Commit-Position: refs/heads/master@{#418275}

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

to

==========
Revert of services/ui: Avoid thread hopping in the gpu process. (patchset #3
id:40001 of https://codereview.chromium.org/2329043002/ )

Reason for revert:
mus_ws_unittests on linux debug builders are crashing:

[11191:11245:0913/111107:210172940:FATAL:lock_impl_posix.cc(65)] Check failed:
rv == 0 (22 vs. 0). Invalid argument
#0 0x7f806072969e base::debug::StackTrace::StackTrace()
#1 0x7f806079332f logging::LogMessage::~LogMessage()
#2 0x7f806089106d base::internal::LockImpl::Lock()
#3 0x7f80606f9d23 base::Lock::Acquire()
#4 0x7f80606f99c3 base::AutoLock::AutoLock()
#5 0x7f8060857364 base::SequenceCheckerImpl::CalledOnValidSequence()
#6 0x000000660eba base::Thread::task_runner()
#7 0x0000006606b2 ui::GpuServiceInternal::Initialize()
#8 0x0000006a14f0 ui::mojom::GpuServiceInternalStub::AcceptWithResponder()
...

Original issue's description:
> services/ui: Avoid thread hopping in the gpu process.
> 
> Bind the GpuServiceInternal request on the gpu thread, instead of in the
> main thread. This means it is possible to service a request from the host
> without thread hopping between the main and gpu threads.
> 
> BUG=643746
> 
> Committed: https://crrev.com/9894bae5c1e1f03ab2b14f7b2325296587002a58
> Cr-Commit-Position: refs/heads/master@{#418275}

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

[commit-bot: I haz the power, 2016-09-13 19:51:55]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-13 19:54:38]

Description was changed from

==========
Revert of services/ui: Avoid thread hopping in the gpu process. (patchset #3
id:40001 of https://codereview.chromium.org/2329043002/ )

Reason for revert:
mus_ws_unittests on linux debug builders are crashing:

[11191:11245:0913/111107:210172940:FATAL:lock_impl_posix.cc(65)] Check failed:
rv == 0 (22 vs. 0). Invalid argument
#0 0x7f806072969e base::debug::StackTrace::StackTrace()
#1 0x7f806079332f logging::LogMessage::~LogMessage()
#2 0x7f806089106d base::internal::LockImpl::Lock()
#3 0x7f80606f9d23 base::Lock::Acquire()
#4 0x7f80606f99c3 base::AutoLock::AutoLock()
#5 0x7f8060857364 base::SequenceCheckerImpl::CalledOnValidSequence()
#6 0x000000660eba base::Thread::task_runner()
#7 0x0000006606b2 ui::GpuServiceInternal::Initialize()
#8 0x0000006a14f0 ui::mojom::GpuServiceInternalStub::AcceptWithResponder()
...

Original issue's description:
> services/ui: Avoid thread hopping in the gpu process.
> 
> Bind the GpuServiceInternal request on the gpu thread, instead of in the
> main thread. This means it is possible to service a request from the host
> without thread hopping between the main and gpu threads.
> 
> BUG=643746
> 
> Committed: https://crrev.com/9894bae5c1e1f03ab2b14f7b2325296587002a58
> Cr-Commit-Position: refs/heads/master@{#418275}

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

to

==========
Revert of services/ui: Avoid thread hopping in the gpu process. (patchset #3
id:40001 of https://codereview.chromium.org/2329043002/ )

Reason for revert:
mus_ws_unittests on linux debug builders are crashing:

[11191:11245:0913/111107:210172940:FATAL:lock_impl_posix.cc(65)] Check failed:
rv == 0 (22 vs. 0). Invalid argument
#0 0x7f806072969e base::debug::StackTrace::StackTrace()
#1 0x7f806079332f logging::LogMessage::~LogMessage()
#2 0x7f806089106d base::internal::LockImpl::Lock()
#3 0x7f80606f9d23 base::Lock::Acquire()
#4 0x7f80606f99c3 base::AutoLock::AutoLock()
#5 0x7f8060857364 base::SequenceCheckerImpl::CalledOnValidSequence()
#6 0x000000660eba base::Thread::task_runner()
#7 0x0000006606b2 ui::GpuServiceInternal::Initialize()
#8 0x0000006a14f0 ui::mojom::GpuServiceInternalStub::AcceptWithResponder()
...

Original issue's description:
> services/ui: Avoid thread hopping in the gpu process.
>
> Bind the GpuServiceInternal request on the gpu thread, instead of in the
> main thread. This means it is possible to service a request from the host
> without thread hopping between the main and gpu threads.
>
> BUG=643746
>
> Committed: https://crrev.com/9894bae5c1e1f03ab2b14f7b2325296587002a58
> Cr-Commit-Position: refs/heads/master@{#418275}

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

Committed: https://crrev.com/d1cca1e62f682bb2c84a9272c603c4d3aa40047b
Cr-Commit-Position: refs/heads/master@{#418344}
==========

[commit-bot: I haz the power, 2016-09-13 19:54:40]

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