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

Unified Diff: content/browser/renderer_host/render_widget_host_impl.cc

Issue 140663003: Removed requirement for the renderer to know it's process ID (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed Windows 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/renderer_host/render_widget_host_impl.cc
diff --git a/content/browser/renderer_host/render_widget_host_impl.cc b/content/browser/renderer_host/render_widget_host_impl.cc
index f506e75b7c644d043d2ad00c553be1df08acc4c0..f2249169230517f153ec99cf159b384a9eaf3503 100644
--- a/content/browser/renderer_host/render_widget_host_impl.cc
+++ b/content/browser/renderer_host/render_widget_host_impl.cc
@@ -1478,6 +1478,7 @@ void RenderWidgetHostImpl::OnCompositorSurfaceBuffersSwapped(
gpu_params.size = params.size;
gpu_params.scale_factor = params.scale_factor;
gpu_params.latency_info = params.latency_info;
+ gpu_params.latency_info.AnnotateComponentsWithProcessId(process_->GetID());
view_->AcceleratedSurfaceBuffersSwapped(gpu_params,
params.gpu_process_host_id);
view_->DidReceiveRendererFrame();
@@ -1493,6 +1494,9 @@ bool RenderWidgetHostImpl::OnSwapCompositorFrame(
uint32 output_surface_id = param.a;
param.b.AssignTo(frame.get());
+ ui::LatencyInfo::AnnotateWithProcessId(frame->metadata.latency_info,
+ process_->GetID());
+
bool fixed_page_scale =
frame->metadata.min_page_scale_factor ==
frame->metadata.max_page_scale_factor;
@@ -1661,9 +1665,17 @@ void RenderWidgetHostImpl::DidUpdateBackingStore(
// Now paint the view. Watch out: it might be destroyed already.
if (view_ && !is_accelerated_compositing_active_) {
+
+ std::vector<ui::LatencyInfo> latency_info;
+ for (size_t i = 0; i < params.latency_info.size(); i++) {
+ ui::LatencyInfo info = params.latency_info[i];
+ info.AnnotateComponentsWithProcessId(process_->GetID());
+ latency_info.push_back(info);
+ }
+
view_being_painted_ = true;
view_->DidUpdateBackingStore(params.scroll_rect, params.scroll_delta,
- params.copy_rects, params.latency_info);
+ params.copy_rects, latency_info);
view_->DidReceiveRendererFrame();
view_being_painted_ = false;
}

Powered by Google App Engine
This is Rietveld 408576698