Material Page Info (Views, 3/3): Update site settings section.

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

Index: chrome/browser/ui/views/website_settings/website_settings_popup_view.h
diff --git a/chrome/browser/ui/views/website_settings/website_settings_popup_view.h b/chrome/browser/ui/views/website_settings/website_settings_popup_view.h
index 6d86ca89f383873999fd77c0d1687c0676c234d4..16df45b2d47010dde02455d0410a629253b3100e 100644
--- a/chrome/browser/ui/views/website_settings/website_settings_popup_view.h
+++ b/chrome/browser/ui/views/website_settings/website_settings_popup_view.h
@@ -130,14 +130,6 @@ class WebsiteSettingsPopupView : public content::WebContentsObserver,
   // returned view is transferred to the caller.
   views::View* CreateSiteSettingsView() WARN_UNUSED_RESULT;
 
-  // The site settings view contains several sections with a |headline|
-  // followed by the section |contents| and an optional |link|. This method
-  // creates a section for the given |headline|, |contents| and |link|. |link|
-  // can be NULL if the section should not contain a link.
-  views::View* CreateSection(const base::string16& headline,
-                             views::View* contents,
-                             views::Link* link) WARN_UNUSED_RESULT;
-
   // Used to asynchronously handle clicks since these calls may cause the
   // destruction of the settings view and the base class window still needs to
   // be alive to finish handling the mouse or keyboard click.
@@ -155,23 +147,22 @@ class WebsiteSettingsPopupView : public content::WebContentsObserver,
   // The separator between the header and the site settings view.
   views::Separator* separator_;
 
-  // The view that contains the site data and permissions sections.
+  // The view that contains the cookie and permissions sections.
   views::View* site_settings_view_;
-  // The view that contains the contents of the "Cookies and Site data" section
-  // of the site settings view.
-  views::View* site_data_content_;
+  // 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 contents of the "Permissions" section
-  // of the site settings view.
-  views::View* permissions_content_;
+  // 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_;
 
-  // The link to open the site settings page that provides full control over
+  // The button to open the site settings page that provides full control over
   // the origin's permissions.
-  views::Link* site_settings_link_;
+  views::Link* site_settings_button_;

[msw, 2016/09/26 20:47:56]

nit: avoid 'button' naming for views::Link instances.

[lgarron, 2016/09/28 21:11:47]

Even when the link is styled like a button?

[msw, 2016/09/30 00:37:04]

If it were actually a views::Link*, then yes.

[lgarron, 2016/09/30 05:05:55]

Acknowledged (although the var has now been removed.)

 
   base::WeakPtrFactory<WebsiteSettingsPopupView> weak_factory_;