Make Task Manager sort more meaningful

chrome/browser/task_management/providers/task.cc

 // 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.
 
 #include "chrome/browser/task_management/providers/task.h"
 
 #include <stddef.h>
 
 #include "base/process/process.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_attributes_entry.h"
 #include "chrome/browser/profiles/profile_attributes_storage.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/task_management/task_manager_observer.h"
 #include "content/public/common/result_codes.h"
 
 namespace task_management {
 
 namespace {
 
 // The last ID given to the previously created task.
 int64_t g_last_id = 0;
 
 }  // namespace
 
-
 Task::Task(const base::string16& title,
            const std::string& rappor_sample,
            const gfx::ImageSkia* icon,
-           base::ProcessHandle handle)
+           base::ProcessHandle handle,
+           base::ProcessId process_id)
     : task_id_(g_last_id++),
       network_usage_(-1),
       current_byte_count_(-1),
       title_(title),
       rappor_sample_name_(rappor_sample),
       icon_(icon ? *icon : gfx::ImageSkia()),
       process_handle_(handle),
-      process_id_(base::GetProcId(handle)) {
-}
+      process_id_(process_id != base::kNullProcessId
+                      ? process_id
+                      : base::GetProcId(handle)) {}
 
-Task::~Task() {
-}
+Task::~Task() {}
 
 // static
 base::string16 Task::GetProfileNameFromProfile(Profile* profile) {
   DCHECK(profile);
   ProfileAttributesEntry* entry;
   if (g_browser_process->profile_manager()->GetProfileAttributesStorage().
       GetProfileAttributesWithPath(profile->GetOriginalProfile()->GetPath(),
                                    &entry)) {
     return entry->GetName();
   }
@@ skipping 42 matching lines @@
 }
 
 base::string16 Task::GetProfileName() const {
   return base::string16();
 }
 
 int Task::GetTabId() const {
   return -1;
 }
 
+const Task* Task::GetParentTask() const {
+  return nullptr;
+}
+
 bool Task::ReportsSqliteMemory() const {
   return GetSqliteMemoryUsed() != -1;
 }
 
 int64_t Task::GetSqliteMemoryUsed() const {
   return -1;
 }
 
 bool Task::ReportsV8Memory() const {
   return GetV8MemoryAllocated() != -1;
@@ skipping 13 matching lines @@
 
 blink::WebCache::ResourceTypeStats Task::GetWebCacheStats() const {
   return blink::WebCache::ResourceTypeStats();
 }
 
 bool Task::ReportsNetworkUsage() const {
   return network_usage_ != -1;
 }
 
 }  // namespace task_management