Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(602)

Side by Side Diff: ppapi/proxy/ppapi_command_buffer_proxy.cc

Issue 2727573003: gpu: Add sync token dependencies to flush metadata. (Closed)
Patch Set: jbauman's review Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #include "ppapi/proxy/ppapi_command_buffer_proxy.h" 5 #include "ppapi/proxy/ppapi_command_buffer_proxy.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/numerics/safe_conversions.h" 9 #include "base/numerics/safe_conversions.h"
10 #include "ppapi/proxy/ppapi_messages.h" 10 #include "ppapi/proxy/ppapi_messages.h"
(...skipping 196 matching lines...) Expand 10 before | Expand all | Expand 10 after
207 bool PpapiCommandBufferProxy::IsFenceSyncReleased(uint64_t release) { 207 bool PpapiCommandBufferProxy::IsFenceSyncReleased(uint64_t release) {
208 NOTIMPLEMENTED(); 208 NOTIMPLEMENTED();
209 return false; 209 return false;
210 } 210 }
211 211
212 void PpapiCommandBufferProxy::SignalSyncToken(const gpu::SyncToken& sync_token, 212 void PpapiCommandBufferProxy::SignalSyncToken(const gpu::SyncToken& sync_token,
213 const base::Closure& callback) { 213 const base::Closure& callback) {
214 NOTIMPLEMENTED(); 214 NOTIMPLEMENTED();
215 } 215 }
216 216
217 void PpapiCommandBufferProxy::WaitSyncToken(const gpu::SyncToken& sync_token) {}
dcheng 2017/03/14 06:53:03 Is this a NOTIMPLEMENTED() kind of thing as well?
sunnyps 2017/03/15 00:03:08 No, pepper can wait on sync tokens and GLES2Implem
piman 2017/03/15 00:30:14 We could add a TODO to forward this to the rendere
sunnyps 2017/03/15 00:56:12 Done.
218
217 bool PpapiCommandBufferProxy::CanWaitUnverifiedSyncToken( 219 bool PpapiCommandBufferProxy::CanWaitUnverifiedSyncToken(
218 const gpu::SyncToken* sync_token) { 220 const gpu::SyncToken& sync_token) {
219 return false; 221 return false;
220 } 222 }
221 223
222 int32_t PpapiCommandBufferProxy::GetExtraCommandBufferData() const { 224 int32_t PpapiCommandBufferProxy::GetExtraCommandBufferData() const {
223 return 0; 225 return 0;
224 } 226 }
225 227
226 void PpapiCommandBufferProxy::SignalQuery(uint32_t query, 228 void PpapiCommandBufferProxy::SignalQuery(uint32_t query,
227 const base::Closure& callback) { 229 const base::Closure& callback) {
228 NOTREACHED(); 230 NOTREACHED();
(...skipping 75 matching lines...) Expand 10 before | Expand all | Expand 10 after
304 message->set_unblock(true); 306 message->set_unblock(true);
305 Send(message); 307 Send(message);
306 308
307 flush_info_->flush_pending = false; 309 flush_info_->flush_pending = false;
308 flush_info_->resource.SetHostResource(0, 0); 310 flush_info_->resource.SetHostResource(0, 0);
309 flushed_fence_sync_release_ = pending_fence_sync_release_; 311 flushed_fence_sync_release_ = pending_fence_sync_release_;
310 } 312 }
311 313
312 } // namespace proxy 314 } // namespace proxy
313 } // namespace ppapi 315 } // namespace ppapi
OLDNEW
« gpu/ipc/client/gpu_channel_host.cc ('K') | « ppapi/proxy/ppapi_command_buffer_proxy.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698