Implement the Chrome Sign-in promo bubble for password manager on Views.

chrome/browser/ui/views/passwords/manage_passwords_bubble_view.cc

 // Copyright 2013 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/views/passwords/manage_passwords_bubble_view.h"
 
 #include "base/macros.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/timer/timer.h"
 #include "chrome/browser/profiles/profile.h"
@@ skipping 334 matching lines @@
 
   parent_->set_initially_focused_view(save_button_);
 }
 
 ManagePasswordsBubbleView::PendingView::~PendingView() {
 }
 
 void ManagePasswordsBubbleView::PendingView::ButtonPressed(
     views::Button* sender,
     const ui::Event& event) {
-  if (sender == save_button_)
+  if (sender == save_button_) {
     parent_->model()->OnSaveClicked();
-  else if (sender == never_button_)
+    if (parent_->model()->ShouldShowSignInPromo()) {
+      parent_->Refresh();
+      return;
+    }
+  } else if (sender == never_button_) {
     parent_->model()->OnNeverForThisSiteClicked();
-  else
+  } else {
     NOTREACHED();
+  }
 
   parent_->CloseBubble();
 }
 
 void ManagePasswordsBubbleView::PendingView::StyledLabelLinkClicked(
     views::StyledLabel* label,
     const gfx::Range& range,
     int event_flags) {
   DCHECK_EQ(range, parent_->model()->title_brand_link_range());
   parent_->model()->OnBrandLinkClicked();
@@ skipping 170 matching lines @@
   parent_->CloseBubble();
 }
 
 void ManagePasswordsBubbleView::SaveConfirmationView::ButtonPressed(
     views::Button* sender, const ui::Event& event) {
   DCHECK_EQ(sender, ok_button_);
   parent_->model()->OnOKClicked();
   parent_->CloseBubble();
 }
 
