DevTools: do not allow using 'this' before call into super.

third_party/WebKit/Source/devtools/front_end/sdk/DOMModel.js

 /*
  * Copyright (C) 2009, 2010 Google Inc. All rights reserved.
  * Copyright (C) 2009 Joseph Pecoraro
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
@@ skipping 15 matching lines @@
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 /**
  * @constructor
  * @extends {WebInspector.SDKObject}
  * @param {!WebInspector.DOMModel} domModel
+ */
+WebInspector.DOMNode = function(domModel)
+{
+    WebInspector.SDKObject.call(this, domModel.target());
+    this._domModel = domModel;
+};
+
+/**
+ * @param {!WebInspector.DOMModel} domModel
  * @param {?WebInspector.DOMDocument} doc
  * @param {boolean} isInShadowTree
  * @param {!DOMAgent.Node} payload
+ * @return {!WebInspector.DOMNode}
  */
-WebInspector.DOMNode = function(domModel, doc, isInShadowTree, payload)
+WebInspector.DOMNode.create = function(domModel, doc, isInShadowTree, payload)
 {
-    WebInspector.SDKObject.call(this, domModel.target());
-    this._domModel = domModel;
-    this._agent = domModel._agent;
-    this.ownerDocument = doc;
-    this._isInShadowTree = isInShadowTree;
-
-    this.id = payload.nodeId;
-    domModel._idToDOMNode[this.id] = this;
-    this._nodeType = payload.nodeType;
-    this._nodeName = payload.nodeName;
-    this._localName = payload.localName;
-    this._nodeValue = payload.nodeValue;
-    this._pseudoType = payload.pseudoType;
-    this._shadowRootType = payload.shadowRootType;
-    this._frameOwnerFrameId = payload.frameId || null;
-    this._xmlVersion = payload.xmlVersion;
-
-    this._shadowRoots = [];
-
-    this._attributes = [];
-    this._attributesMap = {};
-    if (payload.attributes)
-        this._setAttributesPayload(payload.attributes);
-
-    /** @type {!Map<string, ?>} */
-    this._markers = new Map();
-    this._subtreeMarkerCount = 0;
-
-    this._childNodeCount = payload.childNodeCount || 0;
-    this._children = null;
-
-    this.nextSibling = null;
-    this.previousSibling = null;
-    this.firstChild = null;
-    this.lastChild = null;
-    this.parentNode = null;
-
-    if (payload.shadowRoots) {
-        for (var i = 0; i < payload.shadowRoots.length; ++i) {
-            var root = payload.shadowRoots[i];
-            var node = new WebInspector.DOMNode(this._domModel, this.ownerDocument, true, root);
-            this._shadowRoots.push(node);
-            node.parentNode = this;
-        }
-    }
-
-    if (payload.templateContent) {
-        this._templateContent = new WebInspector.DOMNode(this._domModel, this.ownerDocument, true, payload.templateContent);
-        this._templateContent.parentNode = this;
-    }
-
-    if (payload.importedDocument) {
-        this._importedDocument = new WebInspector.DOMNode(this._domModel, this.ownerDocument, true, payload.importedDocument);
-        this._importedDocument.parentNode = this;
-    }
-
-    if (payload.distributedNodes)
-        this._setDistributedNodePayloads(payload.distributedNodes);
-
-    if (payload.children)
-        this._setChildrenPayload(payload.children);
-
-    this._setPseudoElements(payload.pseudoElements);
-
-    if (payload.contentDocument) {
-        this._contentDocument = new WebInspector.DOMDocument(domModel, payload.contentDocument);
-        this._children = [this._contentDocument];
-        this._renumber();
-    }
-
-    if (this._nodeType === Node.ELEMENT_NODE) {
-        // HTML and BODY from internal iframes should not overwrite top-level ones.
-        if (this.ownerDocument && !this.ownerDocument.documentElement && this._nodeName === "HTML")
-            this.ownerDocument.documentElement = this;
-        if (this.ownerDocument && !this.ownerDocument.body && this._nodeName === "BODY")
-            this.ownerDocument.body = this;
-    } else if (this._nodeType === Node.DOCUMENT_TYPE_NODE) {
-        this.publicId = payload.publicId;
-        this.systemId = payload.systemId;
-        this.internalSubset = payload.internalSubset;
-    } else if (this._nodeType === Node.ATTRIBUTE_NODE) {
-        this.name = payload.name;
-        this.value = payload.value;
-    }
-};
+    var node = new WebInspector.DOMNode(domModel);
+    node._init(doc, isInShadowTree, payload);
+    return node;
+}
 
 /**
  * @enum {string}
  */
 WebInspector.DOMNode.PseudoElementNames = {
     Before: "before",
     After: "after"
 };
 
 /**
  * @enum {string}
  */
 WebInspector.DOMNode.ShadowRootTypes = {
     UserAgent: "user-agent",
     Open: "open",
     Closed: "closed"
 };
 
 /** @typedef {{name: string, value: string, _node: WebInspector.DOMNode}} */
 WebInspector.DOMNode.Attribute;
 
 WebInspector.DOMNode.prototype = {
     /**
+     * @param {?WebInspector.DOMDocument} doc
+     * @param {boolean} isInShadowTree
+     * @param {!DOMAgent.Node} payload
+     */
+    _init: function(doc, isInShadowTree, payload)
+    {
+        this._agent = this._domModel._agent;
+        this.ownerDocument = doc;
+        this._isInShadowTree = isInShadowTree;
+
+        this.id = payload.nodeId;
+        this._domModel._idToDOMNode[this.id] = this;
+        this._nodeType = payload.nodeType;
+        this._nodeName = payload.nodeName;
+        this._localName = payload.localName;
+        this._nodeValue = payload.nodeValue;
+        this._pseudoType = payload.pseudoType;
+        this._shadowRootType = payload.shadowRootType;
+        this._frameOwnerFrameId = payload.frameId || null;
+        this._xmlVersion = payload.xmlVersion;
+
+        this._shadowRoots = [];
+
+        this._attributes = [];
+        this._attributesMap = {};
+        if (payload.attributes)
+            this._setAttributesPayload(payload.attributes);
+
+        /** @type {!Map<string, ?>} */
+        this._markers = new Map();
+        this._subtreeMarkerCount = 0;
+
+        this._childNodeCount = payload.childNodeCount || 0;
+        this._children = null;
+
+        this.nextSibling = null;
+        this.previousSibling = null;
+        this.firstChild = null;
+        this.lastChild = null;
+        this.parentNode = null;
+
+        if (payload.shadowRoots) {
+            for (var i = 0; i < payload.shadowRoots.length; ++i) {
+                var root = payload.shadowRoots[i];
+                var node = WebInspector.DOMNode.create(this._domModel, this.ownerDocument, true, root);
+                this._shadowRoots.push(node);
+                node.parentNode = this;
+            }
+        }
+
+        if (payload.templateContent) {
+            this._templateContent = WebInspector.DOMNode.create(this._domModel, this.ownerDocument, true, payload.templateContent);
+            this._templateContent.parentNode = this;
+        }
+
+        if (payload.importedDocument) {
+            this._importedDocument = WebInspector.DOMNode.create(this._domModel, this.ownerDocument, true, payload.importedDocument);
+            this._importedDocument.parentNode = this;
+        }
+
+        if (payload.distributedNodes)
+            this._setDistributedNodePayloads(payload.distributedNodes);
+
+        if (payload.children)
+            this._setChildrenPayload(payload.children);
+
+        this._setPseudoElements(payload.pseudoElements);
+
+        if (payload.contentDocument) {
+            this._contentDocument = new WebInspector.DOMDocument(this._domModel, payload.contentDocument);
+            this._children = [this._contentDocument];
+            this._renumber();
+        }
+
+        if (this._nodeType === Node.ELEMENT_NODE) {
+            // HTML and BODY from internal iframes should not overwrite top-level ones.
+            if (this.ownerDocument && !this.ownerDocument.documentElement && this._nodeName === "HTML")
+                this.ownerDocument.documentElement = this;
+            if (this.ownerDocument && !this.ownerDocument.body && this._nodeName === "BODY")
+                this.ownerDocument.body = this;
+        } else if (this._nodeType === Node.DOCUMENT_TYPE_NODE) {
+            this.publicId = payload.publicId;
+            this.systemId = payload.systemId;
+            this.internalSubset = payload.internalSubset;
+        } else if (this._nodeType === Node.ATTRIBUTE_NODE) {
+            this.name = payload.name;
+            this.value = payload.value;
+        }
+    },
+
+    /**
      * @return {!WebInspector.DOMModel}
      */
     domModel: function()
     {
         return this._domModel;
     },
 
     /**
      * @return {?Array.<!WebInspector.DOMNode>}
      */
@@ skipping 475 matching lines @@
         return attributesChanged;
     },
 
     /**
      * @param {!WebInspector.DOMNode} prev
      * @param {!DOMAgent.Node} payload
      * @return {!WebInspector.DOMNode}
      */
     _insertChild: function(prev, payload)
     {
-        var node = new WebInspector.DOMNode(this._domModel, this.ownerDocument, this._isInShadowTree, payload);
+        var node = WebInspector.DOMNode.create(this._domModel, this.ownerDocument, this._isInShadowTree, payload);
         this._children.splice(this._children.indexOf(prev) + 1, 0, node);
         this._renumber();
         return node;
     },
 
     /**
      * @param {!WebInspector.DOMNode} node
      */
     _removeChild: function(node)
     {
@@ skipping 20 matching lines @@
      */
     _setChildrenPayload: function(payloads)
     {
         // We set children in the constructor.
         if (this._contentDocument)
             return;
 
         this._children = [];
         for (var i = 0; i < payloads.length; ++i) {
             var payload = payloads[i];
-            var node = new WebInspector.DOMNode(this._domModel, this.ownerDocument, this._isInShadowTree, payload);
+            var node = WebInspector.DOMNode.create(this._domModel, this.ownerDocument, this._isInShadowTree, payload);
             this._children.push(node);
         }
         this._renumber();
     },
 
     /**
      * @param {!Array.<!DOMAgent.Node>|undefined} payloads
      */
     _setPseudoElements: function(payloads)
     {
         this._pseudoElements = new Map();
         if (!payloads)
             return;
 
         for (var i = 0; i < payloads.length; ++i) {
-            var node = new WebInspector.DOMNode(this._domModel, this.ownerDocument, this._isInShadowTree, payloads[i]);
+            var node = WebInspector.DOMNode.create(this._domModel, this.ownerDocument, this._isInShadowTree, payloads[i]);
             node.parentNode = this;
             this._pseudoElements.set(node.pseudoType(), node);
         }
     },
 
     /**
      * @param {!Array.<!DOMAgent.BackendNode>} payloads
      */
     _setDistributedNodePayloads: function(payloads)
     {
@@ skipping 356 matching lines @@
 };
 
 /**
  * @extends {WebInspector.DOMNode}
  * @constructor
  * @param {!WebInspector.DOMModel} domModel
  * @param {!DOMAgent.Node} payload
  */
 WebInspector.DOMDocument = function(domModel, payload)
 {
-    WebInspector.DOMNode.call(this, domModel, this, false, payload);
+    WebInspector.DOMNode.call(this, domModel);
+    this._init(this, false, payload);
     this.documentURL = payload.documentURL || "";
     this.baseURL = payload.baseURL || "";
     this._listeners = {};
 };
 
 WebInspector.DOMDocument.prototype = {
     __proto__: WebInspector.DOMNode.prototype
 };
 
 /**
@@ skipping 376 matching lines @@
     },
 
     /**
      * @param {!DOMAgent.Node} payload
      */
     _setDetachedRoot: function(payload)
     {
         if (payload.nodeName === "#document")
             new WebInspector.DOMDocument(this, payload);
         else
-            new WebInspector.DOMNode(this, null, false, payload);
+            WebInspector.DOMNode.create(this, null, false, payload);
     },
 
     /**
      * @param {!DOMAgent.NodeId} parentId
      * @param {!Array.<!DOMAgent.Node>} payloads
      */
     _setChildNodes: function(parentId, payloads)
     {
         if (!parentId && payloads.length) {
             this._setDetachedRoot(payloads[0]);
@@ skipping 47 matching lines @@
 
     /**
      * @param {!DOMAgent.NodeId} hostId
      * @param {!DOMAgent.Node} root
      */
     _shadowRootPushed: function(hostId, root)
     {
         var host = this._idToDOMNode[hostId];
         if (!host)
             return;
-        var node = new WebInspector.DOMNode(this, host.ownerDocument, true, root);
+        var node = WebInspector.DOMNode.create(this, host.ownerDocument, true, root);
         node.parentNode = host;
         this._idToDOMNode[node.id] = node;
         host._shadowRoots.unshift(node);
         this.dispatchEventToListeners(WebInspector.DOMModel.Events.NodeInserted, node);
         this._scheduleMutationEvent(node);
     },
 
     /**
      * @param {!DOMAgent.NodeId} hostId
      * @param {!DOMAgent.NodeId} rootId
@@ skipping 14 matching lines @@
 
     /**
      * @param {!DOMAgent.NodeId} parentId
      * @param {!DOMAgent.Node} pseudoElement
      */
     _pseudoElementAdded: function(parentId, pseudoElement)
     {
         var parent = this._idToDOMNode[parentId];
         if (!parent)
             return;
-        var node = new WebInspector.DOMNode(this, parent.ownerDocument, false, pseudoElement);
+        var node = WebInspector.DOMNode.create(this, parent.ownerDocument, false, pseudoElement);
         node.parentNode = parent;
         this._idToDOMNode[node.id] = node;
         console.assert(!parent._pseudoElements.get(node.pseudoType()));
         parent._pseudoElements.set(node.pseudoType(), node);
         this.dispatchEventToListeners(WebInspector.DOMModel.Events.NodeInserted, node);
         this._scheduleMutationEvent(node);
     },
 
     /**
      * @param {!DOMAgent.NodeId} parentId
@@ skipping 720 matching lines @@
 };
 
 /**
  * @param {!WebInspector.Target} target
  * @return {?WebInspector.DOMModel}
  */
 WebInspector.DOMModel.fromTarget = function(target)
 {
     return /** @type {?WebInspector.DOMModel} */ (target.model(WebInspector.DOMModel));
 };