Changeset 101313 in webkit

Timestamp:

Nov 28, 2011, 6:34:05 PM (15 years ago)

Author:

timothy@apple.com

Message:

Use classList instead of addStyleClass and removeStyleClass in TreeOutline.

​https://webkit.org/b/72803

Reviewed by Dan Bernstein.

• inspector/front-end/treeoutline.js:

(TreeOutline.prototype.appendChild):
(TreeOutline.prototype.insertChild):
(TreeElement.prototype.set hasChildren):
(TreeElement.prototype.set hidden):
(TreeElement.prototype._attach):
(TreeElement.prototype.collapse):
(TreeElement.prototype.expand):
(TreeElement.prototype.select):
(TreeElement.prototype.deselect):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• inspector/front-end/treeoutline.js (modified) (11 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-11-28  Timothy Hatcher  <timothy@apple.com>
+
+        Use classList instead of addStyleClass and removeStyleClass in TreeOutline.
+
+        https://webkit.org/b/72803
+
+        Reviewed by Dan Bernstein.
+
+        * inspector/front-end/treeoutline.js:
+        (TreeOutline.prototype.appendChild):
+        (TreeOutline.prototype.insertChild):
+        (TreeElement.prototype.set hasChildren):
+        (TreeElement.prototype.set hidden):
+        (TreeElement.prototype._attach):
+        (TreeElement.prototype.collapse):
+        (TreeElement.prototype.expand):
+        (TreeElement.prototype.select):
+        (TreeElement.prototype.deselect):
+
 2011-11-28  Leo Yang  <leo.yang@torchmobile.com.cn>
 

trunk/Source/WebCore/inspector/front-end/treeoutline.js

@@ -87 +87 @@
         this._childrenListNode = this.treeOutline._childrenListNode.ownerDocument.createElement("ol");
         this._childrenListNode.parentTreeElement = this;
-        this._childrenListNode.addStyleClass("children");
+        this._childrenListNode.classList.add("children");
         if (this.hidden)
-            this._childrenListNode.addStyleClass("hidden");
+            this._childrenListNode.classList.add("hidden");
     }
 
@@ -135 +135 @@
         this._childrenListNode = this.treeOutline._childrenListNode.ownerDocument.createElement("ol");
         this._childrenListNode.parentTreeElement = this;
-        this._childrenListNode.addStyleClass("children");
+        this._childrenListNode.classList.add("children");
         if (this.hidden)
-            this._childrenListNode.addStyleClass("hidden");
+            this._childrenListNode.classList.add("hidden");
     }
 
@@ -534 +534 @@
 
         if (x)
-            this._listItemNode.addStyleClass("parent");
+            this._listItemNode.classList.add("parent");
         else {
-            this._listItemNode.removeStyleClass("parent");
+            this._listItemNode.classList.remove("parent");
             this.collapse();
         }
@@ -553 +553 @@
         if (x) {
             if (this._listItemNode)
-                this._listItemNode.addStyleClass("hidden");
+                this._listItemNode.classList.add("hidden");
             if (this._childrenListNode)
-                this._childrenListNode.addStyleClass("hidden");
+                this._childrenListNode.classList.add("hidden");
         } else {
             if (this._listItemNode)
-                this._listItemNode.removeStyleClass("hidden");
+                this._listItemNode.classList.remove("hidden");
             if (this._childrenListNode)
-                this._childrenListNode.removeStyleClass("hidden");
+                this._childrenListNode.classList.remove("hidden");
         }
     },
@@ -613 +613 @@
 
         if (this.hidden)
-            this._listItemNode.addStyleClass("hidden");
+            this._listItemNode.classList.add("hidden");
         if (this.hasChildren)
-            this._listItemNode.addStyleClass("parent");
+            this._listItemNode.classList.add("parent");
         if (this.expanded)
-            this._listItemNode.addStyleClass("expanded");
+            this._listItemNode.classList.add("expanded");
         if (this.selected)
-            this._listItemNode.addStyleClass("selected");
+            this._listItemNode.classList.add("selected");
 
         this._listItemNode.addEventListener("mousedown", TreeElement.treeElementMouseDown, false);
@@ -701 +701 @@
 {
     if (this._listItemNode)
-        this._listItemNode.removeStyleClass("expanded");
+        this._listItemNode.classList.remove("expanded");
     if (this._childrenListNode)
-        this._childrenListNode.removeStyleClass("expanded");
+        this._childrenListNode.classList.remove("expanded");
 
     this.expanded = false;
@@ -734 +734 @@
         this._childrenListNode = this.treeOutline._childrenListNode.ownerDocument.createElement("ol");
         this._childrenListNode.parentTreeElement = this;
-        this._childrenListNode.addStyleClass("children");
+        this._childrenListNode.classList.add("children");
 
         if (this.hidden)
-            this._childrenListNode.addStyleClass("hidden");
+            this._childrenListNode.classList.add("hidden");
 
         this.onpopulate();
@@ -748 +748 @@
 
     if (this._listItemNode) {
-        this._listItemNode.addStyleClass("expanded");
+        this._listItemNode.classList.add("expanded");
         if (this._childrenListNode && this._childrenListNode.parentNode != this._listItemNode.parentNode)
             this.parent._childrenListNode.insertBefore(this._childrenListNode, this._listItemNode.nextSibling);
@@ -754 +754 @@
 
     if (this._childrenListNode)
-        this._childrenListNode.addStyleClass("expanded");
+        this._childrenListNode.classList.add("expanded");
 
     this.expanded = true;
@@ -850 +850 @@
     this.treeOutline.selectedTreeElement = this;
     if (this._listItemNode)
-        this._listItemNode.addStyleClass("selected");
+        this._listItemNode.classList.add("selected");
 
     if (this.onselect)
@@ -876 +876 @@
     this.treeOutline.selectedTreeElement = null;
     if (this._listItemNode)
-        this._listItemNode.removeStyleClass("selected");
+        this._listItemNode.classList.remove("selected");
 
     if (this.ondeselect && !supressOnDeselect)