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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: ui/base/latency_info_unittest.cc

Issue 24257011: Revert "Revert 224496 "Move ui/base/latency_info* to ui/events"" (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix gyp bustage Created 7 years, 3 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
« no previous file with comments | « ui/base/latency_info.cc ('k') | ui/events/event.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/base/latency_info_unittest.cc
diff --git a/ui/base/latency_info_unittest.cc b/ui/base/latency_info_unittest.cc
deleted file mode 100644
index bb834787d306756287eab1cb8e35c9311c7a5cb0..0000000000000000000000000000000000000000
--- a/ui/base/latency_info_unittest.cc
+++ /dev/null
@@ -1,142 +0,0 @@
-// Copyright 2013 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "ui/base/latency_info.h"
-
-#include "testing/gtest/include/gtest/gtest.h"
-
-namespace ui {
-
-TEST(LatencyInfoTest, AddTwoSeparateEvent) {
- LatencyInfo info;
- info.AddLatencyNumberWithTimestamp(INPUT_EVENT_LATENCY_RWH_COMPONENT,
- 0,
- 1,
- base::TimeTicks::FromInternalValue(100),
- 1);
- info.AddLatencyNumberWithTimestamp(INPUT_EVENT_LATENCY_ORIGINAL_COMPONENT,
- 1,
- 5,
- base::TimeTicks::FromInternalValue(1000),
- 2);
-
- EXPECT_EQ(info.latency_components.size(), 2u);
- LatencyInfo::LatencyComponent component;
- EXPECT_FALSE(
- info.FindLatency(INPUT_EVENT_LATENCY_UI_COMPONENT, 0, &component));
- EXPECT_FALSE(
- info.FindLatency(INPUT_EVENT_LATENCY_RWH_COMPONENT, 1, &component));
- EXPECT_TRUE(
- info.FindLatency(INPUT_EVENT_LATENCY_RWH_COMPONENT, 0, &component));
- EXPECT_EQ(component.sequence_number, 1);
- EXPECT_EQ(component.event_count, 1u);
- EXPECT_EQ(component.event_time.ToInternalValue(), 100);
- EXPECT_TRUE(
- info.FindLatency(INPUT_EVENT_LATENCY_ORIGINAL_COMPONENT, 1, &component));
- EXPECT_EQ(component.sequence_number, 5);
- EXPECT_EQ(component.event_count, 2u);
- EXPECT_EQ(component.event_time.ToInternalValue(), 1000);
-}
-
-TEST(LatencyInfoTest, AddTwoSameEvent) {
- LatencyInfo info;
- info.AddLatencyNumberWithTimestamp(INPUT_EVENT_LATENCY_RWH_COMPONENT,
- 0,
- 30,
- base::TimeTicks::FromInternalValue(100),
- 2);
- info.AddLatencyNumberWithTimestamp(INPUT_EVENT_LATENCY_RWH_COMPONENT,
- 0,
- 13,
- base::TimeTicks::FromInternalValue(200),
- 3);
-
- EXPECT_EQ(info.latency_components.size(), 1u);
- LatencyInfo::LatencyComponent component;
- EXPECT_FALSE(
- info.FindLatency(INPUT_EVENT_LATENCY_UI_COMPONENT, 0, &component));
- EXPECT_FALSE(
- info.FindLatency(INPUT_EVENT_LATENCY_RWH_COMPONENT, 1, &component));
- EXPECT_TRUE(
- info.FindLatency(INPUT_EVENT_LATENCY_RWH_COMPONENT, 0, &component));
- EXPECT_EQ(component.sequence_number, 30);
- EXPECT_EQ(component.event_count, 5u);
- EXPECT_EQ(component.event_time.ToInternalValue(), (100 * 2 + 200 * 3) / 5);
-}
-
-TEST(LatencyInfoTest, MergeTwoSeparateEvent) {
- LatencyInfo info1;
- LatencyInfo info2;
- info1.AddLatencyNumberWithTimestamp(INPUT_EVENT_LATENCY_RWH_COMPONENT,
- 0,
- 1,
- base::TimeTicks::FromInternalValue(100),
- 1);
- info2.AddLatencyNumberWithTimestamp(INPUT_EVENT_LATENCY_ORIGINAL_COMPONENT,
- 1,
- 5,
- base::TimeTicks::FromInternalValue(1000),
- 2);
- info1.MergeWith(info2);
-
- EXPECT_EQ(info1.latency_components.size(), 2u);
- LatencyInfo::LatencyComponent component;
- EXPECT_FALSE(
- info1.FindLatency(INPUT_EVENT_LATENCY_UI_COMPONENT, 0, &component));
- EXPECT_FALSE(
- info1.FindLatency(INPUT_EVENT_LATENCY_RWH_COMPONENT, 1, &component));
- EXPECT_TRUE(
- info1.FindLatency(INPUT_EVENT_LATENCY_RWH_COMPONENT, 0, &component));
- EXPECT_EQ(component.sequence_number, 1);
- EXPECT_EQ(component.event_count, 1u);
- EXPECT_EQ(component.event_time.ToInternalValue(), 100);
- EXPECT_TRUE(
- info1.FindLatency(INPUT_EVENT_LATENCY_ORIGINAL_COMPONENT, 1, &component));
- EXPECT_EQ(component.sequence_number, 5);
- EXPECT_EQ(component.event_count, 2u);
- EXPECT_EQ(component.event_time.ToInternalValue(), 1000);
-}
-
-TEST(LatencyInfoTest, MergeTwoSameEvent) {
- LatencyInfo info1;
- LatencyInfo info2;
- info1.AddLatencyNumberWithTimestamp(INPUT_EVENT_LATENCY_RWH_COMPONENT,
- 0,
- 30,
- base::TimeTicks::FromInternalValue(100),
- 2);
- info2.AddLatencyNumberWithTimestamp(INPUT_EVENT_LATENCY_RWH_COMPONENT,
- 0,
- 13,
- base::TimeTicks::FromInternalValue(200),
- 3);
- info1.MergeWith(info2);
-
- EXPECT_EQ(info1.latency_components.size(), 1u);
- LatencyInfo::LatencyComponent component;
- EXPECT_FALSE(
- info1.FindLatency(INPUT_EVENT_LATENCY_UI_COMPONENT, 0, &component));
- EXPECT_FALSE(
- info1.FindLatency(INPUT_EVENT_LATENCY_RWH_COMPONENT, 1, &component));
- EXPECT_TRUE(
- info1.FindLatency(INPUT_EVENT_LATENCY_RWH_COMPONENT, 0, &component));
- EXPECT_EQ(component.sequence_number, 30);
- EXPECT_EQ(component.event_count, 5u);
- EXPECT_EQ(component.event_time.ToInternalValue(), (100 * 2 + 200 * 3) / 5);
-}
-
-TEST(LatencyInfoTest, ClearEvents) {
- LatencyInfo info;
- info.AddLatencyNumberWithTimestamp(INPUT_EVENT_LATENCY_RWH_COMPONENT,
- 0,
- 30,
- base::TimeTicks::FromInternalValue(100),
- 2);
-
- EXPECT_EQ(info.latency_components.size(), 1u);
- info.Clear();
- EXPECT_EQ(info.latency_components.size(), 0u);
-}
-
-} // namespace ui
« no previous file with comments | « ui/base/latency_info.cc ('k') | ui/events/event.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698