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

Unified Diff: ppapi/proxy/compositor_layer_resource.cc

Issue 1427543002: Modified old wait sync point functions to also accept new sync tokens. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix mock gpu video accelerator factory Created 5 years, 2 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 side-by-side diff with in-line comments
Download patch
Index: ppapi/proxy/compositor_layer_resource.cc
diff --git a/ppapi/proxy/compositor_layer_resource.cc b/ppapi/proxy/compositor_layer_resource.cc
index c42776eb2935a9e2b367abaca49f6e86f59e5e0b..2822f368e2415ed3dfdb5e86e09871034f071970 100644
--- a/ppapi/proxy/compositor_layer_resource.cc
+++ b/ppapi/proxy/compositor_layer_resource.cc
@@ -10,6 +10,7 @@
#include "gpu/GLES2/gl2extchromium.h"
#include "gpu/command_buffer/client/gles2_implementation.h"
#include "gpu/command_buffer/common/mailbox.h"
+#include "gpu/command_buffer/common/sync_token.h"
#include "ppapi/proxy/compositor_resource.h"
#include "ppapi/shared_impl/ppb_graphics_3d_shared.h"
#include "ppapi/thunk/enter.h"
@@ -30,14 +31,14 @@ float clamp(float value) {
return std::min(std::max(value, 0.0f), 1.0f);
}
-void OnTextureReleased(
- const ScopedPPResource& layer,
- const ScopedPPResource& context,
- uint32_t texture,
- const scoped_refptr<TrackedCallback>& release_callback,
- int32_t result,
- uint32_t sync_point,
- bool is_lost) {
+void OnTextureReleased(const ScopedPPResource& layer,
+ const ScopedPPResource& context,
+ uint32_t texture,
+ const scoped_refptr<TrackedCallback>& release_callback,
+ int32_t result,
+ uint32_t sync_point,
+ const gpu::SyncToken& sync_token,
+ bool is_lost) {
if (!TrackedCallback::IsPending(release_callback))
return;
@@ -47,7 +48,7 @@ void OnTextureReleased(
}
do {
- if (!sync_point)
+ if (!sync_point && !sync_token.HasData())
break;
EnterResourceNoLock<PPB_Graphics3D_API> enter(context.get(), true);
@@ -58,19 +59,19 @@ void OnTextureReleased(
static_cast<PPB_Graphics3D_Shared*>(enter.object());
GLES2Implementation* gl = graphics->gles2_impl();
- gl->WaitSyncPointCHROMIUM(sync_point);
+ gl->WaitSyncPointCHROMIUM(sync_point, sync_token.GetConstData());
} while (false);
release_callback->Run(is_lost ? PP_ERROR_FAILED : PP_OK);
}
-void OnImageReleased(
- const ScopedPPResource& layer,
- const ScopedPPResource& image,
- const scoped_refptr<TrackedCallback>& release_callback,
- int32_t result,
- uint32_t sync_point,
- bool is_lost) {
+void OnImageReleased(const ScopedPPResource& layer,
+ const ScopedPPResource& image,
+ const scoped_refptr<TrackedCallback>& release_callback,
+ int32_t result,
+ uint32_t sync_point,
+ const gpu::SyncToken& sync_token,
+ bool is_lost) {
if (!TrackedCallback::IsPending(release_callback))
return;
release_callback->Run(result);

Powered by Google App Engine
This is Rietveld 408576698