Skeleton for showing "Show all saved passwords row" for Linux/CrOs/Windows platforms

chrome/browser/ui/autofill/chrome_autofill_client.cc

 // Copyright 2014 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/autofill/chrome_autofill_client.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/command_line.h"
@@ skipping 14 matching lines @@
 #include "chrome/browser/ui/autofill/create_card_unmask_prompt_view.h"
 #include "chrome/browser/ui/autofill/credit_card_scanner_controller.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/chrome_pages.h"
 #include "chrome/browser/ui/tabs/tab_strip_model_observer.h"
 #include "chrome/browser/vr/vr_tab_helper.h"
 #include "chrome/browser/web_data_service_factory.h"
 #include "chrome/common/url_constants.h"
 #include "components/autofill/content/browser/content_autofill_driver.h"
 #include "components/autofill/content/browser/content_autofill_driver_factory.h"
+#include "components/autofill/core/browser/popup_item_ids.h"
 #include "components/autofill/core/browser/ui/card_unmask_prompt_view.h"
 #include "components/autofill/core/common/autofill_pref_names.h"
 #include "components/autofill/core/common/autofill_switches.h"
 #include "components/browser_sync/profile_sync_service.h"
 #include "components/password_manager/content/browser/content_password_manager_driver.h"
+#include "components/password_manager/core/browser/password_manager_metrics_util.h"
 #include "components/prefs/pref_service.h"
 #include "components/signin/core/browser/profile_identity_provider.h"
 #include "components/signin/core/browser/signin_header_helper.h"
 #include "components/signin/core/browser/signin_metrics.h"
 #include "components/user_prefs/user_prefs.h"
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/render_frame_host.h"
 #include "content/public/browser/ssl_status.h"
 #include "ui/gfx/geometry/rect.h"
 
@@ skipping 315 matching lines @@
 
 bool ChromeAutofillClient::ShouldShowSigninPromo() {
 #if !defined(OS_ANDROID)
   return false;
 #else
   return signin::ShouldShowPromo(
       Profile::FromBrowserContext(web_contents()->GetBrowserContext()));
 #endif
 }
 
-void ChromeAutofillClient::StartSigninFlow() {
+bool ChromeAutofillClient::ExecuteCommand(int id) {
+  if (id == autofill::POPUP_ITEM_ID_HTTP_NOT_SECURE_WARNING_MESSAGE) {
+    password_manager::metrics_util::LogShowedHttpNotSecureExplanation();

[vasilii, 2017/07/26 12:17:36]

You double record this in autofill_external_delegate.cc

[melandory, 2017/07/26 15:01:30]

Done.

+    ShowHttpNotSecureExplanation();
+    return true;
+  } else if (id == autofill::POPUP_ITEM_ID_ALL_SAVED_PASSWORDS_ENTRY) {

[vasilii, 2017/07/26 12:17:36]

no need for "else"

[melandory, 2017/07/26 15:01:30]

Done.

+#if !defined(OS_ANDROID)
+    chrome::ShowSettingsSubPage(
+        chrome::FindBrowserWithWebContents(web_contents()),
+        chrome::kPasswordManagerSubPage);
+#endif
+
+    return true;
+  } else if (id == autofill::POPUP_ITEM_ID_CREDIT_CARD_SIGNIN_PROMO) {
 #if defined(OS_ANDROID)
   auto* window = web_contents()->GetNativeView()->GetWindowAndroid();

[vasilii, 2017/07/26 12:17:36]

align 

[melandory, 2017/07/26 15:01:30]

Done.

   if (window) {
     chrome::android::SigninPromoUtilAndroid::StartAccountSigninActivityForPromo(
         window, signin_metrics::AccessPoint::ACCESS_POINT_AUTOFILL_DROPDOWN);
   }
 #endif
+  return true;
+  }
+  return false;
 }
 
 void ChromeAutofillClient::ShowHttpNotSecureExplanation() {
 #if !defined(OS_ANDROID)
   // On desktop platforms, open Page Info, which briefly explains the HTTP
   // warning message and provides a link to the Help Center for more details.
   Browser* browser = chrome::FindBrowserWithWebContents(web_contents());
   if (browser && chrome::ShowPageInfo(browser, web_contents()))
     return;
 // Otherwise fall through to the section below that opens the URL directly.
@@ skipping 13 matching lines @@
 
 bool ChromeAutofillClient::IsAutofillSupported() {
   // VR browsing does not support popups at the moment.
   if (vr::VrTabHelper::IsInVr(web_contents()))
     return false;
 
   return true;
 }
 
 }  // namespace autofill