Remove "from all browsers" memory details mode

chrome/browser/memory_details_win.cc

Index: chrome/browser/memory_details_win.cc
diff --git a/chrome/browser/memory_details_win.cc b/chrome/browser/memory_details_win.cc
index e0957bc589b81329ff616929e763bdf745b1980c..7764c39ffff5b496d70c835d563341e289082f11 100644
--- a/chrome/browser/memory_details_win.cc
+++ b/chrome/browser/memory_details_win.cc
@@ -73,7 +73,6 @@ ProcessData* MemoryDetails::ChromeBrowser() {
 }
 
 void MemoryDetails::CollectProcessData(
-    CollectionMode mode,
     const std::vector<ProcessMemoryInformation>& child_info) {
   DCHECK(BrowserThread::GetBlockingPool()->RunsTasksOnCurrentThread());
 
@@ -106,8 +105,7 @@ void MemoryDetails::CollectProcessData(
          (windows_architecture == base::win::OSInfo::IA64_ARCHITECTURE)) &&
         (base::win::OSInfo::GetWOW64StatusForProcess(process_handle.Get()) ==
             base::win::OSInfo::WOW64_DISABLED);
-    const size_t browser_list_size =
-        (mode == FROM_CHROME_ONLY ? 1 : process_data_.size());
+    const size_t browser_list_size = 1;
     for (size_t index2 = 0; index2 < browser_list_size; ++index2) {

[Nico, 2016/04/08 19:47:44]

this loop now always runs exactly once

[Elly Fong-Jones, 2016/04/13 17:18:24]

I'm scared of this code and it has no tests that I can find, so I hacked a ton
of it out and we'll see if anything breaks.

       if (_wcsicmp(process_data_[index2].process_name.c_str(),
                    process_entry.szExeFile) != 0)