Quota: Add close button to storage pressure notification.

This change also makes it so that changing focus does not close the
BubbleView.

Bug: 1062268
Change-Id: Iddcabad42f7b31d1535c073ba79bc68f4ad22da4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2106458Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Jarryd Goodman <jarrydg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#751834}

chrome/browser/ui/views/storage/storage_pressure_bubble_view.cc

@@ -66,6 +66,7 @@ StoragePressureBubbleView::StoragePressureBubbleView(
           IDS_SETTINGS_STORAGE_PRESSURE_BUBBLE_VIEW_BUTTON_LABEL));
   DialogDelegate::SetAcceptCallback(base::BindOnce(
       &StoragePressureBubbleView::OnDialogAccepted, base::Unretained(this)));
+  set_close_on_deactivate(false);
 }
 
 base::string16 StoragePressureBubbleView::GetWindowTitle() const {
@@ -108,3 +109,7 @@ void StoragePressureBubbleView::Init() {
       margins().width());
   AddChildView(std::move(text_label));
 }
+
+bool StoragePressureBubbleView::ShouldShowCloseButton() const {
+  return true;
+}

chrome/browser/ui/views/storage/storage_pressure_bubble_view.h

@@ -27,6 +27,7 @@ class StoragePressureBubbleView : public views::BubbleDialogDelegateView {
 
   // views::BubbleDialogDelegateView:
   void Init() override;
+  bool ShouldShowCloseButton() const override;
 
   Browser* const browser_;
   const GURL all_sites_url_ = GURL("chrome://settings/content/all");