DevTools Profiler UI changes....

webkit/glue/devtools/js/devtools.js

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
- * @fileoverview Tools is a main class that wires all components of the 
+ * @fileoverview Tools is a main class that wires all components of the
  * DevTools frontend together. It is also responsible for overriding existing
  * WebInspector functionality while it is getting upstreamed into WebCore.
  */
 goog.provide('devtools.Tools');
 
 goog.require('devtools.DebuggerAgent');
 goog.require('devtools.DomAgent');
 
 
 /**
@@ skipping 36 matching lines @@
   this.domAgent_ = new devtools.DomAgent();
 };
 
 
 /**
  * Resets tools agent to its initial state.
  */
 devtools.ToolsAgent.prototype.reset = function() {
   this.domAgent_.reset();
   this.debuggerAgent_.reset();
-  
+
   this.domAgent_.getDocumentElementAsync();
 };
 
 
 /**
  * @param {string} script Script exression to be evaluated in the context of the
  *     inspected page.
  * @param {function(string):undefined} callback Function to call with the
  *     result.
  */
@@ skipping 73 matching lines @@
  * @param {string} expr
  */
 devtools.ToolsAgent.prototype.evaluate = function(expr) {
   RemoteToolsAgent.evaluate(expr);
 };
 
 
 /**
  * Asynchronously queries for the resource content.
  * @param {number} identifier Resource identifier.
- * @param {function(string):undefined} opt_callback Callback to call when 
+ * @param {function(string):undefined} opt_callback Callback to call when
  *     result is available.
  */
