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

Side by Side Diff: third_party/WebKit/Source/devtools/front_end/elements/PropertyChangeHighlighter.js

Issue 2440953003: DevTools: use semicolons after each statement. (Closed)
Patch Set: rebaseline 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2015 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /** 5 /**
6 * @constructor 6 * @constructor
7 * @param {!WebInspector.StylesSidebarPane} ssp 7 * @param {!WebInspector.StylesSidebarPane} ssp
8 * @param {!WebInspector.CSSModel} cssModel 8 * @param {!WebInspector.CSSModel} cssModel
9 * @param {!CSSAgent.StyleSheetId} styleSheetId 9 * @param {!CSSAgent.StyleSheetId} styleSheetId
10 * @param {!WebInspector.TextRange} range 10 * @param {!WebInspector.TextRange} range
11 */ 11 */
12 WebInspector.PropertyChangeHighlighter = function(ssp, cssModel, styleSheetId, r ange) 12 WebInspector.PropertyChangeHighlighter = function(ssp, cssModel, styleSheetId, r ange)
13 { 13 {
14 this._styleSidebarPane = ssp; 14 this._styleSidebarPane = ssp;
15 this._target = cssModel.target(); 15 this._target = cssModel.target();
16 this._styleSheetId = styleSheetId; 16 this._styleSheetId = styleSheetId;
17 this._range = range; 17 this._range = range;
18 } 18 };
19 19
20 WebInspector.PropertyChangeHighlighter.prototype = { 20 WebInspector.PropertyChangeHighlighter.prototype = {
21 /** 21 /**
22 */ 22 */
23 perform: function() 23 perform: function()
24 { 24 {
25 var node = this._styleSidebarPane.node(); 25 var node = this._styleSidebarPane.node();
26 if (!node || this._target !== node.target()) 26 if (!node || this._target !== node.target())
27 return; 27 return;
28 28
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
82 * @return {boolean} 82 * @return {boolean}
83 */ 83 */
84 _checkRanges: function(outterRange, innerRange) 84 _checkRanges: function(outterRange, innerRange)
85 { 85 {
86 var startsBefore = outterRange.startLine < innerRange.startLine || (outt erRange.startLine === innerRange.startLine && outterRange.startColumn <= innerRa nge.startColumn); 86 var startsBefore = outterRange.startLine < innerRange.startLine || (outt erRange.startLine === innerRange.startLine && outterRange.startColumn <= innerRa nge.startColumn);
87 // SSP might be outdated, so inner range will a bit bigger than outter. E.g.; "padding-left: 9px" -> "padding-left: 10px" 87 // SSP might be outdated, so inner range will a bit bigger than outter. E.g.; "padding-left: 9px" -> "padding-left: 10px"
88 var eps = 5; 88 var eps = 5;
89 var endsAfter = outterRange.endLine > innerRange.endLine || (outterRange .endLine === innerRange.endLine && outterRange.endColumn + eps >= innerRange.end Column); 89 var endsAfter = outterRange.endLine > innerRange.endLine || (outterRange .endLine === innerRange.endLine && outterRange.endColumn + eps >= innerRange.end Column);
90 return startsBefore && endsAfter; 90 return startsBefore && endsAfter;
91 } 91 }
92 } 92 };
93 93
94 /** 94 /**
95 * @constructor 95 * @constructor
96 * @param {!WebInspector.StylesSidebarPane} ssp 96 * @param {!WebInspector.StylesSidebarPane} ssp
97 * @param {!WebInspector.CSSProperty} cssProperty 97 * @param {!WebInspector.CSSProperty} cssProperty
98 */ 98 */
99 WebInspector.PropertyRevealHighlighter = function(ssp, cssProperty) 99 WebInspector.PropertyRevealHighlighter = function(ssp, cssProperty)
100 { 100 {
101 this._styleSidebarPane = ssp; 101 this._styleSidebarPane = ssp;
102 this._cssProperty = cssProperty; 102 this._cssProperty = cssProperty;
103 } 103 };
104 104
105 WebInspector.PropertyRevealHighlighter.prototype = { 105 WebInspector.PropertyRevealHighlighter.prototype = {
106 perform: function() 106 perform: function()
107 { 107 {
108 // Expand all shorthands. 108 // Expand all shorthands.
109 for (var section of this._styleSidebarPane.allSections()) { 109 for (var section of this._styleSidebarPane.allSections()) {
110 for (var treeElement = section.propertiesTreeOutline.firstChild(); t reeElement; treeElement = treeElement.nextSibling) 110 for (var treeElement = section.propertiesTreeOutline.firstChild(); t reeElement; treeElement = treeElement.nextSibling)
111 treeElement.onpopulate(); 111 treeElement.onpopulate();
112 } 112 }
113 var highlightTreeElement = null; 113 var highlightTreeElement = null;
(...skipping 14 matching lines...) Expand all
128 return; 128 return;
129 129
130 highlightTreeElement.parent.expand(); 130 highlightTreeElement.parent.expand();
131 highlightTreeElement.listItemElement.scrollIntoViewIfNeeded(); 131 highlightTreeElement.listItemElement.scrollIntoViewIfNeeded();
132 highlightTreeElement.listItemElement.animate([ 132 highlightTreeElement.listItemElement.animate([
133 { offset: 0, backgroundColor: "rgba(255, 255, 0, 0.2)"}, 133 { offset: 0, backgroundColor: "rgba(255, 255, 0, 0.2)"},
134 { offset: 0.1, backgroundColor: "rgba(255, 255, 0, 0.7)"}, 134 { offset: 0.1, backgroundColor: "rgba(255, 255, 0, 0.7)"},
135 { offset: 1, backgroundColor: "transparent"} 135 { offset: 1, backgroundColor: "transparent"}
136 ], { duration : 2000, easing: "cubic-bezier(0, 0, 0.2, 1)" }); 136 ], { duration : 2000, easing: "cubic-bezier(0, 0, 0.2, 1)" });
137 }, 137 },
138 } 138 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698