Removed the ProcessHandle from the RendererClosedDetails payload

chrome/browser/chromeos/memory/oom_priority_manager.h

 // 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_MEMORY_OOM_PRIORITY_MANAGER_H_
 #define CHROME_BROWSER_CHROMEOS_MEMORY_OOM_PRIORITY_MANAGER_H_
 
+#include <utility>
 #include <vector>
 
 #include "base/compiler_specific.h"
 #include "base/containers/hash_tables.h"
 #include "base/gtest_prod_util.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/process/process.h"
 #include "base/strings/string16.h"
 #include "base/synchronization/lock.h"
 #include "base/time/time.h"
@@ skipping 54 matching lines @@
     TabStats();
     ~TabStats();
     bool is_app;  // browser window is an app
     bool is_reloadable_ui;  // Reloadable web UI page, like NTP or Settings.
     bool is_playing_audio;
     bool is_pinned;
     bool is_selected;  // selected in the currently active browser window
     bool is_discarded;
     base::TimeTicks last_active;
     base::ProcessHandle renderer_handle;
+    int child_process_host_id;
     base::string16 title;
     int64 tab_contents_id;  // unique ID per WebContents
   };
   typedef std::vector<TabStats> TabStatsList;
 
   // Returns true if the |url| represents an internal Chrome web UI page that
   // can be easily reloaded and hence makes a good choice to discard.
   static bool IsReloadableUI(const GURL& url);
 
   // Discards a tab with the given unique ID.  Returns true if discard occurred.
@@ skipping 12 matching lines @@
   void PurgeBrowserMemory();
 
   // Returns the number of tabs open in all browser instances.
   int GetTabCount() const;
 
   TabStatsList GetTabStatsOnUIThread();
 
   // Called when the timer fires, sets oom_adjust_score for all renderers.
   void AdjustOomPriorities();
 
-  // Returns a list of unique process handles from |stats_list|. If multiple
-  // tabs use the same process, returns the first process handle. This implies
+  // Pair to hold child process host id and ProcessHandle
+  typedef std::pair<int, base::ProcessHandle> ProcessInfo;
+
+  // Returns a list of child process host ids and ProcessHandle from

[James Cook, 2014/12/01 19:25:58]

nit: ProcessHandles

[sramajay, 2014/12/02 15:36:58]

Done.

+  // |stats_list| with unique pids. If multiple tabs use the same process,
+  // returns the first child process host id and correspoding pid. This implies

[James Cook, 2014/12/01 19:25:58]

nit: corresponding

[sramajay, 2014/12/02 15:36:58]

Done.

   // that the processes are selected based on their "most important" tab.
-  static std::vector<base::ProcessHandle> GetProcessHandles(
+  static std::vector<ProcessInfo> GetChildProcessInfos(
       const TabStatsList& stats_list);
 
   // Called by AdjustOomPriorities.
   void AdjustOomPrioritiesOnFileThread(TabStatsList stats_list);
 
   // Posts AdjustFocusedTabScore task to the file thread.
   void OnFocusTabScoreAdjustmentTimeout();
 
   // Sets the score of the focused tab to the least value.
   void AdjustFocusedTabScoreOnFileThread();
 
   static bool CompareTabStats(TabStats first, TabStats second);
 
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) override;
 
   base::RepeatingTimer<OomPriorityManager> timer_;
   base::OneShotTimer<OomPriorityManager> focus_tab_score_adjust_timer_;
   base::RepeatingTimer<OomPriorityManager> recent_tab_discard_timer_;
   content::NotificationRegistrar registrar_;
 
-  // This lock is for pid_to_oom_score_ and focus_tab_pid_.
-  base::Lock pid_to_oom_score_lock_;
-  // map maintaining the process - oom_score mapping.
-  typedef base::hash_map<base::ProcessHandle, int> ProcessScoreMap;
-  ProcessScoreMap pid_to_oom_score_;
-  base::ProcessHandle focused_tab_pid_;
+  // This lock is for |oom_score_map_| and |focused_tab_process_info_|.
+  base::Lock oom_score_lock_;
+  // Map maintaining the child process host id - oom_score mapping.
+  typedef base::hash_map<int, int> ProcessScoreMap;
+  ProcessScoreMap oom_score_map_;
+  // Holds the focused tab's child process host id.
+  ProcessInfo focused_tab_process_info_;
 
   // Observer for the kernel low memory signal.  NULL if tab discarding is
   // disabled.
   scoped_ptr<LowMemoryObserver> low_memory_observer_;
 
   // Wall-clock time when the priority manager started running.
   base::TimeTicks start_time_;
 
   // Wall-clock time of last tab discard during this browsing session, or 0 if
   // no discard has happened yet.
   base::TimeTicks last_discard_time_;
 
   // Wall-clock time of last priority adjustment, used to correct the above
   // times for discontinuities caused by suspend/resume.
   base::TimeTicks last_adjust_time_;
 
   // Number of times we have discarded a tab, for statistics.
   int discard_count_;
 
   // Whether a tab discard event has occurred during the last time interval,
   // used for statistics normalized by usage.
   bool recent_tab_discard_;
 
   DISALLOW_COPY_AND_ASSIGN(OomPriorityManager);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_MEMORY_OOM_PRIORITY_MANAGER_H_