.
Created attachment 382439 [details] Patch
Comment on attachment 382439 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=382439&action=review r=me > Source/WebInspectorUI/UserInterface/Views/ComputedStyleSection.js:112 > + set propertyVisibilityMode(propertyVisibilityMode) This is kind of wordy. I think the context is clear, how about just naming the parameter `mode`?
Comment on attachment 382439 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=382439&action=review >> Source/WebInspectorUI/UserInterface/Views/ComputedStyleSection.js:112 >> + set propertyVisibilityMode(propertyVisibilityMode) > > This is kind of wordy. I think the context is clear, how about just naming the parameter `mode`? This is just putting back what was removed in r251038. Also, `WI.SpreadsheetCSSStyleDeclarationSection` has a similarly named concept.
Comment on attachment 382439 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=382439&action=review >>> Source/WebInspectorUI/UserInterface/Views/ComputedStyleSection.js:112 >>> + set propertyVisibilityMode(propertyVisibilityMode) >> >> This is kind of wordy. I think the context is clear, how about just naming the parameter `mode`? > > This is just putting back what was removed in r251038. Also, `WI.SpreadsheetCSSStyleDeclarationSection` has a similarly named concept. Okie dokie!
Comment on attachment 382439 [details] Patch Clearing flags on attachment: 382439 Committed r251883: <https://trac.webkit.org/changeset/251883>
All reviewed patches have been landed. Closing bug.
<rdar://problem/56796756>