+// ManagePasswordsBubbleView::SignInPromoView ---------------------------------
+
+// A view that offers user to sign in to Chrome.
+class ManagePasswordsBubbleView::SignInPromoView
+    : public views::View,
+      public views::ButtonListener {
+ public:
+  explicit SignInPromoView(ManagePasswordsBubbleView* parent);
+
+ private:
+  // views::ButtonListener:
+  void ButtonPressed(views::Button* sender, const ui::Event& event) override;
+
+  ManagePasswordsBubbleView* parent_;
+
+  views::Button* signin_button_;
+  views::Button* no_button_;
+
+  DISALLOW_COPY_AND_ASSIGN(SignInPromoView);
+};
+
+ManagePasswordsBubbleView::SignInPromoView::SignInPromoView(
+    ManagePasswordsBubbleView* parent)
+    : parent_(parent) {
+  views::GridLayout* layout = new views::GridLayout(this);
+  layout->set_minimum_size(gfx::Size(kDesiredBubbleWidth, 0));
+  SetLayoutManager(layout);
+
+  // Reuse the string because it has the same meaning.

[vabr (Chromium), 2016/06/02 13:10:55]

Are you sure the meaning is the same? Here it means signing into Chrome, in the
account chooser it means signing into a page.

The verbs for that are the same in English, German, Czech and Slovak, at least,
so perhaps it is fine. But then we should drop the "ACCOUNT_CHOOSER" in the
string name and make sure the strings description does not mention which kind of
sign-in is meant.

Or we should stay on the safe side and have two strings for the two different
concepts.

But I'm a bit worried about the current mix. We had issues in the past with
people confusing content-area sign-in with Chrome sign-in, and it had scary
consequences (mixing sync accounts etc.).

[vasilii, 2016/06/02 13:36:18]

I split the strings.

+  signin_button_ = views::MdTextButton::CreateSecondaryUiBlueButton(
+      this,
+      l10n_util::GetStringUTF16(IDS_PASSWORD_MANAGER_ACCOUNT_CHOOSER_SIGN_IN));
+  no_button_ = views::MdTextButton::CreateSecondaryUiButton(
+      this,
+      l10n_util::GetStringUTF16(IDS_PASSWORD_MANAGER_NO_THANKS));
+
+  // Button row.
+  BuildColumnSet(layout, DOUBLE_BUTTON_COLUMN_SET);
+  layout->StartRow(0, DOUBLE_BUTTON_COLUMN_SET);
+  layout->AddView(signin_button_);
+  layout->AddView(no_button_);
+
+  parent_->set_initially_focused_view(signin_button_);
+}
+
+void ManagePasswordsBubbleView::SignInPromoView::ButtonPressed(
+    views::Button* sender,
+    const ui::Event& event) {
+  if (sender == signin_button_)
+    parent_->model()->OnSignInToChromeClicked();
+  else if (sender == no_button_)
+    parent_->model()->OnSkipSignInClicked();
+  else
+    NOTREACHED();
+
+  parent_->CloseBubble();
+}
+
 // ManagePasswordsBubbleView::WebContentMouseHandler --------------------------
 
 // The class listens for WebContentsView events and notifies the bubble if the
 // view was clicked on or received keystrokes.
 class ManagePasswordsBubbleView::WebContentMouseHandler
     : public ui::EventHandler {
  public:
   explicit WebContentMouseHandler(ManagePasswordsBubbleView* bubble);
 
   void OnKeyEvent(ui::KeyEvent* event) override;
@@ skipping 215 matching lines @@
 ManagePasswordsBubbleView::~ManagePasswordsBubbleView() {
   if (manage_passwords_bubble_ == this)
     manage_passwords_bubble_ = NULL;
 }
 
 views::View* ManagePasswordsBubbleView::GetInitiallyFocusedView() {
   return initially_focused_view_;
 }
 
 void ManagePasswordsBubbleView::Init() {
-  views::FillLayout* layout = new views::FillLayout();
-  SetLayoutManager(layout);
+  SetLayoutManager(new views::FillLayout);
 
-  Refresh();
+  CreateChild();
 }
 
 void ManagePasswordsBubbleView::CloseBubble() {
   mouse_handler_.reset();
   LocationBarBubbleDelegateView::CloseBubble();
 }
 
 base::string16 ManagePasswordsBubbleView::GetWindowTitle() const {
   return model_.title();
 }
 
 bool ManagePasswordsBubbleView::ShouldShowWindowTitle() const {
   // Since bubble titles don't support links, fall back to a custom title view
   // if we need to show a link. Only use the normal title path if there's no
   // link.
   return model_.title_brand_link_range().is_empty();
 }
 
 bool ManagePasswordsBubbleView::ShouldShowCloseButton() const {
   return model_.state() == password_manager::ui::PENDING_PASSWORD_STATE;
 }
 
 void ManagePasswordsBubbleView::Refresh() {
   RemoveAllChildViews(true);
   initially_focused_view_ = NULL;
+  CreateChild();
+
+  // Show/hide the close button.
+  GetWidget()->non_client_view()->ResetWindowControls();
+  GetWidget()->UpdateWindowTitle();
+  SizeToContents();
+}
+
+void ManagePasswordsBubbleView::CreateChild() {
   if (model_.state() == password_manager::ui::PENDING_PASSWORD_STATE) {
     AddChildView(new PendingView(this));
   } else if (model_.state() ==
-             password_manager::ui::PENDING_PASSWORD_UPDATE_STATE) {
+      password_manager::ui::PENDING_PASSWORD_UPDATE_STATE) {
     AddChildView(new UpdatePendingView(this));
   } else if (model_.state() == password_manager::ui::CONFIRMATION_STATE) {
     AddChildView(new SaveConfirmationView(this));
   } else if (model_.state() == password_manager::ui::AUTO_SIGNIN_STATE) {
     AddChildView(new AutoSigninView(this));
+  } else if (model_.state() ==
+      password_manager::ui::CHROME_SIGN_IN_PROMO_STATE) {
+    AddChildView(new SignInPromoView(this));
   } else {
     AddChildView(new ManageView(this));
   }
-  if (GetWidget())
-    GetWidget()->UpdateWindowTitle();
-  GetLayoutManager()->Layout(this);
 }