Chromium Code Reviews Chromium Code Reviews
Issue 1348283004:
Use MailboxManagerSync on android. (Closed)
Base URL: https://github.com/domokit/mojo.git@master| OLD | NEW |
|---|---|
| 1 // Copyright 2015 The Chromium Authors. All rights reserved. | 1 // Copyright 2015 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 // | 4 // |
| 5 | 5 |
| 6 #include "services/gles2/gpu_state.h" | 6 #include "services/gles2/gpu_state.h" |
| 7 | 7 |
| 8 #include "gpu/command_buffer/service/mailbox_manager_impl.h" | |
| 9 #include "gpu/command_buffer/service/mailbox_manager_sync.h" | |
| 10 | |
| 8 namespace gles2 { | 11 namespace gles2 { |
| 9 | 12 |
| 10 GpuState::GpuState() | 13 GpuState::GpuState() |
| 11 : control_thread_("gpu_command_buffer_control"), | 14 : control_thread_("gpu_command_buffer_control"), |
| 12 sync_point_manager_(gpu::SyncPointManager::Create(true)), | 15 sync_point_manager_(gpu::SyncPointManager::Create(true)), |
| 13 share_group_(new gfx::GLShareGroup), | 16 share_group_(new gfx::GLShareGroup), |
| 14 mailbox_manager_(new gpu::gles2::MailboxManagerImpl) { | 17 // TODO: use Impl on linux, Sync on android |
| 18 // mailbox_manager_(new gpu::gles2::MailboxManagerImpl) { | |
|
jamesr
2015/09/17 22:23:13
can you just leave a todo and not the commented-ou
cdotstout
2015/09/17 22:31:09
Yes, Linux fails with MailboxManagerSync.
If th
| |
| 19 mailbox_manager_(new gpu::gles2::MailboxManagerSync) { | |
| 15 control_thread_.Start(); | 20 control_thread_.Start(); |
| 16 } | 21 } |
| 17 | 22 |
| 18 GpuState::~GpuState() { | 23 GpuState::~GpuState() { |
| 19 } | 24 } |
| 20 | 25 |
| 21 } // namespace gles2 | 26 } // namespace gles2 |
| OLD | NEW |
