Hide the max queue time / max runtime columns in about:profiler when showing...

Hide the max queue time / max runtime columns in about:profiler when showing the difference between two snapshots.
The reason to hide it is because we don't currently display the correct values.
Review URL: http://codereview.chromium.org/8670011

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@111289 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/resources/profiler.js

@@ -1416,6 +1416,7 @@ var MainView = (function() {
     getVisibleColumnKeys_: function() {
       // Figure out what columns to include, based on the selected checkboxes.
       var columns = this.getSelectionColumns_();
+      columns = columns.slice(0);
 
       // Eliminate columns which we are merging on.
       deleteValuesFromArray(columns, this.getMergeColumns_());
@@ -1431,10 +1432,14 @@ var MainView = (function() {
 
         for (var i = 0; i < randomGroupKey.length; ++i)
           keysToExclude.push(randomGroupKey[i].key);
-        columns = columns.slice(0);
         deleteValuesFromArray(columns, keysToExclude);
       }
 
+      // If we are currently showing a "diff", hide the max columns, since we
+      // are not populating it correctly. See the TODO at the top of this file.
+      if (this.getSelectedSnapshotIndexes_().length > 1)
+        deleteValuesFromArray(columns, [KEY_MAX_RUN_TIME, KEY_MAX_QUEUE_TIME]);
+
       return columns;
     },