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

Unified Diff: third_party/WebKit/Source/devtools/front_end/ui/ZoomManager.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/ui/ZoomManager.js
diff --git a/third_party/WebKit/Source/devtools/front_end/ui/ZoomManager.js b/third_party/WebKit/Source/devtools/front_end/ui/ZoomManager.js
index e619fed2babb31bae70e54ff3a3c2879a20ecba5..7840514fd459fabd637b1aeacef8d879f56c841c 100644
--- a/third_party/WebKit/Source/devtools/front_end/ui/ZoomManager.js
+++ b/third_party/WebKit/Source/devtools/front_end/ui/ZoomManager.js
@@ -1,61 +1,56 @@
// Copyright 2014 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
- * @extends {WebInspector.Object}
- * @param {!Window} window
- * @param {!InspectorFrontendHostAPI} frontendHost
+ * @unrestricted
*/
-WebInspector.ZoomManager = function(window, frontendHost)
-{
+WebInspector.ZoomManager = class extends WebInspector.Object {
+ /**
+ * @param {!Window} window
+ * @param {!InspectorFrontendHostAPI} frontendHost
+ */
+ constructor(window, frontendHost) {
+ super();
this._frontendHost = frontendHost;
this._zoomFactor = this._frontendHost.zoomFactor();
- window.addEventListener("resize", this._onWindowResize.bind(this), true);
+ window.addEventListener('resize', this._onWindowResize.bind(this), true);
+ }
+
+ /**
+ * @return {number}
+ */
+ zoomFactor() {
+ return this._zoomFactor;
+ }
+
+ /**
+ * @param {number} value
+ * @return {number}
+ */
+ cssToDIP(value) {
+ return value * this._zoomFactor;
+ }
+
+ /**
+ * @param {number} valueDIP
+ * @return {number}
+ */
+ dipToCSS(valueDIP) {
+ return valueDIP / this._zoomFactor;
+ }
+
+ _onWindowResize() {
+ var oldZoomFactor = this._zoomFactor;
+ this._zoomFactor = this._frontendHost.zoomFactor();
+ if (oldZoomFactor !== this._zoomFactor)
+ this.dispatchEventToListeners(
+ WebInspector.ZoomManager.Events.ZoomChanged, {from: oldZoomFactor, to: this._zoomFactor});
+ }
};
/** @enum {symbol} */
WebInspector.ZoomManager.Events = {
- ZoomChanged: Symbol("ZoomChanged")
-};
-
-WebInspector.ZoomManager.prototype = {
- /**
- * @return {number}
- */
- zoomFactor: function()
- {
- return this._zoomFactor;
- },
-
- /**
- * @param {number} value
- * @return {number}
- */
- cssToDIP: function(value)
- {
- return value * this._zoomFactor;
- },
-
- /**
- * @param {number} valueDIP
- * @return {number}
- */
- dipToCSS: function(valueDIP)
- {
- return valueDIP / this._zoomFactor;
- },
-
- _onWindowResize: function()
- {
- var oldZoomFactor = this._zoomFactor;
- this._zoomFactor = this._frontendHost.zoomFactor();
- if (oldZoomFactor !== this._zoomFactor)
- this.dispatchEventToListeners(WebInspector.ZoomManager.Events.ZoomChanged, {from: oldZoomFactor, to: this._zoomFactor});
- },
-
- __proto__: WebInspector.Object.prototype
+ ZoomChanged: Symbol('ZoomChanged')
};
/**

Powered by Google App Engine
This is Rietveld 408576698