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

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

Issue 15682010: Support plumbing LatencyInfo through the old software path. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 6 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_view_gtk.cc
diff --git a/content/browser/renderer_host/render_widget_host_view_gtk.cc b/content/browser/renderer_host/render_widget_host_view_gtk.cc
index 5279631d53246a0591be181f6fa575ef86ca8765..240d9092e1418269ecbebed3b0730ea4f8be9ea7 100644
--- a/content/browser/renderer_host/render_widget_host_view_gtk.cc
+++ b/content/browser/renderer_host/render_widget_host_view_gtk.cc
@@ -853,8 +853,10 @@ void RenderWidgetHostViewGtk::ImeCompositionRangeChanged(
void RenderWidgetHostViewGtk::DidUpdateBackingStore(
const gfx::Rect& scroll_rect,
const gfx::Vector2d& scroll_delta,
- const std::vector<gfx::Rect>& copy_rects) {
+ const std::vector<gfx::Rect>& copy_rects,
+ const ui::LatencyInfo& latency_info) {
TRACE_EVENT0("ui::gtk", "RenderWidgetHostViewGtk::DidUpdateBackingStore");
+ software_latency_info_.MergeWith(latency_info);
if (is_hidden_)
return;
@@ -1211,6 +1213,9 @@ void RenderWidgetHostViewGtk::Paint(const gfx::Rect& damage_rect) {
// recorded.
web_contents_switch_paint_time_ = base::TimeTicks();
}
+ software_latency_info_.swap_timestamp = base::TimeTicks::HighResNow();
+ render_widget_host->FrameSwapped(software_latency_info_);
piman 2013/06/07 02:51:30 So compared to the hardware path, this can somewha
jbauman 2013/06/07 07:28:24 Well, even in the hardware path we don't know when
+ software_latency_info_.Clear();
} else {
if (window)
gdk_window_clear(window);

Powered by Google App Engine
This is Rietveld 408576698