Improve the about/discards UI, allowing each tab to be individually closed.

chrome/browser/ui/webui/about_ui.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/ui/webui/about_ui.h"
 
 #include <algorithm>
 #include <string>
 #include <utility>
 #include <vector>
@@ skipping 11 matching lines @@
 #include "base/strings/string_piece.h"
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/threading/thread.h"
 #include "base/values.h"
 #include "chrome/browser/about_flags.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/defaults.h"
 #include "chrome/browser/memory/oom_priority_manager.h"
+#include "chrome/browser/memory/tab_stats.h"
 #include "chrome/browser/memory_details.h"
 #include "chrome/browser/net/predictor.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/browser_dialogs.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/render_messages.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/grit/chromium_strings.h"
 #include "chrome/grit/generated_resources.h"
@@ skipping 409 matching lines @@
       "them into the address bar if you need them.</p>\n<ul>";
   for (int i = 0; i < chrome::kNumberOfChromeDebugURLs; i++)
     html += "<li>" + std::string(chrome::kChromeDebugURLs[i]) + "</li>\n";
   html += "</ul>\n";
   AppendFooter(&html);
   return html;
 }
 
 #if defined(OS_WIN) || defined(OS_CHROMEOS)
 
+const char kAboutDiscardsRunCommand[] = "run";
+
 // Html output helper functions
 
 // Helper function to wrap HTML with a tag.
 std::string WrapWithTag(const std::string& tag, const std::string& text) {
   return "<" + tag + ">" + text + "</" + tag + ">";
 }
 
 // Helper function to wrap Html with <td> tag.
 std::string WrapWithTD(const std::string& text) {
   return "<td>" + text + "</td>";
@@ skipping 11 matching lines @@
   return WrapWithTR(row);
 }
 
 void AddContentSecurityPolicy(std::string* output) {
   output->append("<meta http-equiv='Content-Security-Policy' "
       "content='default-src 'none';'>");
 }
 
 // TODO(stevenjb): L10N AboutDiscards.
 
-std::string AboutDiscardsRun() {
+std::string BuildAboutDiscardsRunPage() {
   std::string output;
   AppendHeader(&output, 0, "About discards");
-  output.append(
-      base::StringPrintf("<meta http-equiv='refresh' content='2;%s'>",
-      chrome::kChromeUIDiscardsURL));
+  output.append(base::StringPrintf("<meta http-equiv='refresh' content='2;%s'>",
+                                   chrome::kChromeUIDiscardsURL));
   AddContentSecurityPolicy(&output);
   output.append(WrapWithTag("p", "Discarding a tab..."));
-  g_browser_process->GetOomPriorityManager()->LogMemoryAndDiscardTab();
   AppendFooter(&output);
   return output;
 }
 
+std::vector<std::string> GetHtmlTabDescriptorsForDiscardPage() {
+  memory::OomPriorityManager* oom = g_browser_process->GetOomPriorityManager();
+  memory::TabStatsList stats = oom->GetTabStats();
+  std::vector<std::string> titles;
+  titles.reserve(stats.size());
+  for (memory::TabStatsList::iterator it = stats.begin(); it != stats.end();
+       ++it) {
+    std::string str;
+    str.reserve(4096);
+    str += "<b>";
+    str += it->is_app ? "[App] " : "";
+    str += it->is_internal_page ? "[Internal] " : "";
+    str += it->is_playing_audio ? "[Audio] " : "";
+    str += it->is_pinned ? "[Pinned] " : "";
+    str += it->is_discarded ? "[Discarded] " : "";
+    str += "</b>";
+    str += net::EscapeForHTML(base::UTF16ToUTF8(it->title));
+#if defined(OS_CHROMEOS)
+    str += base::StringPrintf(" (%d) ", it->oom_score);
+#endif
+    if (!it->is_discarded) {
+      str += base::StringPrintf(" <a href='%s%s/%" PRId64 "'>Discard</a>",
+                                chrome::kChromeUIDiscardsURL,
+                                kAboutDiscardsRunCommand, it->tab_contents_id);
+    }
+    titles.push_back(str);
+  }
+  return titles;
+}
+
 std::string AboutDiscards(const std::string& path) {
   std::string output;
-  const char kRunCommand[] = "run";
-  if (path == kRunCommand)
-    return AboutDiscardsRun();
+  std::vector<std::string> path_split;
+  int64 web_content_id;
+  memory::OomPriorityManager* oom = g_browser_process->GetOomPriorityManager();
+
+  base::SplitString(path, '/', &path_split);
+  if (path_split.size() == 2 && path_split[0] == kAboutDiscardsRunCommand &&
+      base::StringToInt64(path_split[1], &web_content_id)) {
+    oom->DiscardTabById(web_content_id);
+    return BuildAboutDiscardsRunPage();
+  } else if (path_split.size() == 1 &&
+             path_split[0] == kAboutDiscardsRunCommand) {
+    oom->DiscardTab();
+    return BuildAboutDiscardsRunPage();
+  }
+
   AppendHeader(&output, 0, "About discards");
   AddContentSecurityPolicy(&output);
   AppendBody(&output);
-  output.append("<h3>About discards</h3>");
+  output.append("<h3>Discarded Tabs</h3>");
   output.append(
       "<p>Tabs sorted from most interesting to least interesting. The least "
       "interesting tab may be discarded if we run out of physical memory.</p>");
 
-  memory::OomPriorityManager* oom = g_browser_process->GetOomPriorityManager();
-  std::vector<base::string16> titles = oom->GetTabTitles();
+  std::vector<std::string> titles = GetHtmlTabDescriptorsForDiscardPage();
   if (!titles.empty()) {
     output.append("<ul>");
-    std::vector<base::string16>::iterator it = titles.begin();
+    std::vector<std::string>::iterator it = titles.begin();
     for ( ; it != titles.end(); ++it) {
-      std::string title = base::UTF16ToUTF8(*it);
-      title = net::EscapeForHTML(title);
-      output.append(WrapWithTag("li", title));
+      output.append(WrapWithTag("li", *it));
     }
     output.append("</ul>");
   } else {
     output.append("<p>None found.  Wait 10 seconds, then refresh.</p>");
   }
-  output.append(base::StringPrintf("%d discards this session. ",
-                             oom->discard_count()));
+  output.append(
+      base::StringPrintf("%d discards this session. ", oom->discard_count()));
   output.append(base::StringPrintf("<a href='%s%s'>Discard tab now</a>",
                                    chrome::kChromeUIDiscardsURL,
-                                   kRunCommand));
+                                   kAboutDiscardsRunCommand));
+
 #if defined(OS_CHROMEOS)
   base::SystemMemoryInfoKB meminfo;
   base::GetSystemMemoryInfo(&meminfo);
   output.append("<h3>System memory information in MB</h3>");
   output.append("<table>");
   // Start with summary statistics.
   output.append(AddStringRow(
       "Total", base::IntToString(meminfo.total / 1024)));
   output.append(AddStringRow(
       "Free", base::IntToString(meminfo.free / 1024)));
@@ skipping 439 matching lines @@
   Profile* profile = Profile::FromWebUI(web_ui);
 
 #if defined(ENABLE_THEMES)
   // Set up the chrome://theme/ source.
   ThemeSource* theme = new ThemeSource(profile);
   content::URLDataSource::Add(profile, theme);
 #endif
 
   content::URLDataSource::Add(profile, new AboutUIHTMLSource(name, profile));
 }