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

Unified Diff: third_party/WebKit/Source/devtools/front_end/main/OverlayController.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/main/OverlayController.js
diff --git a/third_party/WebKit/Source/devtools/front_end/main/OverlayController.js b/third_party/WebKit/Source/devtools/front_end/main/OverlayController.js
index 95a35349717984554b533db49a498dbd67bc61bb..85cca5c74e748c677a7652b334514560f25a84d0 100644
--- a/third_party/WebKit/Source/devtools/front_end/main/OverlayController.js
+++ b/third_party/WebKit/Source/devtools/front_end/main/OverlayController.js
@@ -1,43 +1,45 @@
// Copyright 2015 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.
-
/**
- * @constructor
+ * @unrestricted
*/
-WebInspector.OverlayController = function()
-{
- WebInspector.moduleSetting("disablePausedStateOverlay").addChangeListener(this._updateAllOverlays, this);
- WebInspector.targetManager.addModelListener(WebInspector.DebuggerModel, WebInspector.DebuggerModel.Events.DebuggerPaused, this._updateOverlay, this);
- WebInspector.targetManager.addModelListener(WebInspector.DebuggerModel, WebInspector.DebuggerModel.Events.DebuggerResumed, this._updateOverlay, this);
+WebInspector.OverlayController = class {
+ constructor() {
+ WebInspector.moduleSetting('disablePausedStateOverlay').addChangeListener(this._updateAllOverlays, this);
+ WebInspector.targetManager.addModelListener(
+ WebInspector.DebuggerModel, WebInspector.DebuggerModel.Events.DebuggerPaused, this._updateOverlay, this);
+ WebInspector.targetManager.addModelListener(
+ WebInspector.DebuggerModel, WebInspector.DebuggerModel.Events.DebuggerResumed, this._updateOverlay, this);
// TODO(dgozman): we should get DebuggerResumed on navigations instead of listening to GlobalObjectCleared.
- WebInspector.targetManager.addModelListener(WebInspector.DebuggerModel, WebInspector.DebuggerModel.Events.GlobalObjectCleared, this._updateOverlay, this);
- WebInspector.targetManager.addEventListener(WebInspector.TargetManager.Events.SuspendStateChanged, this._updateAllOverlays, this);
-};
+ WebInspector.targetManager.addModelListener(
+ WebInspector.DebuggerModel, WebInspector.DebuggerModel.Events.GlobalObjectCleared, this._updateOverlay, this);
+ WebInspector.targetManager.addEventListener(
+ WebInspector.TargetManager.Events.SuspendStateChanged, this._updateAllOverlays, this);
+ }
-WebInspector.OverlayController.prototype = {
- _updateAllOverlays: function()
- {
- for (var target of WebInspector.targetManager.targets(WebInspector.Target.Capability.Browser))
- this._updateTargetOverlay(/** @type {!WebInspector.DebuggerModel} */ (WebInspector.DebuggerModel.fromTarget(target)));
- },
+ _updateAllOverlays() {
+ for (var target of WebInspector.targetManager.targets(WebInspector.Target.Capability.Browser))
+ this._updateTargetOverlay(
+ /** @type {!WebInspector.DebuggerModel} */ (WebInspector.DebuggerModel.fromTarget(target)));
+ }
- /**
- * @param {!WebInspector.Event} event
- */
- _updateOverlay: function(event)
- {
- this._updateTargetOverlay(/** @type {!WebInspector.DebuggerModel} */ (event.target));
- },
+ /**
+ * @param {!WebInspector.Event} event
+ */
+ _updateOverlay(event) {
+ this._updateTargetOverlay(/** @type {!WebInspector.DebuggerModel} */ (event.target));
+ }
- /**
- * @param {!WebInspector.DebuggerModel} debuggerModel
- */
- _updateTargetOverlay: function(debuggerModel)
- {
- if (!debuggerModel.target().hasBrowserCapability())
- return;
- var message = debuggerModel.isPaused() && !WebInspector.moduleSetting("disablePausedStateOverlay").get() ? WebInspector.UIString("Paused in debugger") : undefined;
- debuggerModel.target().pageAgent().configureOverlay(WebInspector.targetManager.allTargetsSuspended(), message);
- }
+ /**
+ * @param {!WebInspector.DebuggerModel} debuggerModel
+ */
+ _updateTargetOverlay(debuggerModel) {
+ if (!debuggerModel.target().hasBrowserCapability())
+ return;
+ var message = debuggerModel.isPaused() && !WebInspector.moduleSetting('disablePausedStateOverlay').get() ?
+ WebInspector.UIString('Paused in debugger') :
+ undefined;
+ debuggerModel.target().pageAgent().configureOverlay(WebInspector.targetManager.allTargetsSuspended(), message);
+ }
};

Powered by Google App Engine
This is Rietveld 408576698