PageInfo bubble: Use the non-client view's window title and close button.

chrome/browser/ui/views/website_settings/website_settings_popup_view.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_WEBSITE_SETTINGS_WEBSITE_SETTINGS_POPUP_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_WEBSITE_SETTINGS_WEBSITE_SETTINGS_POPUP_VIEW_H_
 
 #include <memory>
 
 #include "base/compiler_specific.h"
@@ skipping 89 matching lines @@
 
   // PermissionSelectorRowObserver implementation.
   void OnPermissionChanged(
       const WebsiteSettingsUI::PermissionInfo& permission) override;
 
   // ChosenObjectRowObserver implementation.
   void OnChosenObjectDeleted(
       const WebsiteSettingsUI::ChosenObjectInfo& info) override;
 
   // views::BubbleDialogDelegateView implementation.
+  base::string16 GetWindowTitle() const override;
+  bool ShouldShowCloseButton() const override;
   void OnWidgetDestroying(views::Widget* widget) override;
   int GetDialogButtons() const override;
+  const gfx::FontList& GetTitleFontList() const override;
 
   // views::ButtonListener implementation.
   void ButtonPressed(views::Button* button, const ui::Event& event) override;
 
   // views::LinkListener implementation.
   void LinkClicked(views::Link* source, int event_flags) override;
 
   // views::StyledLabelListener implementation.
   void StyledLabelLinkClicked(views::StyledLabel* label,
                               const gfx::Range& range,
@@ skipping 23 matching lines @@
   bool is_devtools_disabled_;
 
   // The presenter that controls the Website Settings UI.
   std::unique_ptr<WebsiteSettings> presenter_;
 
   Profile* profile_;
 
   // The header section (containing security-related information).
   PopupHeaderView* header_;
 
+  // The label that displays security summary for the current page.

[lgarron, 2016/12/14 23:46:23]

Nit: Remove "label" from comment. (e.g. "The security summary for the current
page.")

[tapted, 2017/01/04 00:36:49]

Done.

+  base::string16 summary_text_;
+
   // The separator between the header and the site settings view.
   views::Separator* separator_;
 
   // The view that contains the cookie and permissions sections.
   views::View* site_settings_view_;
   // The view that contains the contents of the "Cookies" part of the site
   // settings view.
   views::View* cookies_view_;
   // The link that opens the "Cookies" dialog.
   views::Link* cookie_dialog_link_;
   // The view that contains the "Permissions" table of the site settings view.
   views::View* permissions_view_;
 
   // The certificate provided by the site, if one exists.
   scoped_refptr<net::X509Certificate> certificate_;
 
   base::WeakPtrFactory<WebsiteSettingsPopupView> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(WebsiteSettingsPopupView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_WEBSITE_SETTINGS_WEBSITE_SETTINGS_POPUP_VIEW_H_