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

Unified Diff: tools/telemetry/telemetry/core/backends/chrome/tracing_backend_unittest.py

Issue 144423003: Make timeline model able to return browser process (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: get browser process through thread "CrBrowserMain" Created 6 years, 11 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: tools/telemetry/telemetry/core/backends/chrome/tracing_backend_unittest.py
diff --git a/tools/telemetry/telemetry/core/backends/chrome/tracing_backend_unittest.py b/tools/telemetry/telemetry/core/backends/chrome/tracing_backend_unittest.py
index c9a182fb1f49460a0c953be218ccb213c8458ce6..6ba91ce8ecd6f66d1b46393211e7f7fdaf912258 100644
--- a/tools/telemetry/telemetry/core/backends/chrome/tracing_backend_unittest.py
+++ b/tools/telemetry/telemetry/core/backends/chrome/tracing_backend_unittest.py
@@ -133,3 +133,14 @@ class ChromeTraceResultTest(unittest.TestCase):
assert 'traceEvents' in j
self.assertEquals(j['traceEvents'],
['foo', 'bar', 'baz'])
+
+ def testBrowserProcess(self):
+ ri = self._chromeTraceResultClass([
+ '{"name": "process_name",'
+ '"args": {"name": "Browser"},'
+ '"pid": 5, "ph": "M"}',
+ '{"name": "thread_name",'
+ '"args": {"name": "CrBrowserMain"},'
+ '"pid": 5, "tid": 32578, "ph": "M"}'])
+ model = ri.AsTimelineModel()
+ self.assertEquals(model.browser_process.pid, 5)

Powered by Google App Engine
This is Rietveld 408576698