Material Page Info (Views, 1/3): Rename {ChosenObject, PermissionSelector}View classes to {...}Row.

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

Index: chrome/browser/ui/views/website_settings/permission_selector_row.h
diff --git a/chrome/browser/ui/views/website_settings/permission_selector_view.h b/chrome/browser/ui/views/website_settings/permission_selector_row.h
similarity index 79%
rename from chrome/browser/ui/views/website_settings/permission_selector_view.h
rename to chrome/browser/ui/views/website_settings/permission_selector_row.h
index ec41b7a108414fba3413b52fae211163c458203a..4c3711f0062dda97ee1b65e6b43fcce5bc78e7ad 100644
--- a/chrome/browser/ui/views/website_settings/permission_selector_view.h
+++ b/chrome/browser/ui/views/website_settings/permission_selector_row.h
@@ -11,7 +11,7 @@
 #include "base/macros.h"
 #include "base/observer_list.h"
 #include "base/strings/string16.h"
-#include "chrome/browser/ui/views/website_settings/permission_selector_view_observer.h"
+#include "chrome/browser/ui/views/website_settings/permission_selector_row_observer.h"
 #include "chrome/browser/ui/website_settings/permission_menu_model.h"
 #include "chrome/browser/ui/website_settings/website_settings_ui.h"
 #include "components/content_settings/core/common/content_settings.h"
@@ -31,14 +31,15 @@ class ImageView;
 class MenuRunner;
 }
 
-// A custom view for selecting a permission setting for the given permission
-// |type|.
-class PermissionSelectorView : public views::View {
+// A PermissionSelectorRow is a row in the Page Info bubble that shows a
+// permission that a site can have ambient access to, and allows the user to
+// control whether tha access is granted.

[msw, 2016/09/01 01:42:08]

nit: s/tha/that/

[lgarron, 2016/09/02 02:56:49]

Done.

+class PermissionSelectorRow : public views::View {
  public:
-  PermissionSelectorView(const GURL& url,
+  PermissionSelectorRow(const GURL& url,
                          const WebsiteSettingsUI::PermissionInfo& permission);

[msw, 2016/09/01 01:42:08]

nit: git cl format

[lgarron, 2016/09/02 02:56:49]

Done.

 
-  void AddObserver(PermissionSelectorViewObserver* observer);
+  void AddObserver(PermissionSelectorRowObserver* observer);
 
   void PermissionChanged(const WebsiteSettingsUI::PermissionInfo& permission);
 
@@ -47,7 +48,7 @@ class PermissionSelectorView : public views::View {
   void ChildPreferredSizeChanged(View* child) override;
 
  private:
-  ~PermissionSelectorView() override;
+  ~PermissionSelectorRow() override;
 
   void InitializeMenuButtonView(
       views::GridLayout* layout,
@@ -64,9 +65,9 @@ class PermissionSelectorView : public views::View {
   internal::PermissionMenuButton* menu_button_;  // Owned by the views hierachy.
   internal::PermissionCombobox* combobox_;  // Owned by the views hierarchy.
 
-  base::ObserverList<PermissionSelectorViewObserver, false> observer_list_;
+  base::ObserverList<PermissionSelectorRowObserver, false> observer_list_;
 
-  DISALLOW_COPY_AND_ASSIGN(PermissionSelectorView);
+  DISALLOW_COPY_AND_ASSIGN(PermissionSelectorRow);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_WEBSITE_SETTINGS_PERMISSION_SELECTOR_VIEW_H_