Task Manager: Remove dead FPS code.

chrome/browser/task_manager/task_manager.cc

 // Copyright (c) 2012 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.
 
 #include "chrome/browser/task_manager/task_manager.h"
 
 #include "base/bind.h"
 #include "base/i18n/number_formatting.h"
 #include "base/i18n/rtl.h"
 #include "base/prefs/pref_registry_simple.h"
@@ skipping 182 matching lines @@
 
 TaskManagerModel::PerResourceValues::PerResourceValues()
     : is_title_valid(false),
       is_profile_name_valid(false),
       network_usage(0),
       is_process_id_valid(false),
       process_id(0),
       is_goats_teleported_valid(false),
       goats_teleported(0),
       is_webcore_stats_valid(false),
-      is_fps_valid(false),
-      fps(0),
       is_sqlite_memory_bytes_valid(false),
       sqlite_memory_bytes(0),
       is_v8_memory_valid(false),
       v8_memory_allocated(0),
       v8_memory_used(0) {}
 
 TaskManagerModel::PerResourceValues::~PerResourceValues() {}
 
 TaskManagerModel::PerProcessValues::PerProcessValues()
     : is_cpu_usage_valid(false),
@@ skipping 154 matching lines @@
 
     case IDS_TASK_MANAGER_WEBCORE_IMAGE_CACHE_COLUMN:
       return GetResourceWebCoreImageCacheSize(index);
 
     case IDS_TASK_MANAGER_WEBCORE_SCRIPTS_CACHE_COLUMN:
       return GetResourceWebCoreScriptsCacheSize(index);
 
     case IDS_TASK_MANAGER_WEBCORE_CSS_CACHE_COLUMN:
       return GetResourceWebCoreCSSCacheSize(index);
 
-    case IDS_TASK_MANAGER_FPS_COLUMN:
-      return GetResourceFPS(index);
-
     case IDS_TASK_MANAGER_VIDEO_MEMORY_COLUMN:
       return GetResourceVideoMemory(index);
 
     case IDS_TASK_MANAGER_SQLITE_MEMORY_USED_COLUMN:
       return GetResourceSqliteMemoryUsed(index);
 
     case IDS_TASK_MANAGER_JAVASCRIPT_MEMORY_ALLOCATED_COLUMN:
       return GetResourceV8MemoryAllocatedSize(index);
 
     case IDS_TASK_MANAGER_NACL_DEBUG_STUB_PORT_COLUMN:
@@ skipping 122 matching lines @@
   size_t video_memory;
   bool has_duplicates;
   if (!GetVideoMemory(index, &video_memory, &has_duplicates) || !video_memory)
     return base::ASCIIToUTF16("N/A");
   if (has_duplicates) {
     return GetMemCellText(video_memory) + base::ASCIIToUTF16("*");
   }
   return GetMemCellText(video_memory);
 }
 
-base::string16 TaskManagerModel::GetResourceFPS(
-    int index) const {
-  float fps = 0;
-  if (!GetFPS(index, &fps))
-    return l10n_util::GetStringUTF16(IDS_TASK_MANAGER_NA_CELL_TEXT);
-  return base::UTF8ToUTF16(base::StringPrintf("%.0f", fps));
-}
-
 base::string16 TaskManagerModel::GetResourceSqliteMemoryUsed(int index) const {
   size_t bytes = 0;
   if (!GetSqliteMemoryUsedBytes(index, &bytes))
     return l10n_util::GetStringUTF16(IDS_TASK_MANAGER_NA_CELL_TEXT);
   return GetMemCellText(bytes);
 }
 
 base::string16 TaskManagerModel::GetResourceIdleWakeupsPerSecond(int index)
     const {
   return base::FormatNumber(GetIdleWakeupsPerSecond(GetResource(index)));
@@ skipping 130 matching lines @@
       return false;
     values.is_video_memory_valid = true;
     values.video_memory = i->second.video_memory;
     values.video_memory_has_duplicates = i->second.has_duplicates;
   }
   *video_memory = values.video_memory;
   *has_duplicates = values.video_memory_has_duplicates;
   return true;
 }
 
-bool TaskManagerModel::GetFPS(int index, float* result) const {
-  *result = 0;
-  PerResourceValues& values(GetPerResourceValues(index));
-  if (!values.is_fps_valid) {
-    if (!GetResource(index)->ReportsFPS())
-      return false;
-    values.is_fps_valid = true;
-    values.fps = GetResource(index)->GetFPS();
-  }
-  *result = values.fps;
-  return true;
-}
-
 bool TaskManagerModel::GetSqliteMemoryUsedBytes(
     int index,
     size_t* result) const {
   *result = 0;
   PerResourceValues& values(GetPerResourceValues(index));
   if (!values.is_sqlite_memory_bytes_valid) {
     if (!GetResource(index)->ReportsSqliteMemoryUsed())
       return false;
     values.is_sqlite_memory_bytes_valid = true;
     values.sqlite_memory_bytes = GetResource(index)->SqliteMemoryUsedBytes();
@@ skipping 218 matching lines @@
             return ValueCompare(stats1.cssStyleSheets.size,
                                 stats2.cssStyleSheets.size);
           default:
             NOTREACHED();
             return 0;
         }
       }
       return OrderUnavailableValue(row1_stats_valid, row2_stats_valid);
     }
 
-    case IDS_TASK_MANAGER_FPS_COLUMN:
-      return ValueCompareMember(
-          this, &TaskManagerModel::GetFPS, row1, row2);
-
     case IDS_TASK_MANAGER_VIDEO_MEMORY_COLUMN: {
       size_t value1;
       size_t value2;
       bool has_duplicates;
       bool value1_valid = GetVideoMemory(row1, &value1, &has_duplicates);
       bool value2_valid = GetVideoMemory(row2, &value2, &has_duplicates);
       return value1_valid && value2_valid ? ValueCompare(value1, value2) :
           OrderUnavailableValue(value1_valid, value2_valid);
     }
 
@@ skipping 307 matching lines @@
     base::ProcessId renderer_id,
     const blink::WebCache::ResourceTypeStats& stats) {
   for (ResourceList::iterator it = resources_.begin();
        it != resources_.end(); ++it) {
     if (base::GetProcId((*it)->GetProcess()) == renderer_id) {
       (*it)->NotifyResourceTypeStats(stats);
     }
   }
 }
 
-void TaskManagerModel::NotifyFPS(base::ProcessId renderer_id,
-                                 int routing_id,
-                                 float fps) {
-  for (ResourceList::iterator it = resources_.begin();
-       it != resources_.end(); ++it) {
-    if (base::GetProcId((*it)->GetProcess()) == renderer_id &&
-        (*it)->GetRoutingID() == routing_id) {
-      (*it)->NotifyFPS(fps);
-    }
-  }
-}
-
 void TaskManagerModel::NotifyVideoMemoryUsageStats(
     const content::GPUVideoMemoryUsageStats& video_memory_usage_stats) {
   DCHECK(pending_video_memory_usage_stats_update_);
   video_memory_usage_stats_ = video_memory_usage_stats;
   pending_video_memory_usage_stats_update_ = false;
 }
 
 void TaskManagerModel::NotifyV8HeapStats(base::ProcessId renderer_id,
                                          size_t v8_memory_allocated,
                                          size_t v8_memory_used) {
@@ skipping 308 matching lines @@
   params.host_desktop_type = desktop_type;
   chrome::Navigate(&params);
 }
 
 TaskManager::TaskManager()
     : model_(new TaskManagerModel(this)) {
 }
 
 TaskManager::~TaskManager() {
 }