[go: nahoru, domu]

Password bubble: Add the arrow back. It's more understandable.

UI mocks will have an arrow again at some point in the near future. This
patch aligns the implementation with the future mocks by reverting
r275111[1], which removed the arrow.

[1]: https://codereview.chromium.org/315153002

BUG=261628

Review URL: https://codereview.chromium.org/410423004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285284 0039d316-1c4b-4281-b951-d872f2087c98
diff --git a/chrome/browser/ui/views/passwords/manage_passwords_bubble_view.cc b/chrome/browser/ui/views/passwords/manage_passwords_bubble_view.cc
index fa0a563..8a4a2f2 100644
--- a/chrome/browser/ui/views/passwords/manage_passwords_bubble_view.cc
+++ b/chrome/browser/ui/views/passwords/manage_passwords_bubble_view.cc
@@ -539,7 +539,6 @@
         browser_view->GetBoundsInScreen());
   }
   manage_passwords_bubble_->GetWidget()->Show();
-  manage_passwords_bubble_->SetArrowPaintType(views::BubbleBorder::PAINT_NONE);
   // set_use_focusless(true) has adverse effect on Windows.
   // 1) The bubble can't be the active window (which is not desired behavior).
   // 2) The bubble doesn't get WM_LBUTTONDOWN event. Therefore, all the buttons
@@ -572,7 +571,7 @@
       anchor_view_(anchor_view),
       never_save_passwords_(false) {
   // Compensate for built-in vertical padding in the anchor view's image.
-  set_anchor_view_insets(gfx::Insets(2, 0, 2, 0));
+  set_anchor_view_insets(gfx::Insets(5, 0, 5, 0));
   // Don't focus by default.
   set_use_focusless(true);
   if (anchor_view)