Stop compiling password infobar code on Mac.

chrome/browser/password_manager/password_manager_test_base.cc

 // Copyright 2015 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/password_manager/password_manager_test_base.h"
 
 #include <string>
 
 #include "base/macros.h"
 #include "base/run_loop.h"
 #include "base/strings/stringprintf.h"
-#include "chrome/browser/infobars/infobar_service.h"
 #include "chrome/browser/password_manager/chrome_password_manager_client.h"
 #include "chrome/browser/password_manager/password_store_factory.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/passwords/manage_passwords_ui_controller.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/test/base/ui_test_utils.h"
 #include "components/autofill/core/browser/autofill_test_utils.h"
-#include "components/infobars/core/confirm_infobar_delegate.h"
-#include "components/infobars/core/infobar.h"
-#include "components/infobars/core/infobar_manager.h"
 #include "components/password_manager/core/browser/password_manager_test_utils.h"
 #include "components/password_manager/core/browser/test_password_store.h"
 #include "components/password_manager/core/common/password_manager_features.h"
 #include "content/public/browser/navigation_details.h"
 #include "content/public/browser/render_frame_host.h"
 #include "content/public/test/browser_test_utils.h"
 #include "content/public/test/test_utils.h"
 #include "net/test/embedded_test_server/embedded_test_server.h"
 
 NavigationObserver::NavigationObserver(content::WebContents* web_contents)
@@ skipping 41 matching lines @@
   ASSERT_TRUE(IsShowingPrompt());
   AcceptImpl();
 }
 
 void PromptObserver::AcceptUpdatePrompt(
     const autofill::PasswordForm& form) const {
   EXPECT_TRUE(IsShowingUpdatePrompt());
   AcceptUpdatePromptImpl(form);
 }
 
-class InfoBarObserver : public PromptObserver,
-                        public infobars::InfoBarManager::Observer {
- public:
-  explicit InfoBarObserver(content::WebContents* web_contents)
-      : infobar_is_being_shown_(false),
-        infobar_service_(InfoBarService::FromWebContents(web_contents)) {
-    infobar_service_->AddObserver(this);
-  }
-
-  ~InfoBarObserver() override {
-    if (infobar_service_)
-      infobar_service_->RemoveObserver(this);
-  }
-
-  void Dismiss() const override {
-    NOTIMPLEMENTED();
-  }
-
- private:
-  // PromptObserver:
-  bool IsShowingPrompt() const override { return infobar_is_being_shown_; }
-
-  void AcceptImpl() const override {
-    EXPECT_EQ(1u, infobar_service_->infobar_count());
-    if (!infobar_service_->infobar_count())
-      return;  // Let the test finish to gather possibly more diagnostics.
-
-    // ConfirmInfoBarDelegate::Accept returning true means the infobar is
-    // immediately closed. Checking the return value is preferred to testing
-    // IsShowingPrompt() here, for it avoids the delay until the closing
-    // notification is received.
-    EXPECT_TRUE(infobar_service_->infobar_at(0)
-                    ->delegate()
-                    ->AsConfirmInfoBarDelegate()
-                    ->Accept());
-  }
-
-  // infobars::InfoBarManager::Observer:
-  void OnInfoBarAdded(infobars::InfoBar* infobar) override {
-    infobar_is_being_shown_ = true;
-  }
-
-  void OnInfoBarRemoved(infobars::InfoBar* infobar, bool animate) override {
-    infobar_is_being_shown_ = false;
-  }
-
-  void OnManagerShuttingDown(infobars::InfoBarManager* manager) override {
-    ASSERT_EQ(infobar_service_, manager);
-    infobar_service_->RemoveObserver(this);
-    infobar_service_ = nullptr;
-  }
-
-  bool infobar_is_being_shown_;
-  InfoBarService* infobar_service_;
-
-  DISALLOW_COPY_AND_ASSIGN(InfoBarObserver);
-};
-
 class BubbleObserver : public PromptObserver {

[vabr (Chromium), 2016/05/12 16:19:29]

We should merge PromptObserver into BubbleObserver, because there is no other
PromptObserver subclass.

[vasilii, 2016/05/12 16:23:30]

Yes, but I refrain from renaming in this CL.

  public:
   explicit BubbleObserver(content::WebContents* web_contents)
       : passwords_model_delegate_(
             PasswordsModelDelegateFromWebContents(web_contents)) {}
 
   ~BubbleObserver() override {}
 
   void Dismiss() const override {
     passwords_model_delegate_->OnBubbleHidden();
     // Navigate away to reset the state to inactive.
@@ skipping 27 matching lines @@
     passwords_model_delegate_->UpdatePassword(form);
     EXPECT_FALSE(IsShowingUpdatePrompt());
   }
   PasswordsModelDelegate* const passwords_model_delegate_;
 
   DISALLOW_COPY_AND_ASSIGN(BubbleObserver);
 };
 
 std::unique_ptr<PromptObserver> PromptObserver::Create(
     content::WebContents* web_contents) {
-  if (ChromePasswordManagerClient::IsTheHotNewBubbleUIEnabled()) {
-    return std::unique_ptr<PromptObserver>(new BubbleObserver(web_contents));
-  } else {
-    return std::unique_ptr<PromptObserver>(new InfoBarObserver(web_contents));
-  }
+  return base::WrapUnique(new BubbleObserver(web_contents));
 }
 
 PasswordManagerBrowserTestBase::PasswordManagerBrowserTestBase() {
 }
 PasswordManagerBrowserTestBase::~PasswordManagerBrowserTestBase() {
 }
 
 void PasswordManagerBrowserTestBase::SetUpOnMainThread() {
   // Use TestPasswordStore to remove a possible race. Normally the
   // PasswordStore does its database manipulation on the DB thread, which
@@ skipping 145 matching lines @@
       "  var element = document.getElementById('%s');"
       "window.domAutomationController.send(element && element.value == '%s');",
       iframe_id.c_str(), iframe_id.c_str(), element_id.c_str(),
       element_id.c_str(), expected_value.c_str());
   bool return_value = false;
   ASSERT_TRUE(content::ExecuteScriptAndExtractBool(
       RenderViewHost(), value_check_script, &return_value));
   EXPECT_TRUE(return_value) << "element_id = " << element_id
                             << ", expected_value = " << expected_value;
 }