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

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: Rebased and addressed feedback from jamesr@ 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 06aa4dd1a62230a234903d7d2d8f8b2aa1b2a322..34432b7ff1d1ecf1522c2652ac80685fc4c38389 100644
--- a/content/browser/renderer_host/render_widget_host_impl.cc
+++ b/content/browser/renderer_host/render_widget_host_impl.cc
@@ -1498,6 +1498,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.FixMissingComponentIds(GetLatencyComponentId());
view_->AcceleratedSurfaceBuffersSwapped(gpu_params,
params.gpu_process_host_id);
view_->DidReceiveRendererFrame();
@@ -1513,6 +1514,11 @@ bool RenderWidgetHostImpl::OnSwapCompositorFrame(
uint32 output_surface_id = param.a;
param.b.AssignTo(frame.get());
+ for (size_t i = 0; i < frame->metadata.latency_info.size(); i++) {
+ frame->metadata.latency_info[i].FixMissingComponentIds(
+ GetLatencyComponentId());
+ }
+
input_router_->OnViewUpdated(
GetInputRouterViewFlagsFromCompositorFrameMetadata(frame->metadata));
@@ -1677,9 +1683,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.FixMissingComponentIds(GetLatencyComponentId());
+ 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