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

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
Committed: https://crrev.com/08f982ad2a3cf6d5b0ce5fc7326e01b3f6073b10
Cr-Original-Commit-Position: refs/heads/master@{#418275}
Cr-Commit-Position: refs/heads/master@{#418468}

[sadrul, 2016-09-09 20:42:20]

The CQ bit was checked by sadrul@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-09 20:42:57]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sadrul, 2016-09-09 20:43:23]

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

[Fady Samuel, 2016-09-09 20:45:47]

Hooray! Thanks for making the world a better place! LGTM

[commit-bot: I haz the power, 2016-09-09 22:00:32]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-09 22:00:33]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[sadrul, 2016-09-12 22:06:13]

The CQ bit was checked by sadrul@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-12 22:06:45]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sadrul, 2016-09-12 22:22:58]

The CQ bit was checked by sadrul@chromium.org

[sadrul, 2016-09-12 22:22:58]

The patchset sent to the CQ was uploaded after l-g-t-m from fsamuel@chromium.org
Link to the patchset: https://codereview.chromium.org/2329043002/#ps20001
(title: "tot merge")

[commit-bot: I haz the power, 2016-09-12 22:23: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, 2016-09-12 23:39:39]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-12 23:39:40]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[sadrul, 2016-09-13 14:31:10]

The CQ bit was checked by sadrul@chromium.org

[commit-bot: I haz the power, 2016-09-13 14:31:31]

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 15:43:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-13 15:44:00]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[sadrul, 2016-09-13 15:46:51]

The CQ bit was checked by sadrul@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-13 15:47:28]

Dry run: 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 16:44:04]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-13 16:44:05]

Dry run: This issue passed the CQ dry run.

[sadrul, 2016-09-13 16:47:01]

The CQ bit was checked by sadrul@chromium.org

[sadrul, 2016-09-13 16:47:02]

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

[commit-bot: I haz the power, 2016-09-13 16:47:20]

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 16:52:25]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-13 16:55:20]

Description was changed from

==========
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
==========

to

==========
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}
==========

[commit-bot: I haz the power, 2016-09-13 16:55:21]

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

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

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/2339653002/ by sadrul@chromium.org.

The reason for reverting is: 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()
....

[sadrul, 2016-09-13 20:03:26]

Description was changed from

==========
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}
==========

to

==========
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}
==========

[sadrul, 2016-09-13 22:37:06]

Patchset #4 (id:60001) has been deleted

[sadrul, 2016-09-14 01:18:03]

The CQ bit was checked by sadrul@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-14 01:18:44]

Dry run: 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-14 02:39:58]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-14 02:39:59]

Dry run: This issue passed the CQ dry run.

[sadrul, 2016-09-14 02:59:41]

The CQ bit was checked by sadrul@chromium.org

[sadrul, 2016-09-14 02:59:42]

The patchset sent to the CQ was uploaded after l-g-t-m from fsamuel@chromium.org
Link to the patchset: https://codereview.chromium.org/2329043002/#ps80001
(title: "fix debug")

[commit-bot: I haz the power, 2016-09-14 03:00:07]

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-14 03:05:11]

Description was changed from

==========
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}
==========

to

==========
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}
==========

[commit-bot: I haz the power, 2016-09-14 03:05:12]

Committed patchset #4 (id:80001)

[commit-bot: I haz the power, 2016-09-14 03:08:03]

Description was changed from

==========
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}
==========

to

==========
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
Committed: https://crrev.com/08f982ad2a3cf6d5b0ce5fc7326e01b3f6073b10
Cr-Original-Commit-Position: refs/heads/master@{#418275}
Cr-Commit-Position: refs/heads/master@{#418468}
==========

[commit-bot: I haz the power, 2016-09-14 03:08:04]

Patchset 4 (id:??) landed as
https://crrev.com/08f982ad2a3cf6d5b0ce5fc7326e01b3f6073b10
Cr-Commit-Position: refs/heads/master@{#418468}