Make Task Manager sort more meaningful

chrome/browser/task_management/sampling/task_group.h

 // Copyright 2015 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.
 
 #ifndef CHROME_BROWSER_TASK_MANAGEMENT_SAMPLING_TASK_GROUP_H_
 #define CHROME_BROWSER_TASK_MANAGEMENT_SAMPLING_TASK_GROUP_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 27 matching lines @@
 
   // Adds and removes the given |task| to this group. |task| must be running on
   // the same process represented by this group.
   void AddTask(Task* task);
   void RemoveTask(Task* task);
 
   void Refresh(const gpu::VideoMemoryUsageStats& gpu_memory_stats,
                base::TimeDelta update_interval,
                int64_t refresh_flags);
 
-  // Appends the sorted IDs of the tasks that belong to this group to
-  // |out_list|.
-  void AppendSortedTaskIds(TaskIdList* out_list) const;
-
   Task* GetTaskById(TaskId task_id) const;
 
   // This is to be called after the task manager had informed its observers with
   // OnTasksRefreshedWithBackgroundCalculations() to begin another cycle for
   // this notification type.
   void ClearCurrentBackgroundCalculationsFlags();
 
   // True if all enabled background operations calculating resource usage of the
   // process represented by this TaskGroup have completed.
   bool AreBackgroundCalculationsDone() const;
 
   const base::ProcessHandle& process_handle() const { return process_handle_; }
   const base::ProcessId& process_id() const { return process_id_; }
 
-  size_t num_tasks() const { return tasks_.size(); }
-  bool empty() const { return tasks_.empty(); }
+  const std::vector<Task*>& tasks() const { return tasks_; }
+  size_t num_tasks() const { return tasks().size(); }
+  bool empty() const { return tasks().empty(); }
 
   double cpu_usage() const { return cpu_usage_; }
   int64_t private_bytes() const { return memory_usage_.private_bytes; }
   int64_t shared_bytes() const { return memory_usage_.shared_bytes; }
   int64_t physical_bytes() const { return memory_usage_.physical_bytes; }
 #if defined(OS_CHROMEOS)
   int64_t swapped_bytes() const { return memory_usage_.swapped_bytes; }
 #endif
   int64_t gpu_memory() const { return gpu_memory_; }
   bool gpu_memory_has_duplicates() const { return gpu_memory_has_duplicates_; }
@@ skipping 44 matching lines @@
   // The process' handle and ID.
   base::ProcessHandle process_handle_;
   base::ProcessId process_id_;
 
   // This is a callback into the TaskManagerImpl to inform it that the
   // background calculations for this TaskGroup has finished.
   const base::Closure on_background_calculations_done_;
 
   scoped_refptr<TaskGroupSampler> worker_thread_sampler_;
 
-  // Maps the Tasks by their IDs.
+  // Lists the Tasks in this TaskGroup.
   // Tasks are not owned by the TaskGroup. They're owned by the TaskProviders.
-  std::map<TaskId, Task*> tasks_;
+  std::vector<Task*> tasks_;
 
   // Flags will be used to determine when the background calculations has
   // completed for the enabled refresh types for this TaskGroup.
   int64_t expected_on_bg_done_flags_;
   int64_t current_on_bg_done_flags_;
 
   // The per process resources usages.
   double cpu_usage_;
   MemoryUsageStats memory_usage_;
   int64_t gpu_memory_;
@@ skipping 22 matching lines @@
   // destroyed.
   base::WeakPtrFactory<TaskGroup> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(TaskGroup);
 };
 
 }  // namespace task_management
 
 
 #endif  // CHROME_BROWSER_TASK_MANAGEMENT_SAMPLING_TASK_GROUP_H_