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

Unified Diff: content/browser/renderer_host/render_widget_host_view_mac.mm

Issue 2914023002: Remove LatencyInfo::sequence_number. (May break metrics).
Patch Set: Rebase Created 3 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_mac.mm
diff --git a/content/browser/renderer_host/render_widget_host_view_mac.mm b/content/browser/renderer_host/render_widget_host_view_mac.mm
index a4d10aa0b52bda08d79876ec1d7e9e382882653a..24c00b19aa27d7eabf847eaa1a2b65cab34a367a 100644
--- a/content/browser/renderer_host/render_widget_host_view_mac.mm
+++ b/content/browser/renderer_host/render_widget_host_view_mac.mm
@@ -751,7 +751,7 @@ void RenderWidgetHostViewMac::Show() {
ui::LatencyInfo renderer_latency_info;
renderer_latency_info.AddLatencyNumber(
- ui::TAB_SHOW_COMPONENT, render_widget_host_->GetLatencyComponentId(), 0);
+ ui::TAB_SHOW_COMPONENT, render_widget_host_->GetLatencyComponentId());
render_widget_host_->WasShown(renderer_latency_info);
// If there is not a frame being currently drawn, kick one, so that the below
@@ -1954,7 +1954,7 @@ void RenderWidgetHostViewMac::OnDisplayMetricsChanged(
enterEvent.SetType(WebInputEvent::kMouseMove);
enterEvent.button = WebMouseEvent::Button::kNoButton;
ui::LatencyInfo latency_info(ui::SourceEventType::OTHER);
- latency_info.AddLatencyNumber(ui::INPUT_EVENT_LATENCY_UI_COMPONENT, 0, 0);
+ latency_info.AddLatencyNumber(ui::INPUT_EVENT_LATENCY_UI_COMPONENT, 0);
if (renderWidgetHostView_->ShouldRouteEvent(enterEvent)) {
renderWidgetHostView_->render_widget_host_->delegate()
->GetInputEventRouter()
@@ -1991,7 +1991,7 @@ void RenderWidgetHostViewMac::OnDisplayMetricsChanged(
WebMouseEvent event =
WebMouseEventBuilder::Build(theEvent, self, pointerType_);
ui::LatencyInfo latency_info(ui::SourceEventType::OTHER);
- latency_info.AddLatencyNumber(ui::INPUT_EVENT_LATENCY_UI_COMPONENT, 0, 0);
+ latency_info.AddLatencyNumber(ui::INPUT_EVENT_LATENCY_UI_COMPONENT, 0);
if (renderWidgetHostView_->ShouldRouteEvent(event)) {
renderWidgetHostView_->render_widget_host_->delegate()
->GetInputEventRouter()
@@ -2112,7 +2112,7 @@ void RenderWidgetHostViewMac::OnDisplayMetricsChanged(
latency_info.set_source_event_type(ui::SourceEventType::KEY_PRESS);
}
- latency_info.AddLatencyNumber(ui::INPUT_EVENT_LATENCY_UI_COMPONENT, 0, 0);
+ latency_info.AddLatencyNumber(ui::INPUT_EVENT_LATENCY_UI_COMPONENT, 0);
// Force fullscreen windows to close on Escape so they won't keep the keyboard
// grabbed or be stuck onscreen if the renderer is hanging.
@@ -2350,7 +2350,7 @@ void RenderWidgetHostViewMac::OnDisplayMetricsChanged(
.AddPhaseIfNeededAndScheduleEndEvent(webEvent, false);
} else {
ui::LatencyInfo latency_info(ui::SourceEventType::WHEEL);
- latency_info.AddLatencyNumber(ui::INPUT_EVENT_LATENCY_UI_COMPONENT, 0, 0);
+ latency_info.AddLatencyNumber(ui::INPUT_EVENT_LATENCY_UI_COMPONENT, 0);
renderWidgetHostView_->render_widget_host_
->ForwardWheelEventWithLatencyInfo(webEvent, latency_info);
}
@@ -2558,7 +2558,7 @@ void RenderWidgetHostViewMac::OnDisplayMetricsChanged(
WebMouseWheelEvent webEvent = WebMouseWheelEventBuilder::Build(event, self);
webEvent.rails_mode = mouseWheelFilter_.UpdateRailsMode(webEvent);
ui::LatencyInfo latency_info(ui::SourceEventType::WHEEL);
- latency_info.AddLatencyNumber(ui::INPUT_EVENT_LATENCY_UI_COMPONENT, 0, 0);
+ latency_info.AddLatencyNumber(ui::INPUT_EVENT_LATENCY_UI_COMPONENT, 0);
if (renderWidgetHostView_->wheel_scroll_latching_enabled()) {
renderWidgetHostView_->mouse_wheel_phase_handler_
.AddPhaseIfNeededAndScheduleEndEvent(

Powered by Google App Engine
This is Rietveld 408576698