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

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

Issue 2466123002: DevTools: reformat front-end code to match chromium style. (Closed)
Patch Set: all done Created 4 years, 1 month 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/audits2_worker/Audits2Service.js
diff --git a/third_party/WebKit/Source/devtools/front_end/audits2_worker/Audits2Service.js b/third_party/WebKit/Source/devtools/front_end/audits2_worker/Audits2Service.js
index 1aa11408500c670c79639ce0799d11ed2a49df20..4ed23ba06ec11e115ac294e82009e04ab0196521 100644
--- a/third_party/WebKit/Source/devtools/front_end/audits2_worker/Audits2Service.js
+++ b/third_party/WebKit/Source/devtools/front_end/audits2_worker/Audits2Service.js
@@ -1,136 +1,126 @@
// Copyright (c) 2016 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.
-
/**
* @interface
*/
class LighthousePort {
- /**
- * @param {!string} eventName, 'message', 'close'
- * @param {function(string|undefined)} cb
- */
- on(eventName, cb) { }
+ /**
+ * @param {!string} eventName, 'message', 'close'
+ * @param {function(string|undefined)} cb
+ */
+ on(eventName, cb) {
+ }
- /**
- * @param {string} message
- */
- send(message) { }
+ /**
+ * @param {string} message
+ */
+ send(message) {
+ }
- close() { }
+ close() {
+ }
}
/**
- * @constructor
* @implements {LighthousePort}
* @implements {Service}
+ * @unrestricted
*/
-function Audits2Service()
-{
-}
+var Audits2Service = class {
+ /**
+ * @override
+ * @param {function(string)}
+ */
+ setNotify(notify) {
+ this._notify = notify;
+ }
-Audits2Service.prototype = {
- /**
- * @override
- * @param {function(string)}
- */
- setNotify: function(notify)
- {
- this._notify = notify;
- },
+ /**
+ * @return {!Promise}
+ */
+ start() {
+ return window.runLighthouseInWorker(this, 'https://www.webkit.org', {flags: {mobile: true}}, [
+ 'is-on-https',
+ 'redirects-http',
+ 'service-worker',
+ 'works-offline',
+ 'viewport',
+ 'manifest-display',
+ 'without-javascript',
+ 'first-meaningful-paint',
+ 'speed-index-metric',
+ 'estimated-input-latency',
+ 'time-to-interactive',
+ 'user-timings',
+ 'screenshots',
+ 'critical-request-chains',
+ 'manifest-exists',
+ 'manifest-background-color',
+ 'manifest-theme-color',
+ 'manifest-icons-min-192',
+ 'manifest-icons-min-144',
+ 'manifest-name',
+ 'manifest-short-name',
+ 'manifest-short-name-length',
+ 'manifest-start-url',
+ 'meta-theme-color',
+ 'aria-valid-attr',
+ 'aria-allowed-attr',
+ 'color-contrast',
+ 'image-alt',
+ 'label',
+ 'tabindex',
+ 'content-width',
+ 'geolocation-on-start'
+ ]);
+ }
- /**
- * @return {!Promise}
- */
- start: function()
- {
- return window.runLighthouseInWorker(this, "https://www.webkit.org", { flags: { mobile: true } }, [
- "is-on-https",
- "redirects-http",
- "service-worker",
- "works-offline",
- "viewport",
- "manifest-display",
- "without-javascript",
- "first-meaningful-paint",
- "speed-index-metric",
- "estimated-input-latency",
- "time-to-interactive",
- "user-timings",
- "screenshots",
- "critical-request-chains",
- "manifest-exists",
- "manifest-background-color",
- "manifest-theme-color",
- "manifest-icons-min-192",
- "manifest-icons-min-144",
- "manifest-name",
- "manifest-short-name",
- "manifest-short-name-length",
- "manifest-start-url",
- "meta-theme-color",
- "aria-valid-attr",
- "aria-allowed-attr",
- "color-contrast",
- "image-alt",
- "label",
- "tabindex",
- "content-width",
- "geolocation-on-start"
- ]);
- },
+ /**
+ * @return {!Promise}
+ */
+ stop() {
+ this._onClose();
+ return Promise.resolve();
+ }
- /**
- * @return {!Promise}
- */
- stop: function()
- {
- this._onClose();
- return Promise.resolve();
- },
+ /**
+ * @param {!Object=} params
+ * @return {!Promise}
+ */
+ dispatchProtocolMessage(params) {
+ this._onMessage(params['message']);
+ return Promise.resolve();
+ }
- /**
- * @param {!Object=} params
- * @return {!Promise}
- */
- dispatchProtocolMessage: function(params)
- {
- this._onMessage(params["message"]);
- return Promise.resolve();
- },
+ /**
+ * @override
+ * @return {!Promise}
+ */
+ dispose() {
+ return Promise.resolve();
+ }
- /**
- * @override
- * @return {!Promise}
- */
- dispose: function()
- {
- return Promise.resolve();
- },
+ /**
+ * @param {string} message
+ */
+ send(message) {
+ this._notify('sendProtocolMessage', {message: message});
+ }
- /**
- * @param {string} message
- */
- send: function(message)
- {
- this._notify("sendProtocolMessage", { message: message });
- },
+ close() {
+ }
- close: function()
- {
- },
-
- /**
- * @param {string} eventName
- * @param {function(string|undefined)} cb
- */
- on: function(eventName, cb)
- {
- if (eventName === "message")
- this._onMessage = cb;
- if (eventName === "close")
- this._onClose = cb;
- }
+ /**
+ * @param {string} eventName
+ * @param {function(string|undefined)} cb
+ */
+ on(eventName, cb) {
+ if (eventName === 'message')
+ this._onMessage = cb;
+ if (eventName === 'close')
+ this._onClose = cb;
+ }
};
// Make lighthouse happy.
@@ -138,4 +128,6 @@ global = self;
global.isVinn = true;
global.document = {};
global.document.documentElement = {};
-global.document.documentElement.style = { WebkitAppearance: "WebkitAppearance"};
+global.document.documentElement.style = {
+ WebkitAppearance: 'WebkitAppearance'
+};

Powered by Google App Engine
This is Rietveld 408576698