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

Side by Side Diff: third_party/WebKit/Source/devtools/front_end/audits2_worker/Audits2Service.js

Issue 2861053003: DevTools: [lighthouse] Implement performance metrics filmstrip (Closed)
Patch Set: Created 3 years, 7 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2016 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 /** 4 /**
5 * @interface 5 * @interface
6 */ 6 */
7 class LighthousePort { 7 class LighthousePort {
8 /** 8 /**
9 * @param {!string} eventName, 'message', 'close' 9 * @param {!string} eventName, 'message', 'close'
10 * @param {function(string|undefined)} cb 10 * @param {function(string|undefined)} cb
(...skipping 27 matching lines...) Expand all
38 38
39 /** 39 /**
40 * @return {!Promise<!ReportRenderer.ReportJSON>} 40 * @return {!Promise<!ReportRenderer.ReportJSON>}
41 */ 41 */
42 start(params) { 42 start(params) {
43 self.listenForStatus(message => { 43 self.listenForStatus(message => {
44 this.statusUpdate(message[1]); 44 this.statusUpdate(message[1]);
45 }); 45 });
46 46
47 return Promise.resolve() 47 return Promise.resolve()
48 .then(_ => self.runLighthouseInWorker(this, params.url, undefined, params. categoryIDs)) 48 .then(_ => self.runLighthouseInWorker(this, params.url, undefined, param s.categoryIDs))
49 .then(/** @type {!ReportRenderer.ReportJSON} */ result => result) 49 .then(/** @type {!ReportRenderer.ReportJSON} */ result => {
50 .catchException(null); 50 // Filter out artifacts except for screenshots in traces to minimize r eport size.
51 var traces = result['artifacts']['traces'];
52 for (var pass in traces) {
53 traces[pass]['traceEvents'] =
54 traces[pass]['traceEvents'].filter(e => e['cat'] === 'disabled-b y-default-devtools.screenshot');
55 }
56 result['artifacts'] = {traces: traces};
57 return result;
58 })
59 .catchException(null);
51 } 60 }
52 61
53 /** 62 /**
54 * @return {!Promise} 63 * @return {!Promise}
55 */ 64 */
56 stop() { 65 stop() {
57 this.close(); 66 this.close();
58 return Promise.resolve(); 67 return Promise.resolve();
59 } 68 }
60 69
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
105 }; 114 };
106 115
107 // Make lighthouse and traceviewer happy. 116 // Make lighthouse and traceviewer happy.
108 global = self; 117 global = self;
109 global.isVinn = true; 118 global.isVinn = true;
110 global.document = {}; 119 global.document = {};
111 global.document.documentElement = {}; 120 global.document.documentElement = {};
112 global.document.documentElement.style = { 121 global.document.documentElement.style = {
113 WebkitAppearance: 'WebkitAppearance' 122 WebkitAppearance: 'WebkitAppearance'
114 }; 123 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698