cros: Log per-process memory use on low memory events

chrome/browser/automation/automation_provider_observers.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/automation/automation_provider_observers.h"
 
 #include <deque>
 #include <string>
 #include <vector>
 
@@ skipping 2700 matching lines @@
 
 ProcessInfoObserver::ProcessInfoObserver(
     AutomationProvider* automation,
     IPC::Message* reply_message)
     : automation_(automation->AsWeakPtr()),
       reply_message_(reply_message) {}
 
 ProcessInfoObserver::~ProcessInfoObserver() {}
 
 void ProcessInfoObserver::OnDetailsAvailable() {
+  // TODO(jamescook): Remove this.  I don't think we want it for automated
+  // tests, but it's how the system has always worked.

[Greg Spencer (Chromium), 2012/04/20 00:28:21]

Can you elaborate on what "this" and "it's" refer to here? (I mean, I know
"this" refers to the function call, but what does the function call imply?)
Otherwise I'm unclear why you added it if you put in a TODO to remove it.

+  UpdateHistograms();
+
   scoped_ptr<DictionaryValue> return_value(new DictionaryValue);
   ListValue* browser_proc_list = new ListValue();
   const std::vector<ProcessData>& all_processes = processes();
   for (size_t index = 0; index < all_processes.size(); ++index) {
     DictionaryValue* browser_data = new DictionaryValue();
     browser_data->SetString("name", all_processes[index].name);
     browser_data->SetString("process_name", all_processes[index].process_name);
 
     ListValue* proc_list = new ListValue();
     for (ProcessMemoryInformationList::const_iterator iterator =
@@ skipping 322 matching lines @@
   }
 
   ExtensionHost* host = content::Details<ExtensionHost>(details).ptr();
   if (host->extension_id() == extension_id_ &&
       host->extension_host_type() == chrome::VIEW_TYPE_EXTENSION_POPUP) {
     AutomationJSONReply(automation_, reply_message_.release())
         .SendSuccess(NULL);
     delete this;
   }
 }