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

Unified Diff: content/browser/gpu/gpu_process_host.cc

Issue 123563002: Remove gpu side LatencyInfo merging (Closed) Base URL: http://git.chromium.org/chromium/src.git@gpu-per-event-latency-6-small
Patch Set: fix mac_rel compiler error. Move kMaxLatencyInfoNumber to .cc file Created 6 years, 11 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: content/browser/gpu/gpu_process_host.cc
diff --git a/content/browser/gpu/gpu_process_host.cc b/content/browser/gpu/gpu_process_host.cc
index 9e5861e3e6fe446bd48f8d29612263c436683fec..ae95f48a5c26747120ad17ee4d1d78b35ced7ed3 100644
--- a/content/browser/gpu/gpu_process_host.cc
+++ b/content/browser/gpu/gpu_process_host.cc
@@ -132,7 +132,7 @@ void AcceleratedSurfaceBuffersSwappedCompletedForRenderer(
int surface_id,
base::TimeTicks timebase,
base::TimeDelta interval,
- const ui::LatencyInfo& latency_info) {
+ const std::vector<ui::LatencyInfo>& latency_info) {
if (!BrowserThread::CurrentlyOn(BrowserThread::UI)) {
BrowserThread::PostTask(
BrowserThread::UI,
@@ -156,7 +156,8 @@ void AcceleratedSurfaceBuffersSwappedCompletedForRenderer(
RenderWidgetHostImpl::From(rwh)->AcknowledgeSwapBuffersToRenderer();
if (interval != base::TimeDelta())
RenderWidgetHostImpl::From(rwh)->UpdateVSyncParameters(timebase, interval);
- RenderWidgetHostImpl::From(rwh)->FrameSwapped(latency_info);
+ for (size_t i = 0; i < latency_info.size(); i++)
+ RenderWidgetHostImpl::From(rwh)->FrameSwapped(latency_info[i]);
RenderWidgetHostImpl::From(rwh)->DidReceiveRendererFrame();
}
@@ -167,7 +168,7 @@ void AcceleratedSurfaceBuffersSwappedCompleted(
bool alive,
base::TimeTicks timebase,
base::TimeDelta interval,
- const ui::LatencyInfo& latency_info) {
+ const std::vector<ui::LatencyInfo>& latency_info) {
AcceleratedSurfaceBuffersSwappedCompletedForGPU(
host_id, route_id, alive, timebase, interval);
AcceleratedSurfaceBuffersSwappedCompletedForRenderer(
@@ -877,6 +878,10 @@ void GpuProcessHost::OnAcceleratedSurfaceBuffersSwapped(
const GpuHostMsg_AcceleratedSurfaceBuffersSwapped_Params& params) {
TRACE_EVENT0("gpu", "GpuProcessHost::OnAcceleratedSurfaceBuffersSwapped");
+ if (!ui::LatencyInfo::Verify(params.latency_info,
+ "GpuHostMsg_AcceleratedSurfaceBuffersSwapped"))
+ return;
+
gfx::GLSurfaceHandle surface_handle =
GpuSurfaceTracker::Get()->GetSurfaceHandle(params.surface_id);
// Compositor window is always gfx::kNullPluginWindow.
@@ -929,10 +934,15 @@ void GpuProcessHost::OnAcceleratedSurfaceBuffersSwapped(
const GpuHostMsg_AcceleratedSurfaceBuffersSwapped_Params& params) {
TRACE_EVENT0("gpu", "GpuProcessHost::OnAcceleratedSurfaceBuffersSwapped");
+ if (!ui::LatencyInfo::Verify(params.latency_info,
+ "GpuHostMsg_AcceleratedSurfaceBuffersSwapped"))
+ return;
+
base::ScopedClosureRunner scoped_completion_runner(
base::Bind(&AcceleratedSurfaceBuffersSwappedCompleted,
host_id_, params.route_id, params.surface_id,
- true, base::TimeTicks(), base::TimeDelta(), ui::LatencyInfo()));
+ true, base::TimeTicks(), base::TimeDelta(),
+ std::vector<ui::LatencyInfo>()));
gfx::GLSurfaceHandle handle =
GpuSurfaceTracker::Get()->GetSurfaceHandle(params.surface_id);
@@ -999,6 +1009,10 @@ void GpuProcessHost::OnAcceleratedSurfacePostSubBuffer(
const GpuHostMsg_AcceleratedSurfacePostSubBuffer_Params& params) {
TRACE_EVENT0("gpu", "GpuProcessHost::OnAcceleratedSurfacePostSubBuffer");
+ if (!ui::LatencyInfo::Verify(params.latency_info,
+ "GpuHostMsg_AcceleratedSurfacePostSubBuffer"))
+ return;
+
NOTIMPLEMENTED();
}
« no previous file with comments | « content/browser/aura/software_browser_compositor_output_surface.cc ('k') | content/browser/gpu/gpu_process_host_ui_shim.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698