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

Unified Diff: third_party/WebKit/Source/devtools/front_end/sources/StyleSheetOutlineDialog.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/sources/StyleSheetOutlineDialog.js
diff --git a/third_party/WebKit/Source/devtools/front_end/sources/StyleSheetOutlineDialog.js b/third_party/WebKit/Source/devtools/front_end/sources/StyleSheetOutlineDialog.js
index 1dfad8034a31f7037cd3fddacdc23a193534ea5c..b8a15438122fe4495d2552f9455bf655cd8ef972 100644
--- a/third_party/WebKit/Source/devtools/front_end/sources/StyleSheetOutlineDialog.js
+++ b/third_party/WebKit/Source/devtools/front_end/sources/StyleSheetOutlineDialog.js
@@ -25,97 +25,93 @@
* (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
- * @extends {WebInspector.FilteredListWidget.Delegate}
- * @param {!WebInspector.UISourceCode} uiSourceCode
- * @param {function(number, number)} selectItemCallback
+ * @unrestricted
*/
-WebInspector.StyleSheetOutlineDialog = function(uiSourceCode, selectItemCallback)
-{
- WebInspector.FilteredListWidget.Delegate.call(this, []);
+WebInspector.StyleSheetOutlineDialog = class extends WebInspector.FilteredListWidget.Delegate {
+ /**
+ * @param {!WebInspector.UISourceCode} uiSourceCode
+ * @param {function(number, number)} selectItemCallback
+ */
+ constructor(uiSourceCode, selectItemCallback) {
+ super([]);
this._selectItemCallback = selectItemCallback;
this._cssParser = new WebInspector.CSSParser();
this._cssParser.addEventListener(WebInspector.CSSParser.Events.RulesParsed, this.refresh.bind(this));
this._cssParser.parse(uiSourceCode.workingCopy());
-};
+ }
-/**
- * @param {!WebInspector.UISourceCode} uiSourceCode
- * @param {function(number, number)} selectItemCallback
- */
-WebInspector.StyleSheetOutlineDialog.show = function(uiSourceCode, selectItemCallback)
-{
- WebInspector.StyleSheetOutlineDialog._instanceForTests = new WebInspector.StyleSheetOutlineDialog(uiSourceCode, selectItemCallback);
+ /**
+ * @param {!WebInspector.UISourceCode} uiSourceCode
+ * @param {function(number, number)} selectItemCallback
+ */
+ static show(uiSourceCode, selectItemCallback) {
+ WebInspector.StyleSheetOutlineDialog._instanceForTests =
+ new WebInspector.StyleSheetOutlineDialog(uiSourceCode, selectItemCallback);
new WebInspector.FilteredListWidget(WebInspector.StyleSheetOutlineDialog._instanceForTests).showAsDialog();
-};
-
-WebInspector.StyleSheetOutlineDialog.prototype = {
- /**
- * @override
- * @return {number}
- */
- itemCount: function()
- {
- return this._cssParser.rules().length;
- },
+ }
- /**
- * @override
- * @param {number} itemIndex
- * @return {string}
- */
- itemKeyAt: function(itemIndex)
- {
- var rule = this._cssParser.rules()[itemIndex];
- return rule.selectorText || rule.atRule;
- },
+ /**
+ * @override
+ * @return {number}
+ */
+ itemCount() {
+ return this._cssParser.rules().length;
+ }
- /**
- * @override
- * @param {number} itemIndex
- * @param {string} query
- * @return {number}
- */
- itemScoreAt: function(itemIndex, query)
- {
- var rule = this._cssParser.rules()[itemIndex];
- return -rule.lineNumber;
- },
+ /**
+ * @override
+ * @param {number} itemIndex
+ * @return {string}
+ */
+ itemKeyAt(itemIndex) {
+ var rule = this._cssParser.rules()[itemIndex];
+ return rule.selectorText || rule.atRule;
+ }
- /**
- * @override
- * @param {number} itemIndex
- * @param {string} query
- * @param {!Element} titleElement
- * @param {!Element} subtitleElement
- */
- renderItem: function(itemIndex, query, titleElement, subtitleElement)
- {
- var rule = this._cssParser.rules()[itemIndex];
- titleElement.textContent = rule.selectorText || rule.atRule;
- this.highlightRanges(titleElement, query);
- subtitleElement.textContent = ":" + (rule.lineNumber + 1);
- },
+ /**
+ * @override
+ * @param {number} itemIndex
+ * @param {string} query
+ * @return {number}
+ */
+ itemScoreAt(itemIndex, query) {
+ var rule = this._cssParser.rules()[itemIndex];
+ return -rule.lineNumber;
+ }
- /**
- * @override
- * @param {number} itemIndex
- * @param {string} promptValue
- */
- selectItem: function(itemIndex, promptValue)
- {
- var rule = this._cssParser.rules()[itemIndex];
- var lineNumber = rule.lineNumber;
- if (!isNaN(lineNumber) && lineNumber >= 0)
- this._selectItemCallback(lineNumber, rule.columnNumber);
- },
+ /**
+ * @override
+ * @param {number} itemIndex
+ * @param {string} query
+ * @param {!Element} titleElement
+ * @param {!Element} subtitleElement
+ */
+ renderItem(itemIndex, query, titleElement, subtitleElement) {
+ var rule = this._cssParser.rules()[itemIndex];
+ titleElement.textContent = rule.selectorText || rule.atRule;
+ this.highlightRanges(titleElement, query);
+ subtitleElement.textContent = ':' + (rule.lineNumber + 1);
+ }
- dispose: function()
- {
- this._cssParser.dispose();
- },
+ /**
+ * @override
+ * @param {number} itemIndex
+ * @param {string} promptValue
+ */
+ selectItem(itemIndex, promptValue) {
+ var rule = this._cssParser.rules()[itemIndex];
+ var lineNumber = rule.lineNumber;
+ if (!isNaN(lineNumber) && lineNumber >= 0)
+ this._selectItemCallback(lineNumber, rule.columnNumber);
+ }
- __proto__: WebInspector.FilteredListWidget.Delegate.prototype
+ /**
+ * @override
+ */
+ dispose() {
+ this._cssParser.dispose();
+ }
};
+
+

Powered by Google App Engine
This is Rietveld 408576698