-devtools.ToolsAgent.prototype.getResourceContentAsync = function(identifier, 
+devtools.ToolsAgent.prototype.getResourceContentAsync = function(identifier,
     opt_callback) {
   var resource = WebInspector.resources[identifier];
   if (!resource) {
     return;
   }
   RemoteToolsAgent.GetResourceContent(
       devtools.Callback.wrap(opt_callback), identifier);
 };
 
 
@@ skipping 63 matching lines @@
       goog.bind(this.performSearchCallback_, this));
 };
 
 
 WebInspector.ElementsPanel.prototype.performSearchCallback_ = function(nodes) {
   for (var i = 0; i < nodes.length; ++i) {
     var treeElement = this.treeOutline.findTreeElement(nodes[i]);
     if (treeElement)
       treeElement.highlighted = true;
   }
-  
+
   if (nodes.length) {
     this.currentSearchResultIndex_ = 0;
     this.focusedDOMNode = nodes[0];
   }
-  
+
   this.searchResultCount_ = nodes.length;
 };
 
 
 /**
  * @override
  */
 WebInspector.ElementsPanel.prototype.searchCanceled = function() {
   this.currentSearchResultIndex_ = 0;
   this.searchResultCount_ = 0;
@@ skipping 94 matching lines @@
  * Temporarily sets style fetched from the inspectable tab to the currently
  * focused node, invokes updateUI callback and clears the styles.
  * @param {function(Node):undefined} updateUI Callback to call while styles are
  *     set.
  */
 WebInspector.ElementsPanel.prototype.invokeWithStyleSet_ =
     function(updateUI) {
   var node = this.focusedDOMNode;
   if (node && node.nodeType === Node.TEXT_NODE && node.parentNode)
     node = node.parentNode;
-  
+
   if (node && node.nodeType == Node.ELEMENT_NODE) {
     var callback = function(stylesStr) {
       var styles = JSON.parse(stylesStr);
       if (!styles.computedStyle) {
         return;
       }
       node.setStyles(styles.computedStyle, styles.inlineStyle,
           styles.styleAttributes, styles.matchedCSSRules);
       updateUI(node);
       node.clearStyles();
@@ skipping 43 matching lines @@
 WebInspector.PropertiesSidebarPane.prototype.update = function(object) {
   var body = this.bodyElement;
   body.removeChildren();
 
   this.sections = [];
 
   if (!object) {
     return;
   }
 
-  
+
   var self = this;
-  devtools.tools.getDomAgent().getNodePrototypesAsync(object.id_, 
+  devtools.tools.getDomAgent().getNodePrototypesAsync(object.id_,
       function(json) {
         // Get array of prototype user-friendly names.
         var prototypes = JSON.parse(json);
         for (var i = 0; i < prototypes.length; ++i) {
           var prototype = {};
           prototype.id_ = object.id_;
           prototype.protoDepth_ = i;
           var section = new WebInspector.SidebarObjectPropertiesSection(
               prototype,
               prototypes[i]);
@@ skipping 49 matching lines @@
 
 
 /**
  * @override
  */
 WebInspector.SidebarObjectPropertyTreeElement.prototype.onpopulate =
     function() {
   var nodeId = this.parentObject.devtools$$nodeId_;
   var path = this.parentObject.devtools$$path_.slice(0);
   path.push(this.propertyName);
-  devtools.tools.getDomAgent().getNodePropertiesAsync(nodeId, path, -1, 
+  devtools.tools.getDomAgent().getNodePropertiesAsync(nodeId, path, -1,
       goog.partial(
           WebInspector.didGetNodePropertiesAsync_,
           this,
           this.treeOutline.section.treeElementConstructor,
           nodeId, path));
 };
 
 
 /**
  * This override is necessary for starting highlighting after the resource
@@ skipping 32 matching lines @@
 /**
  * This override is necessary for adding script source asynchronously.
  * @override
  */
 WebInspector.ScriptView.prototype.setupSourceFrameIfNeeded = function() {
   if (!this._frameNeedsSetup) {
     return;
   }
 
   this.attach();
-  
+
   if (this.script.source) {
     this.didResolveScriptSource_();
   } else {
     var self = this;
     devtools.tools.getDebuggerAgent().resolveScriptSource(
         this.script.sourceID,
         function(source) {
           self.script.source = source || '<source is not available>';
           self.didResolveScriptSource_();
         });
@@ skipping 350 matching lines @@
    WebInspector.DataGrid = function(columns) {
      if (('average' in columns) && ('calls' in columns)) {
        delete columns['average'];
        delete columns['calls'];
      }
      return new originalDataGrid(columns);
    };
 })();
 
 
+// WebKit's profiler displays milliseconds with high resolution (shows
+// three digits after the decimal point). We never have such resolution,
+// as our minimal sampling rate is 1 ms. So we are disabling high resolution
+// to avoid visual clutter caused by meaningless ".000" parts.
+(function InterceptTimeDisplayInProfiler() {
+   var originalDataGetter =
+       WebInspector.ProfileDataGridNode.prototype.__lookupGetter__('data');
+   WebInspector.ProfileDataGridNode.prototype.__defineGetter__('data',
+     function() {
+       var oldNumberSecondsToString = Number.secondsToString;
+       Number.secondsToString = function(seconds, formatterFunction) {
+         return oldNumberSecondsToString(seconds, formatterFunction, false);
+       };
+       var data = originalDataGetter.call(this);
+       Number.secondsToString = oldNumberSecondsToString;
+       return data;
+     });
+})();
+
+
 /**
  * @override
  * TODO(pfeldman): Add l10n.
  */
 WebInspector.UIString = function(string) {
   return String.vsprintf(string, Array.prototype.slice.call(arguments, 1));
 };
 
 
 // There is no clear way of setting frame title yet. So sniffing main resource
 // load.
 (function OverrideUpdateResource() {
   var originalUpdateResource = WebInspector.updateResource;
   WebInspector.updateResource = function(identifier, payload) {
     originalUpdateResource.call(this, identifier, payload);
     var resource = this.resources[identifier];
     if (resource && resource.mainResource && resource.finished) {
       document.title = 'Developer Tools - ' + resource.url;
     }
   };
 })();