Make JavaScript dialogs auto-dismiss on tab switch.

components/app_modal/javascript_dialog_manager.cc

Index: components/app_modal/javascript_dialog_manager.cc
diff --git a/components/app_modal/javascript_dialog_manager.cc b/components/app_modal/javascript_dialog_manager.cc
index 7628f213971b00f14c59ec058dd2c1b7dc228e99..4998cad521ac53ae642b2937d4622fde06bd8995 100644
--- a/components/app_modal/javascript_dialog_manager.cc
+++ b/components/app_modal/javascript_dialog_manager.cc
@@ -70,9 +70,6 @@ void LogUMAMessageLengthStats(const base::string16& message) {
 
 }  // namespace
 
-////////////////////////////////////////////////////////////////////////////////
-// JavaScriptDialogManager, public:
-
 // static
 JavaScriptDialogManager* JavaScriptDialogManager::GetInstance() {
   return base::Singleton<JavaScriptDialogManager>::get();
@@ -88,9 +85,6 @@ void JavaScriptDialogManager::SetExtensionsClient(
   extensions_client_ = std::move(extensions_client);
 }
 
-////////////////////////////////////////////////////////////////////////////////
-// JavaScriptDialogManager, private:
-
 JavaScriptDialogManager::JavaScriptDialogManager()
     : extensions_client_(new DefaultExtensionsClient) {
 }
@@ -98,6 +92,39 @@ JavaScriptDialogManager::JavaScriptDialogManager()
 JavaScriptDialogManager::~JavaScriptDialogManager() {
 }
 
+base::string16 JavaScriptDialogManager::GetTitle(
+    content::WebContents* web_contents,
+    const GURL& origin_url) {
+  // For extensions, show the extension name, but only if the origin of
+  // the alert matches the top-level WebContents.
+  std::string name;
+  if (extensions_client_->GetExtensionName(web_contents, origin_url, &name))
+    return base::UTF8ToUTF16(name);
+
+  // Otherwise, return the formatted URL. For non-standard URLs such as |data:|,
+  // just say "This page".
+  bool is_same_origin_as_main_frame =
+      (web_contents->GetURL().GetOrigin() == origin_url.GetOrigin());
+  if (origin_url.IsStandard() && !origin_url.SchemeIsFile() &&
+      !origin_url.SchemeIsFileSystem()) {
+#if !defined(OS_ANDROID)

[Peter Kasting, 2016/10/18 04:31:04]

Nit: I know you're just moving this code... but could you reverse the arms of
this #if?  "#if !X...else" makes my head hurt.

[Avi (use Gerrit), 2016/10/18 16:31:49]

Done.

+    base::string16 url_string =
+        url_formatter::ElideHost(origin_url, gfx::FontList(), kUrlElideWidth);
+#else
+    base::string16 url_string = url_formatter::FormatUrlForSecurityDisplay(
+        origin_url, url_formatter::SchemeDisplay::OMIT_HTTP_AND_HTTPS);
+#endif
+    return l10n_util::GetStringFUTF16(
+        is_same_origin_as_main_frame ? IDS_JAVASCRIPT_MESSAGEBOX_TITLE
+                                     : IDS_JAVASCRIPT_MESSAGEBOX_TITLE_IFRAME,
+        base::i18n::GetDisplayStringInLTRDirectionality(url_string));
+  }
+  return l10n_util::GetStringUTF16(
+      is_same_origin_as_main_frame
+          ? IDS_JAVASCRIPT_MESSAGEBOX_TITLE_NONSTANDARD_URL
+          : IDS_JAVASCRIPT_MESSAGEBOX_TITLE_NONSTANDARD_URL_IFRAME);
+}
+
 void JavaScriptDialogManager::RunJavaScriptDialog(
     content::WebContents* web_contents,
     const GURL& origin_url,
@@ -150,8 +177,7 @@ void JavaScriptDialogManager::RunJavaScriptDialog(
     last_close_time_ = base::TimeTicks();
   }
 
-  bool is_alert = message_type == content::JAVASCRIPT_MESSAGE_TYPE_ALERT;
-  base::string16 dialog_title = GetTitle(web_contents, origin_url, is_alert);
+  base::string16 dialog_title = GetTitle(web_contents, origin_url);
 
   extensions_client_->OnDialogOpened(web_contents);
 
@@ -241,40 +267,6 @@ bool JavaScriptDialogManager::HandleJavaScriptDialog(
   return true;
 }
 
-base::string16 JavaScriptDialogManager::GetTitle(
-    content::WebContents* web_contents,
-    const GURL& origin_url,
-    bool is_alert) {
-  // For extensions, show the extension name, but only if the origin of
-  // the alert matches the top-level WebContents.
-  std::string name;
-  if (extensions_client_->GetExtensionName(web_contents, origin_url, &name))
-    return base::UTF8ToUTF16(name);
-
-  // Otherwise, return the formatted URL. For non-standard URLs such as |data:|,
-  // just say "This page".
-  bool is_same_origin_as_main_frame =
-      (web_contents->GetURL().GetOrigin() == origin_url.GetOrigin());
-  if (origin_url.IsStandard() && !origin_url.SchemeIsFile() &&
-      !origin_url.SchemeIsFileSystem()) {
-#if !defined(OS_ANDROID)
-    base::string16 url_string =
-        url_formatter::ElideHost(origin_url, gfx::FontList(), kUrlElideWidth);
-#else
-    base::string16 url_string = url_formatter::FormatUrlForSecurityDisplay(
-        origin_url, url_formatter::SchemeDisplay::OMIT_HTTP_AND_HTTPS);
-#endif
-    return l10n_util::GetStringFUTF16(
-        is_same_origin_as_main_frame ? IDS_JAVASCRIPT_MESSAGEBOX_TITLE
-                                     : IDS_JAVASCRIPT_MESSAGEBOX_TITLE_IFRAME,
-        base::i18n::GetDisplayStringInLTRDirectionality(url_string));
-  }
-  return l10n_util::GetStringUTF16(
-      is_same_origin_as_main_frame
-          ? IDS_JAVASCRIPT_MESSAGEBOX_TITLE_NONSTANDARD_URL
-          : IDS_JAVASCRIPT_MESSAGEBOX_TITLE_NONSTANDARD_URL_IFRAME);
-}
-
 void JavaScriptDialogManager::CancelDialogs(content::WebContents* web_contents,
                                             bool suppress_callbacks,
                                             bool reset_state) {