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

Unified Diff: content/common/gpu/texture_image_transport_surface.cc

Issue 11293121: Add latency measurement system (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: create cc::LatencyInfo Created 7 years, 12 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
« no previous file with comments | « content/common/gpu/texture_image_transport_surface.h ('k') | content/common/view_messages.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/common/gpu/texture_image_transport_surface.cc
diff --git a/content/common/gpu/texture_image_transport_surface.cc b/content/common/gpu/texture_image_transport_surface.cc
index 5ec5e2e2d0b9e20c163188ac238b47fef9ff81df..f55d2cb5d5fcd837fe28d776b575ad175a0cf6f0 100644
--- a/content/common/gpu/texture_image_transport_surface.cc
+++ b/content/common/gpu/texture_image_transport_surface.cc
@@ -202,8 +202,25 @@ void TextureImageTransportSurface::OnWillDestroyStub(
stub_destroyed_ = true;
}
+void TextureImageTransportSurface::SetLatencyInfo(
+ const cc::LatencyInfo& latency_info) {
+ if (latency_info_.rendererImplFrameNumber !=
+ latency_info.rendererImplFrameNumber) {
+ TRACE_EVENT_ASYNC_BEGIN2("frame", "Renderer Frame", this,
+ "frame_number",
+ latency_info.rendererImplFrameNumber,
+ "input_number", latency_info.inputNumber);
+ }
+
+ latency_info_ = latency_info;
+}
+
bool TextureImageTransportSurface::SwapBuffers() {
DCHECK(backbuffer_suggested_allocation_);
+ TRACE_EVENT_ASYNC_END2("frame", "Renderer Frame", this,
+ "frame_number", latency_info_.rendererImplFrameNumber,
+ "input_number", latency_info_.inputNumber);
+
if (!frontbuffer_suggested_allocation_)
return true;
@@ -221,6 +238,7 @@ bool TextureImageTransportSurface::SwapBuffers() {
// so we do not leak a texture in the mailbox.
AddRef();
+ params.latency_info = latency_info_;
helper_->SendAcceleratedSurfaceBuffersSwapped(params);
DCHECK(!is_swap_buffers_pending_);
@@ -231,6 +249,9 @@ bool TextureImageTransportSurface::SwapBuffers() {
bool TextureImageTransportSurface::PostSubBuffer(
int x, int y, int width, int height) {
DCHECK(backbuffer_suggested_allocation_);
+ TRACE_EVENT_ASYNC_END2("frame", "Renderer Frame", this,
+ "frame_number", latency_info_.rendererImplFrameNumber,
+ "input_number", latency_info_.inputNumber);
if (!frontbuffer_suggested_allocation_)
return true;
const gfx::Rect new_damage_rect(x, y, width, height);
@@ -258,6 +279,7 @@ bool TextureImageTransportSurface::PostSubBuffer(
// so we do not leak a texture in the mailbox.
AddRef();
+ params.latency_info = latency_info_;
helper_->SendAcceleratedSurfacePostSubBuffer(params);
DCHECK(!is_swap_buffers_pending_);
« no previous file with comments | « content/common/gpu/texture_image_transport_surface.h ('k') | content/common/view_messages.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698