[Devtools] Cleanup DataGrid.js

third_party/WebKit/Source/devtools/front_end/ui_lazy/ViewportDataGrid.js

 // 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.DataGrid}
  * @param {!Array.<!WebInspector.DataGrid.ColumnDescriptor>} columnsArray
  * @param {function(!WebInspector.DataGridNode, string, string, string)=} editCallback
  * @param {function(!WebInspector.DataGridNode)=} deleteCallback
@@ skipping 225 matching lines @@
                     this._hiddenWheelTarget = oldNode.abandonElement();
                 else
                     element.remove();
                 oldNode.wasDetached();
             }
         }
 
         var previousElement = this._topFillerRow;
         if (previousElement.nextSibling === this._hiddenWheelTarget)
             previousElement = this._hiddenWheelTarget;
-        var tBody = this.dataTableBody;
+        var tBody = this._dataTableBody;

[lushnikov, 2016/10/24 23:34:55]

this is a file-private field; you should expose it to the descendant via public
or protected getter method on the DataGrid

[allada, 2016/10/24 23:49:11]

Done.

         var offset = viewportState.offset;
         for (var i = 0; i < visibleNodes.length; ++i) {
             var node = visibleNodes[i];
             var element = node.element();
             node.willAttach();
             element.classList.toggle("odd", (offset + i) % 2 === 0);
             tBody.insertBefore(element, previousElement.nextSibling);
             previousElement = element;
         }
 
@@ skipping 219 matching lines @@
         return result;
     },
 
     reveal: function()
     {
         this.dataGrid._revealViewportNode(this);
     },
 
     __proto__: WebInspector.DataGridNode.prototype
 }