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

Unified Diff: third_party/WebKit/Source/devtools/front_end/main/Main.js

Issue 2837083003: DevTools: create test infrastructure so devtools drives the test (Closed)
Patch Set: rebaseline Created 3 years, 6 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: third_party/WebKit/Source/devtools/front_end/main/Main.js
diff --git a/third_party/WebKit/Source/devtools/front_end/main/Main.js b/third_party/WebKit/Source/devtools/front_end/main/Main.js
index 949c8444908830089abfcc3eeb5d1b9dbf0dfc5e..217e11c7457f8781e86805d8a7aee7ebe9b06811 100644
--- a/third_party/WebKit/Source/devtools/front_end/main/Main.js
+++ b/third_party/WebKit/Source/devtools/front_end/main/Main.js
@@ -47,6 +47,24 @@ Main.Main = class {
SDK.ResourceTreeModel.reloadAllPages(hard);
}
+ /**
+ * @param {string} label
+ */
+ static time(label) {
+ if (Host.isUnderTest())
+ return;
+ console.time(label);
+ }
+
+ /**
+ * @param {string} label
+ */
+ static timeEnd(label) {
+ if (Host.isUnderTest())
+ return;
+ console.timeEnd(label);
+ }
+
_loaded() {
console.timeStamp('Main._loaded');
Runtime.setPlatform(Host.platform());
@@ -149,7 +167,7 @@ Main.Main = class {
* @suppressGlobalPropertiesCheck
*/
_createAppUI() {
- console.time('Main._createAppUI');
+ Main.Main.time('Main._createAppUI');
UI.viewManager = new UI.ViewManager();
@@ -218,7 +236,7 @@ Main.Main = class {
this._registerMessageSinkListener();
self.runtime.extension(Common.AppProvider).instance().then(this._showAppUI.bind(this));
- console.timeEnd('Main._createAppUI');
+ Main.Main.timeEnd('Main._createAppUI');
}
/**
@@ -226,7 +244,7 @@ Main.Main = class {
* @suppressGlobalPropertiesCheck
*/
_showAppUI(appProvider) {
- console.time('Main._showAppUI');
+ Main.Main.time('Main._showAppUI');
var app = /** @type {!Common.AppProvider} */ (appProvider).createApp();
// It is important to kick controller lifetime after apps are instantiated.
Components.dockController.initialize();
@@ -265,17 +283,17 @@ Main.Main = class {
// Allow UI cycles to repaint prior to creating connection.
setTimeout(this._initializeTarget.bind(this), 0);
- console.timeEnd('Main._showAppUI');
+ Main.Main.timeEnd('Main._showAppUI');
}
_initializeTarget() {
- console.time('Main._initializeTarget');
+ Main.Main.time('Main._initializeTarget');
SDK.targetManager.connectToMainTarget(webSocketConnectionLost);
InspectorFrontendHost.readyForTest();
// Asynchronously run the extensions.
setTimeout(this._lateInitialization.bind(this), 100);
- console.timeEnd('Main._initializeTarget');
+ Main.Main.timeEnd('Main._initializeTarget');
function webSocketConnectionLost() {
if (!Main._disconnectedScreenWithReasonWasShown)

Powered by Google App Engine
This is Rietveld 408576698