[Password Manager] Show omnibox icon and anchored prompt once user start typing password

chrome/browser/ui/passwords/manage_passwords_ui_controller_unittest.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 <map>
 #include <memory>
 #include <utility>
 #include <vector>
 
 #include "base/bind.h"
@@ skipping 787 matching lines @@
   // Open the bubble again.
   local_credentials.emplace_back(new autofill::PasswordForm(test_local_form()));
   EXPECT_CALL(*controller(), OnUpdateBubbleAndIconVisibility());
   controller()->OnAutoSignin(std::move(local_credentials),
                              test_local_form().origin);
   EXPECT_EQ(password_manager::ui::AUTO_SIGNIN_STATE, controller()->GetState());
   // Check the delegate is destroyed. Thus, the first bubble has no way to mess
   // up with the controller's state.
   EXPECT_FALSE(proxy_delegate);
 }
+
+TEST_F(ManagePasswordsUIControllerTest, ManualFallbackForSaving_UseFallback) {
+  for (bool is_update : {false, true}) {
+    SCOPED_TRACE(testing::Message("is_update = ") << is_update);
+    std::unique_ptr<password_manager::PasswordFormManager> test_form_manager(
+        CreateFormManager());
+    test_form_manager->ProvisionallySave(
+        test_local_form(),
+        password_manager::PasswordFormManager::IGNORE_OTHER_POSSIBLE_USERNAMES);
+    EXPECT_CALL(*controller(), OnUpdateBubbleAndIconVisibility());
+    controller()->OnShowManualFallback(std::move(test_form_manager), is_update);

[vasilii, 2017/07/21 12:48:19]

You can also check opened_bubble()

[kolos1, 2017/07/24 15:33:30]

Done.

+    ExpectIconStateIs(is_update

[vasilii, 2017/07/21 12:48:20]

Why not ExpectIconAndControllerStateIs?

[kolos1, 2017/07/24 15:33:30]

Done.

+                          ? password_manager::ui::PENDING_PASSWORD_UPDATE_STATE
+                          : password_manager::ui::PENDING_PASSWORD_STATE);
+
+    if (is_update) {
+      EXPECT_CALL(*controller(), OnUpdateBubbleAndIconVisibility());

[vasilii, 2017/07/21 12:48:19]

Why do we expect it only here?

[kolos1, 2017/07/24 15:33:30]

It is strange to me too. UpdatePassword calls UpdateBubbleAndIconVisibility, but
SavePassword doesn't.

[vasilii, 2017/07/25 11:17:46]

Ahh, that's my own trick. The function still should be called. But after
OnBubbleHidden(). In the real life that method always follows
UpdatePassword/SavePassword

No need to address.

[kolos1, 2017/07/26 13:40:57]

Acknowledged.

+      controller()->UpdatePassword(autofill::PasswordForm());
+    } else {
+      controller()->SavePassword(test_local_form().username_value);
+    }
+    ExpectIconStateIs(password_manager::ui::MANAGE_STATE);
+  }
+}
+
+TEST_F(ManagePasswordsUIControllerTest, ManualFallbackForSaving_HideFallback) {
+  for (bool is_update : {false, true}) {
+    SCOPED_TRACE(testing::Message("is_update = ") << is_update);
+    std::unique_ptr<password_manager::PasswordFormManager> test_form_manager(
+        CreateFormManager());
+    test_form_manager->ProvisionallySave(
+        test_local_form(),
+        password_manager::PasswordFormManager::IGNORE_OTHER_POSSIBLE_USERNAMES);
+    EXPECT_CALL(*controller(), OnUpdateBubbleAndIconVisibility());
+    controller()->OnShowManualFallback(std::move(test_form_manager), is_update);
+    ExpectIconStateIs(is_update
+                          ? password_manager::ui::PENDING_PASSWORD_UPDATE_STATE
+                          : password_manager::ui::PENDING_PASSWORD_STATE);
+
+    EXPECT_CALL(*controller(), OnUpdateBubbleAndIconVisibility());
+    controller()->OnHideManualFallback();
+    ExpectIconStateIs(password_manager::ui::MANAGE_STATE);
+  }
+}
+
+TEST_F(ManagePasswordsUIControllerTest,
+       ManualFallbackForSaving_GeneratedPassword) {
+  for (bool user_closed_bubble : {false, true}) {
+    SCOPED_TRACE(testing::Message("user_closed_bubble = ")
+                 << user_closed_bubble);
+    std::unique_ptr<password_manager::PasswordFormManager> test_form_manager(
+        CreateFormManager());
+    test_form_manager->ProvisionallySave(
+        test_local_form(),
+        password_manager::PasswordFormManager::IGNORE_OTHER_POSSIBLE_USERNAMES);
+    test_form_manager->SetHasGeneratedPassword(true);
+    EXPECT_CALL(*controller(), OnUpdateBubbleAndIconVisibility());
+    controller()->OnShowManualFallback(std::move(test_form_manager), false);
+    ExpectIconStateIs(password_manager::ui::CONFIRMATION_STATE);
+
+    EXPECT_CALL(*controller(), OnUpdateBubbleAndIconVisibility());
+    if (user_closed_bubble)
+      controller()->OnBubbleHidden();
+    else  // The user removed the generated password.
+      controller()->OnHideManualFallback();

[vasilii, 2017/07/21 12:48:20]

I need more info what's being tested here.
OnBubbleHidden() is a bug because you don't pop up the bubble.

[kolos1, 2017/07/24 15:33:29]

Added to comments to all tests.

+    ExpectIconStateIs(password_manager::ui::MANAGE_STATE);
+  }
+}