Hook up the Mac password bubble to the browser and add browser tests.

chrome/browser/password_manager/chrome_password_manager_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/password_manager/chrome_password_manager_client.h"
 
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
 #include "base/memory/singleton.h"
 #include "base/metrics/histogram.h"
@@ skipping 336 matching lines @@
       web_contents()->GetRenderViewHost()->GetRoutingID(),
       can_use_log_router_));
 }
 
 void ChromePasswordManagerClient::CommitFillPasswordForm(
     autofill::PasswordFormFillData* data) {
   driver_.FillPasswordForm(*data);
 }
 
 bool ChromePasswordManagerClient::IsTheHotNewBubbleUIEnabled() {
-#if !defined(USE_AURA)
+#if !defined(USE_AURA) && !defined(OS_MACOSX)

[vabr (Chromium), 2014/08/05 13:14:26]

This is within //chrome, which means desktop only. I believe it is safe to just
remove lines 357-359 altogether.

[dconnelly, 2014/08/06 12:46:12]

Done.

   return false;
 #endif
   CommandLine* command_line = CommandLine::ForCurrentProcess();
   if (command_line->HasSwitch(switches::kDisableSavePasswordBubble))
     return false;
 
   if (command_line->HasSwitch(switches::kEnableSavePasswordBubble))
     return true;
 
   std::string group_name =
@@ skipping 11 matching lines @@
 
   if (command_line->HasSwitch(
           password_manager::switches::kEnableManagerForSyncSignin))
     return true;
 
   // Default is enabled.
   std::string group_name =
       base::FieldTrialList::FindFullName("PasswordManagerStateForSyncSignin");
   return group_name != "Disabled";
 }