DevTools: Add JSDoc for static methods, fix JSDoc types and induced errors

Source/devtools/front_end/ui/treeoutline.js

 /*
  * Copyright (C) 2007 Apple Inc.  All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 397 matching lines @@
 TreeOutline.prototype.expand = function()
 {
     // this is the root, do nothing
 }
 
 TreeOutline.prototype.collapse = function()
 {
     // this is the root, do nothing
 }
 
+/**
+ * @return {boolean}
+ */
 TreeOutline.prototype.revealed = function()
 {
     return true;
 }
 
 TreeOutline.prototype.reveal = function()
 {
     // this is the root, do nothing
 }
 
@@ skipping 330 matching lines @@
         maxDepth = 3;
 
     while (item) {
         if (depth < maxDepth)
             item.expand();
         item = item.traverseNextTreeElement(false, this, (depth >= maxDepth), info);
         depth += info.depthChange;
     }
 }
 
+/**
+ * @param {?TreeElement} ancestor
+ * @return {boolean}
+ */
 TreeElement.prototype.hasAncestor = function(ancestor) {
     if (!ancestor)
         return false;
 
     var currentNode = this.parent;
     while (currentNode) {
         if (ancestor === currentNode)
             return true;
         currentNode = currentNode.parent;
     }
 
     return false;
 }
 
 TreeElement.prototype.reveal = function()
 {
     var currentAncestor = this.parent;
     while (currentAncestor && !currentAncestor.root) {
         if (!currentAncestor.expanded)
             currentAncestor.expand();
         currentAncestor = currentAncestor.parent;
     }
 
     this.onreveal();
 }
 
+/**
+ * @return {boolean}
+ */
 TreeElement.prototype.revealed = function()
 {
     var currentAncestor = this.parent;
     while (currentAncestor && !currentAncestor.root) {
         if (!currentAncestor.expanded)
             return false;
         currentAncestor = currentAncestor.parent;
     }
 
     return true;
@@ skipping 37 matching lines @@
  * @param {boolean=} omitFocus
  */
 TreeElement.prototype.revealAndSelect = function(omitFocus)
 {
     this.reveal();
     this.select(omitFocus);
 }
 
 /**
  * @param {boolean=} supressOnDeselect
+ * @return {boolean}
  */
 TreeElement.prototype.deselect = function(supressOnDeselect)
 {
     if (!this.treeOutline || this.treeOutline.selectedTreeElement !== this || !this.selected)
         return false;
 
     this.selected = false;
     this.treeOutline.selectedTreeElement = null;
     if (this._listItemNode)
         this._listItemNode.classList.remove("selected");
@@ skipping 99 matching lines @@
 
     if (element)
         return element;
 
     if (!this.parent || this.parent.root)
         return null;
 
     return this.parent;
 }
 
+/**
+ * @return {boolean}
+ */
 TreeElement.prototype.isEventWithinDisclosureTriangle = function(event)
 {
     // FIXME: We should not use getComputedStyle(). For that we need to get rid of using ::before for disclosure triangle. (http://webk.it/74446)
     var paddingLeftValue = window.getComputedStyle(this._listItemNode).getPropertyCSSValue("padding-left");
     var computedLeftPadding = paddingLeftValue ? paddingLeftValue.getFloatValue(CSSPrimitiveValue.CSS_PX) : 0;
     var left = this._listItemNode.totalOffsetLeft() + computedLeftPadding;
     return event.pageX >= left && event.pageX <= left + this.arrowToggleWidth && this.hasChildren;
 }