Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(329)

Unified Diff: chrome/browser/ui/page_info/page_info.h

Issue 2754383004: Rename WebsiteSettings code to PageInfo. (Closed)
Patch Set: Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/page_info/page_info.h
diff --git a/chrome/browser/ui/page_info/website_settings.h b/chrome/browser/ui/page_info/page_info.h
similarity index 87%
rename from chrome/browser/ui/page_info/website_settings.h
rename to chrome/browser/ui/page_info/page_info.h
index 2f91e3fa8e9d64b86284bcc38a5902ddd3066c2f..bfd2f176250aefb9ad904231dbc06cb68417484f 100644
--- a/chrome/browser/ui/page_info/website_settings.h
+++ b/chrome/browser/ui/page_info/page_info.h
@@ -2,8 +2,8 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef CHROME_BROWSER_UI_PAGE_INFO_WEBSITE_SETTINGS_H_
-#define CHROME_BROWSER_UI_PAGE_INFO_WEBSITE_SETTINGS_H_
+#ifndef CHROME_BROWSER_UI_PAGE_INFO_PAGE_INFO_H_
+#define CHROME_BROWSER_UI_PAGE_INFO_PAGE_INFO_H_
#include "base/macros.h"
#include "base/strings/string16.h"
@@ -26,15 +26,15 @@ class ChromeSSLHostStateDelegate;
class ChooserContextBase;
class HostContentSettingsMap;
class Profile;
-class WebsiteSettingsUI;
+class PageInfoUI;
-// The |WebsiteSettings| provides information about a website's permissions,
+// The |PageInfo| provides information about a website's permissions,
// connection state and its identity. It owns a UI that displays the
-// information and allows users to change the permissions. |WebsiteSettings|
+// information and allows users to change the permissions. |PageInfo|
// objects must be created on the heap. They destroy themselves after the UI is
// closed.
-class WebsiteSettings : public TabSpecificContentSettings::SiteDataObserver,
- public content::WebContentsObserver {
+class PageInfo : public TabSpecificContentSettings::SiteDataObserver,
+ public content::WebContentsObserver {
public:
// TODO(palmer): Figure out if it is possible to unify SiteConnectionStatus
// and SiteIdentityStatus.
@@ -83,10 +83,10 @@ class WebsiteSettings : public TabSpecificContentSettings::SiteDataObserver,
SITE_IDENTITY_STATUS_UNWANTED_SOFTWARE,
};
- // UMA statistics for WebsiteSettings. Do not reorder or remove existing
+ // UMA statistics for PageInfo. Do not reorder or remove existing
// fields. A Java counterpart will be generated for this enum.
// GENERATED_JAVA_ENUM_PACKAGE: org.chromium.chrome.browser.pageinfo
- enum WebsiteSettingsAction {
+ enum PageInfoAction {
WEBSITE_SETTINGS_OPENED = 0,
// No longer used; indicated actions for the old version of Page Info that
// had a "Permissions" tab and a "Connection" tab.
@@ -114,18 +114,18 @@ class WebsiteSettings : public TabSpecificContentSettings::SiteDataObserver,
const char* ui_name_key;
};
- // Creates a WebsiteSettings for the passed |url| using the given |ssl| status
+ // Creates a PageInfo for the passed |url| using the given |ssl| status
// object to determine the status of the site's connection. The
- // |WebsiteSettings| takes ownership of the |ui|.
- WebsiteSettings(WebsiteSettingsUI* ui,
- Profile* profile,
- TabSpecificContentSettings* tab_specific_content_settings,
- content::WebContents* web_contents,
- const GURL& url,
- const security_state::SecurityInfo& security_info);
- ~WebsiteSettings() override;
+ // |PageInfo| takes ownership of the |ui|.
+ PageInfo(PageInfoUI* ui,
+ Profile* profile,
+ TabSpecificContentSettings* tab_specific_content_settings,
+ content::WebContents* web_contents,
+ const GURL& url,
+ const security_state::SecurityInfo& security_info);
+ ~PageInfo() override;
- void RecordWebsiteSettingsAction(WebsiteSettingsAction action);
+ void RecordPageInfoAction(PageInfoAction action);
// This method is called when ever a permission setting is changed.
void OnSitePermissionChanged(ContentSettingsType type, ContentSetting value);
@@ -157,7 +157,7 @@ class WebsiteSettings : public TabSpecificContentSettings::SiteDataObserver,
void OnSiteDataAccessed() override;
private:
- // Initializes the |WebsiteSettings|.
+ // Initializes the |PageInfo|.
void Init(const GURL& url, const security_state::SecurityInfo& security_info);
// Sets (presents) the information about the site's permissions in the |ui_|.
@@ -174,7 +174,7 @@ class WebsiteSettings : public TabSpecificContentSettings::SiteDataObserver,
// specific data (local stored objects like cookies), site-specific
// permissions (location, pop-up, plugin, etc. permissions) and site-specific
// information (identity, connection status, etc.).
- WebsiteSettingsUI* ui_;
+ PageInfoUI* ui_;
// The flag that controls whether an infobar is displayed after the website
// settings UI is closed or not.
@@ -235,7 +235,7 @@ class WebsiteSettings : public TabSpecificContentSettings::SiteDataObserver,
security_state::SecurityLevel security_level_;
- DISALLOW_COPY_AND_ASSIGN(WebsiteSettings);
+ DISALLOW_COPY_AND_ASSIGN(PageInfo);
};
-#endif // CHROME_BROWSER_UI_PAGE_INFO_WEBSITE_SETTINGS_H_
+#endif // CHROME_BROWSER_UI_PAGE_INFO_PAGE_INFO_H_

Powered by Google App Engine
This is Rietveld 408576698