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

Unified Diff: chrome/browser/ui/views/website_settings/permission_selector_view.h

Issue 2272793007: Material Page Info (Views, 1/3): Rename {ChosenObject, PermissionSelector}View classes to {...}Row. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address comments. Created 4 years, 3 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/views/website_settings/permission_selector_view.h
diff --git a/chrome/browser/ui/views/website_settings/permission_selector_view.h b/chrome/browser/ui/views/website_settings/permission_selector_view.h
deleted file mode 100644
index ec41b7a108414fba3413b52fae211163c458203a..0000000000000000000000000000000000000000
--- a/chrome/browser/ui/views/website_settings/permission_selector_view.h
+++ /dev/null
@@ -1,72 +0,0 @@
-// 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_PERMISSION_SELECTOR_VIEW_H_
-#define CHROME_BROWSER_UI_VIEWS_WEBSITE_SETTINGS_PERMISSION_SELECTOR_VIEW_H_
-
-#include <memory>
-
-#include "base/compiler_specific.h"
-#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/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"
-#include "components/content_settings/core/common/content_settings_types.h"
-#include "ui/views/controls/button/menu_button_listener.h"
-#include "ui/views/view.h"
-
-namespace internal {
-class ComboboxModelAdapter;
-class PermissionCombobox;
-class PermissionMenuButton;
-}
-
-namespace views {
-class GridLayout;
-class ImageView;
-class MenuRunner;
-}
-
-// A custom view for selecting a permission setting for the given permission
-// |type|.
-class PermissionSelectorView : public views::View {
- public:
- PermissionSelectorView(const GURL& url,
- const WebsiteSettingsUI::PermissionInfo& permission);
-
- void AddObserver(PermissionSelectorViewObserver* observer);
-
- void PermissionChanged(const WebsiteSettingsUI::PermissionInfo& permission);
-
- protected:
- // Overridden from views::View.
- void ChildPreferredSizeChanged(View* child) override;
-
- private:
- ~PermissionSelectorView() override;
-
- void InitializeMenuButtonView(
- views::GridLayout* layout,
- const WebsiteSettingsUI::PermissionInfo& permission);
- void InitializeComboboxView(
- views::GridLayout* layout,
- const WebsiteSettingsUI::PermissionInfo& permission);
-
- // Model for the permission's menu.
- std::unique_ptr<PermissionMenuModel> menu_model_;
- std::unique_ptr<internal::ComboboxModelAdapter> combobox_model_adapter_;
-
- views::ImageView* icon_; // Owned by the views hierachy.
- internal::PermissionMenuButton* menu_button_; // Owned by the views hierachy.
- internal::PermissionCombobox* combobox_; // Owned by the views hierarchy.
-
- base::ObserverList<PermissionSelectorViewObserver, false> observer_list_;
-
- DISALLOW_COPY_AND_ASSIGN(PermissionSelectorView);
-};
-
-#endif // CHROME_BROWSER_UI_VIEWS_WEBSITE_SETTINGS_PERMISSION_SELECTOR_VIEW_H_

Powered by Google App Engine
This is Rietveld 408576698