Several changes to the benchmark extension:...

chrome/test/data/extensions/samples/benchmark/toolstrip.html

 <style>
 #options {
   position: absolute;
   background-color: #FFFFCC;
   display: none;
   font-family: "Courier New";
   font-size: 9pt;
   padding: 5px;
   border: 1px solid #CCCC88;
   z-index: 3;
 }
 </style>
 
 <script>
-var optionsForm;
-function show_options() {
-  optionsForm = window.open("options.html", "optionswindow");
-}
-
 // Round a number to the 1's place.
 function formatNumber(str) {
   str += '';
   if (str == '0') {
     return 'N/A ';
   }
   var x = str.split('.');
   var x1 = x[0];
   var x2 = x.length > 1 ? '.' + x[1] : '';
   var regex = /(\d+)(\d{3})/;
   while (regex.test(x1)) {
     x1 = x1.replace(regex, '$1' + ',' + '$2');
   }
   return x1;
 }
 
 // Configuration and results are stored globally.
 window.iterations = 10;
 window.clearConnections = true;
 window.clearCache = true;
 window.results = {};
 window.results.data = new Array();
+window.testUrl = "";
+window.windowId = 0;
+
+var optionsForm = 0;
+function show_options() {
+  // This functionality is a bit racey, probably due to the extension API.

[Erik does not do reviews, 2009/07/17 17:15:52]

Is this still racey the way you've implemented it?  Like optionsForm will always
get created before the getSelected callback completed.

+  // If we open the window from within the getSelected() callback, then the
+  // window opens in a new window rather than a new tab (is that a bug?).
+  // If thwe open the window before we call getSelected(), then the selected

[Erik does not do reviews, 2009/07/17 17:15:52]

thwe -> we

+  // tab URL will be NULL, because the page won't have loaded yet, but it will
+  // be the selected tab.  So, we do this weird thing where we first call
+  // get selected, knowing that it is asynchronous.  Then we open the window,
+  // and finally when the callback occurs for getSelected, the window will
+  // be created and we can use it.
+  chrome.tabs.getSelected(windowId, function(tab) {

[Erik does not do reviews, 2009/07/17 17:15:52]

windowId is an optional parameter.  If you don't provide it, it will default to
the current window id which is what you want here.  The docs talk about this a
little:
http://dev.chromium.org/developers/design-documents/extensions/tabs-api

+    window.testUrl = tab.url;
+    optionsForm.setUrl(window.testUrl);
+  });
+  optionsForm = window.open("options.html", "optionswindow");
+}
 
 function Benchmark() {
   var runCount_ = 0;
   var count_;
   var totalTime_;
   var lastWin_;
   var me_ = this;
   var current_;
+  var initialReadBytes_;
+  var initialWriteBytes_;
 
   // Start a test run
   this.start = function(url) {
     // Check if a run is already in progress.
     if (me_.isRunning()) {
       return;
     }
 
+    console.log("Starting test for url: " + url);
+
     runCount_ = window.iterations;
     count_ = 0;
     totalTime_ = 0;
     lastWin_ = 0;
 
     current_ = {};
     current_.url = url;
-    current_.results = new Array();
+    current_.docLoadResults = new Array();  // times to docload
+    current_.paintResults = new Array();    // times to paint
+    current_.totalResults = new Array();    // times to complete load
+    initialReadBytes = chromium.benchmarking.counter("tcp.read_bytes");
+    initialWriteBytes = chromium.benchmarking.counter("tcp.write_bytes");
 
     me_.runPage();
   }
 
   // Is the benchmark currently in progress.
   this.isRunning = function() {
     return runCount_ > 0;
   }
 
   // Called when the test run completes.
   this.finish = function() {
     lastWin_.close();
     lastWin_ = 0;
 
+    // Record some more stats.
+    current_.bytesRead = chromium.benchmarking.counter("tcp.read_bytes") -
+                          initialReadBytes;
+    current_.bytesWritten = chromium.benchmarking.counter("tcp.write_bytes") -
+                            initialWriteBytes;
+    current_.totalTime = totalTime_;
+
     // push the result
     window.results.data.push(current_);
     current_ = 0;
 
     // show the latest
     show_options();
   }
 
   // Update the UI after a test run.
   this.displayResults = function() {
@@ skipping 16 matching lines @@
 
     if (lastWin_) {
       lastWin_.location = current_.url;
     } else {
       lastWin_ = window.open(current_.url);
     }
   }
 
   // Called when a page finishes loading.
   this.pageFinished = function(csi) {
-    var t = Math.round((csi.finishLoadTime - csi.startLoadTime) * 1000.0);
+    var docLoadTime =
+        Math.round((csi.finishDocumentLoadTime - csi.startLoadTime) * 1000.0);
+    var paintTime =
+        Math.round((csi.firstPaintTime - csi.startLoadTime) * 1000.0);
+    var totalTime =
+        Math.round((csi.finishLoadTime - csi.startLoadTime) * 1000.0);
 
     // Record the result
-    current_.results.push(t);
+    current_.docLoadResults.push(docLoadTime);
+    current_.paintResults.push(paintTime);
+    current_.totalResults.push(totalTime);
 
     // For our toolbar counters
-    totalTime_ += t;
+    totalTime_ += totalTime;
     count_++;
 
     if (--runCount_ > 0) {
       setTimeout(me_.runPage, 100);
     } else {
       me_.finish();
     }
 
     // Update the UI
     me_.displayResults();
   }
 }
 
 var benchmarks = new Array();
 
 chrome.self.onConnect.addListener(function(port) {
   port.onMessage.addListener(function(data) {
     if (data.message == "load") {
       var benchmark = benchmarks[data.url];
       if (benchmark != undefined && benchmark.isRunning()) {
         benchmark.pageFinished(data.values);
       }
     }
   });
 });
 
 function run() {
   show_options();
-  var urls = document.getElementById("url").value.split(",");
+  var urls = testUrl.split(",");
   for (var i = 0; i < urls.length; i++) {
     var benchmark = new Benchmark();
     benchmarks[urls[i]] = benchmark;
     benchmark.start(urls[i]);  // XXXMB - move to constructor
   }
 }
+
+// Run at startup
+chrome.windows.getCurrent(function(currentWindow) {

[Erik does not do reviews, 2009/07/17 17:15:52]

so you don't need this...

+  window.windowId = currentWindow.id;
+});
 </script>
 
-<style> 
+<style>
 #result {
   color: green;
   text-align: center;
   vertical-align: center;
 }
-</style> 
+</style>
 
 <div id="bench">
 <img src="stopwatch.jpg" height="25" width="25" align=top onclick="show_options()">
-<input type="text" id="url" value="http://www.google.com/"></input>
-<div class="toolstrip-button"> 
-<span id="run" class="open" onclick="run()">Go</span> 
-</div> 
-<span id="result"></span> 
+<span id="result"></span>
 </div>