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

Unified Diff: content/browser/renderer_host/input/render_widget_host_latency_tracker.cc

Issue 1244443003: android: Use a hard coded BeginFrame deadline (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 5 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/input/render_widget_host_latency_tracker.cc
diff --git a/content/browser/renderer_host/input/render_widget_host_latency_tracker.cc b/content/browser/renderer_host/input/render_widget_host_latency_tracker.cc
index 2ca7f74da3878b796e488a34f54fe1c2cf7952dd..c7d1c6dff13af9bffff82e9ce3e34a55198b4956 100644
--- a/content/browser/renderer_host/input/render_widget_host_latency_tracker.cc
+++ b/content/browser/renderer_host/input/render_widget_host_latency_tracker.cc
@@ -19,9 +19,6 @@ namespace content {
namespace {
const uint32 kMaxInputCoordinates = LatencyInfo::kMaxInputCoordinates;
-const size_t kBrowserCompositeLatencyHistorySize = 60;
-const double kBrowserCompositeLatencyEstimationPercentile = 90.0;
-const double kBrowserCompositeLatencyEstimationSlack = 1.1;
void UpdateLatencyCoordinatesImpl(const blink::WebTouchEvent& touch,
LatencyInfo* latency) {
@@ -291,8 +288,7 @@ RenderWidgetHostLatencyTracker::RenderWidgetHostLatencyTracker()
: last_event_id_(0),
latency_component_id_(0),
device_scale_factor_(1),
- has_seent_first_gesture_scroll_update_(false),
- browser_composite_latency_history_(kBrowserCompositeLatencyHistorySize) {
+ has_seent_first_gesture_scroll_update_(false) {
}
RenderWidgetHostLatencyTracker::~RenderWidgetHostLatencyTracker() {
@@ -444,27 +440,6 @@ void RenderWidgetHostLatencyTracker::OnFrameSwapped(
ComputeScrollLatencyHistograms(gpu_swap_begin_component,
gpu_swap_end_component, latency_component_id_,
latency);
-
- LatencyInfo::LatencyComponent browser_swap_component;
- if (latency.FindLatency(
- ui::INPUT_EVENT_BROWSER_RECEIVED_RENDERER_SWAP_COMPONENT, 0,
- &browser_swap_component)) {
- base::TimeDelta delta =
- gpu_swap_begin_component.event_time - browser_swap_component.event_time;
- browser_composite_latency_history_.InsertSample(delta);
- }
-}
-
-base::TimeDelta
-RenderWidgetHostLatencyTracker::GetEstimatedBrowserCompositeTime() const {
- // TODO(brianderson): Remove lower bound on estimate once we're sure it won't
- // cause regressions.
- return std::max(
- browser_composite_latency_history_.Percentile(
- kBrowserCompositeLatencyEstimationPercentile) *
- kBrowserCompositeLatencyEstimationSlack,
- base::TimeDelta::FromMicroseconds(
- (1.0f * base::Time::kMicrosecondsPerSecond) / (3.0f * 60)));
}
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698