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

Unified Diff: third_party/WebKit/Source/devtools/front_end/common/Worker.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/common/Worker.js
diff --git a/third_party/WebKit/Source/devtools/front_end/common/Worker.js b/third_party/WebKit/Source/devtools/front_end/common/Worker.js
index 262a7ef8a98ba51f01007213e1433030113608af..a019734bc71c96f7b94362a33954e6c876cf4af0 100644
--- a/third_party/WebKit/Source/devtools/front_end/common/Worker.js
+++ b/third_party/WebKit/Source/devtools/front_end/common/Worker.js
@@ -27,75 +27,69 @@
* (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*/
-
/**
- * @constructor
- * @param {string} appName
+ * @unrestricted
*/
-WebInspector.Worker = function(appName)
-{
- var url = appName + ".js";
- var remoteBase = Runtime.queryParam("remoteBase");
+WebInspector.Worker = class {
+ /**
+ * @param {string} appName
+ */
+ constructor(appName) {
+ var url = appName + '.js';
+ var remoteBase = Runtime.queryParam('remoteBase');
if (remoteBase)
- url += "?remoteBase=" + remoteBase;
+ url += '?remoteBase=' + remoteBase;
/** @type {!Promise<!Worker>} */
this._workerPromise = new Promise(fulfill => {
- this._worker = new Worker(url);
- this._worker.onmessage = onMessage.bind(this);
+ this._worker = new Worker(url);
+ this._worker.onmessage = onMessage.bind(this);
- /**
- * @param {!Event} event
- * @this {WebInspector.Worker}
- */
- function onMessage(event)
- {
- console.assert(event.data === "workerReady");
- this._worker.onmessage = null;
- fulfill(this._worker);
- // No need to hold a reference to worker anymore as it's stored in
- // the resolved promise.
- this._worker = null;
- }
+ /**
+ * @param {!Event} event
+ * @this {WebInspector.Worker}
+ */
+ function onMessage(event) {
+ console.assert(event.data === 'workerReady');
+ this._worker.onmessage = null;
+ fulfill(this._worker);
+ // No need to hold a reference to worker anymore as it's stored in
+ // the resolved promise.
+ this._worker = null;
+ }
});
-};
+ }
-WebInspector.Worker.prototype = {
- /**
- * @param {*} message
- */
- postMessage: function(message)
- {
- this._workerPromise.then(worker => {
- if (!this._disposed)
- worker.postMessage(message);
- });
- },
+ /**
+ * @param {*} message
+ */
+ postMessage(message) {
+ this._workerPromise.then(worker => {
+ if (!this._disposed)
+ worker.postMessage(message);
+ });
+ }
- dispose: function()
- {
- this._disposed = true;
- this._workerPromise.then(worker => worker.terminate());
- },
+ dispose() {
+ this._disposed = true;
+ this._workerPromise.then(worker => worker.terminate());
+ }
- terminate: function()
- {
- this.dispose();
- },
+ terminate() {
+ this.dispose();
+ }
- /**
- * @param {?function(!MessageEvent<*>)} listener
- */
- set onmessage(listener)
- {
- this._workerPromise.then(worker => worker.onmessage = listener);
- },
+ /**
+ * @param {?function(!MessageEvent<*>)} listener
+ */
+ set onmessage(listener) {
+ this._workerPromise.then(worker => worker.onmessage = listener);
+ }
- /**
- * @param {?function(!Event)} listener
- */
- set onerror(listener)
- {
- this._workerPromise.then(worker => worker.onerror = listener);
- }
+ /**
+ * @param {?function(!Event)} listener
+ */
+ set onerror(listener) {
+ this._workerPromise.then(worker => worker.onerror = listener);
+ }
};

Powered by Google App Engine
This is Rietveld 408576698