Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(61)

Side by Side Diff: chrome/browser/resources/net_internals/util.js

Issue 7530020: Re-design the "data" tab on about:net-internals. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Remove a TODO which I don't intend to do anymore Created 9 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /** 5 /**
6 * Inherit the prototype methods from one constructor into another. 6 * Inherit the prototype methods from one constructor into another.
7 */ 7 */
8 function inherits(childCtor, parentCtor) { 8 function inherits(childCtor, parentCtor) {
9 function tempCtor() {}; 9 function tempCtor() {};
10 tempCtor.prototype = parentCtor.prototype; 10 tempCtor.prototype = parentCtor.prototype;
(...skipping 23 matching lines...) Expand all
34 node.style.left = leftPx.toFixed(0) + 'px'; 34 node.style.left = leftPx.toFixed(0) + 'px';
35 node.style.top = topPx.toFixed(0) + 'px'; 35 node.style.top = topPx.toFixed(0) + 'px';
36 setNodeWidth(node, widthPx); 36 setNodeWidth(node, widthPx);
37 setNodeHeight(node, heightPx); 37 setNodeHeight(node, heightPx);
38 } 38 }
39 39
40 /** 40 /**
41 * Sets the visibility for a DOM node. 41 * Sets the visibility for a DOM node.
42 */ 42 */
43 function setNodeDisplay(node, isVisible) { 43 function setNodeDisplay(node, isVisible) {
44 if (!isVisible && node.id == "importTab") {
45 debugger;
mmenke 2011/07/29 14:18:53 What's this for, exactly? Break into the debugger
eroman 2011/07/29 15:15:08 Oops, that was left-over from my debugging session
46 }
44 node.style.display = isVisible ? '' : 'none'; 47 node.style.display = isVisible ? '' : 'none';
45 } 48 }
46 49
47 /** 50 /**
48 * Adds a node to |parentNode|, of type |tagName|. 51 * Adds a node to |parentNode|, of type |tagName|.
49 */ 52 */
50 function addNode(parentNode, tagName) { 53 function addNode(parentNode, tagName) {
51 var elem = parentNode.ownerDocument.createElement(tagName); 54 var elem = parentNode.ownerDocument.createElement(tagName);
52 parentNode.appendChild(elem); 55 parentNode.appendChild(elem);
53 return elem; 56 return elem;
(...skipping 313 matching lines...) Expand 10 before | Expand all | Expand 10 after
367 linkNode.href = cell.link; 370 linkNode.href = cell.link;
368 } else { 371 } else {
369 addTextNode(tableCell, cell.text); 372 addTextNode(tableCell, cell.text);
370 } 373 }
371 } 374 }
372 } 375 }
373 } 376 }
374 return table; 377 return table;
375 }; 378 };
376 379
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698