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

Side by Side Diff: tools/traceline/svgui/traceline.js

Issue 9910010: Fix some grammatical errors in comments in tools/ (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: remediate to review Created 8 years, 8 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
« no previous file with comments | « tools/playback_benchmark/playback.js ('k') | tools/traceline/traceline/syscall_map.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2009 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2009 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 // TODO 5 // TODO
6 // - spacial partitioning of the data so that we don't have to scan the 6 // - spacial partitioning of the data so that we don't have to scan the
7 // entire scene every time we render. 7 // entire scene every time we render.
8 // - properly clip the SVG elements when they render, right now we are just 8 // - properly clip the SVG elements when they render, right now we are just
9 // letting them go negative or off the screen. This might give us a little 9 // letting them go negative or off the screen. This might give us a little
10 // bit better performance? 10 // bit better performance?
(...skipping 444 matching lines...) Expand 10 before | Expand all | Expand 10 after
455 // The area we update with the info on mouseovers. 455 // The area we update with the info on mouseovers.
456 'infoareadiv': infoareadiv, 456 'infoareadiv': infoareadiv,
457 // Called when we detected new scroll a should redraw 457 // Called when we detected new scroll a should redraw
458 'redraw': function() { }, 458 'redraw': function() { },
459 'attached': false, 459 'attached': false,
460 'attach': function() { 460 'attach': function() {
461 document.getElementsByTagNameNS(xhtmlNS, 'body')[0].appendChild( 461 document.getElementsByTagNameNS(xhtmlNS, 'body')[0].appendChild(
462 this.totaldiv); 462 this.totaldiv);
463 this.attached = true; 463 this.attached = true;
464 }, 464 },
465 // The fatty div will have it's width adjusted based on the zoom level and 465 // The fatty div will have its width adjusted based on the zoom level and
466 // the duration of the graph, to get the scrolling correct for the size. 466 // the duration of the graph, to get the scrolling correct for the size.
467 'zoom': function(curzoom) { 467 'zoom': function(curzoom) {
468 var width = widthms * curzoom; 468 var width = widthms * curzoom;
469 fattydiv.style.width = width + 'px'; 469 fattydiv.style.width = width + 'px';
470 }, 470 },
471 'detach': function() { 471 'detach': function() {
472 this.totaldiv.parentNode.removeChild(this.totaldiv); 472 this.totaldiv.parentNode.removeChild(this.totaldiv);
473 this.attached = false; 473 this.attached = false;
474 }, 474 },
475 }; 475 };
(...skipping 208 matching lines...) Expand 10 before | Expand all | Expand 10 after
684 } 684 }
685 685
686 ++count; 686 ++count;
687 } 687 }
688 688
689 // Append the 'g' element on after we've build it. 689 // Append the 'g' element on after we've build it.
690 svg.appendChild(svgg); 690 svg.appendChild(svgg);
691 691
692 return count; 692 return count;
693 }; 693 };
OLDNEW
« no previous file with comments | « tools/playback_benchmark/playback.js ('k') | tools/traceline/traceline/syscall_map.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698