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

Unified Diff: tools/perf/benchmarks/blink_perf_unittest.py

Issue 2819343002: Support tracing metrics for measureTime & measureFrameTime method in blink_perf (Closed)
Patch Set: Created 3 years, 8 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
« tools/perf/benchmarks/blink_perf.py ('K') | « tools/perf/benchmarks/blink_perf.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/perf/benchmarks/blink_perf_unittest.py
diff --git a/tools/perf/benchmarks/blink_perf_unittest.py b/tools/perf/benchmarks/blink_perf_unittest.py
new file mode 100644
index 0000000000000000000000000000000000000000..3d93604797ca039f0d075ae88d86f14245b12a2d
--- /dev/null
+++ b/tools/perf/benchmarks/blink_perf_unittest.py
@@ -0,0 +1,70 @@
+# Copyright 2017 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.
+import os
+
+from telemetry import story
+from telemetry.page import page as page_module
+from telemetry.testing import options_for_unittests
+from telemetry.testing import page_test_test_case
+
+from benchmarks import blink_perf
+
+
+class BlinkPerfTest(page_test_test_case.PageTestTestCase):
+ _BLINK_PERF_TEST_DATA_DIR = os.path.join(os.path.dirname(__file__),
+ '..', '..', '..', 'third_party', 'WebKit', 'PerformanceTests',
+ 'TestData')
+
+ _BLINK_PERF_RESOURCES_DIR = os.path.join(os.path.dirname(__file__),
+ '..', '..', '..', 'third_party', 'WebKit', 'PerformanceTests',
+ 'resources')
+ def setUp(self):
+ self._options = options_for_unittests.GetCopy()
+ # pylint: disable=protected-access
+ self._measurement = blink_perf._BlinkPerfMeasurement()
+ # pylint: enable=protected-access
+
+ def _CreateStorySetForTestFile(self, test_file_name):
+ story_set = story.StorySet(base_dir=self._BLINK_PERF_TEST_DATA_DIR,
+ serving_dirs={self._BLINK_PERF_TEST_DATA_DIR,
+ self._BLINK_PERF_RESOURCES_DIR})
+ page = page_module.Page('file://' + test_file_name, story_set,
+ base_dir=story_set.base_dir)
+ story_set.AddStory(page)
+ return story_set
+
+ def testBlinkPerfTracingMetricsForMeasureTime(self):
+ results = self.RunMeasurement(measurement=self._measurement,
+ ps=self._CreateStorySetForTestFile('append-child-measure-time.html'),
+ options=self._options)
+ self.assertFalse(results.failures)
+ self.assertEquals(len(results.FindAllTraceValues()), 1)
+
+ frame_view_layouts = results.FindAllPageSpecificValuesNamed(
+ 'FrameView::layout')
+ self.assertEquals(1, len(frame_view_layouts))
+ self.assertGreater(frame_view_layouts[0].GetRepresentativeNumber, 0.1)
+
+ update_layout_trees = results.FindAllPageSpecificValuesNamed(
+ 'UpdateLayoutTree')
+ self.assertEquals(1, len(update_layout_trees))
+ self.assertGreater(update_layout_trees[0].GetRepresentativeNumber, 0.1)
+
+ def testBlinkPerfTracingMetricsForMeasureFrameTime(self):
+ results = self.RunMeasurement(measurement=self._measurement,
+ ps=self._CreateStorySetForTestFile(
+ 'color-changes-measure-frame-time.html'),
+ options=self._options)
+ self.assertFalse(results.failures)
+ self.assertEquals(len(results.FindAllTraceValues()), 1)
+
+ frame_view_prepaints = results.FindAllPageSpecificValuesNamed(
+ 'FrameView::prePaint')
+ self.assertEquals(1, len(frame_view_prepaints))
+ self.assertGreater(frame_view_prepaints[0].GetRepresentativeNumber, 0.1)
+
+ frame_view_painttrees = results.FindAllPageSpecificValuesNamed(
+ 'FrameView::paintTree')
+ self.assertEquals(1, len(frame_view_painttrees))
+ self.assertGreater(frame_view_painttrees[0].GetRepresentativeNumber, 0.1)
« tools/perf/benchmarks/blink_perf.py ('K') | « tools/perf/benchmarks/blink_perf.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698