MD Downloads: reset list tracker when renderer crashes

chrome/browser/ui/webui/md_downloads/md_downloads_dom_handler.cc

Index: chrome/browser/ui/webui/md_downloads/md_downloads_dom_handler.cc
diff --git a/chrome/browser/ui/webui/md_downloads/md_downloads_dom_handler.cc b/chrome/browser/ui/webui/md_downloads/md_downloads_dom_handler.cc
index c980c14d4b6d0c146c20345aa2913a74468e5bd6..966a36461cf4a2877628465dfca17409e561aec6 100644
--- a/chrome/browser/ui/webui/md_downloads/md_downloads_dom_handler.cc
+++ b/chrome/browser/ui/webui/md_downloads/md_downloads_dom_handler.cc
@@ -131,6 +131,8 @@ void MdDownloadsDOMHandler::RegisterMessages() {
   web_ui()->RegisterMessageCallback("openDownloadsFolder",
       base::Bind(&MdDownloadsDOMHandler::HandleOpenDownloadsFolder,
                  weak_ptr_factory_.GetWeakPtr()));
+
+  Observe(GetWebUIWebContents());
 }
 
 void MdDownloadsDOMHandler::OnJavascriptDisallowed() {
@@ -139,6 +141,14 @@ void MdDownloadsDOMHandler::OnJavascriptDisallowed() {
   CheckForRemovedFiles();
 }
 
+void MdDownloadsDOMHandler::RenderProcessGone(base::TerminationStatus status) {
+  if (status != base::TERMINATION_STATUS_NORMAL_TERMINATION &&

[tommycli, 2016/05/09 17:54:35]

Does using WebContents::IsCrashed() work here instead?

Also... maybe we should just always DisallowJavascript. I can't think of any
case where the RenderProcess is Gone (even for non-crash reasons) that we would
not want to disallow JS.

[Dan Beam, 2016/05/09 18:29:32]

Done.

+      status != base::TERMINATION_STATUS_STILL_RUNNING) {
+    // TODO(dbeam): WebUI + WebUIMessageHandler should do this automatically.
+    DisallowJavascript();
+  }
+}
+
 void MdDownloadsDOMHandler::HandleGetDownloads(const base::ListValue* args) {
   AllowJavascript();