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

Unified Diff: chrome/browser/ui/views/extensions/media_galleries_dialog_views.h

Issue 310383004: Make a controller interface for media galleries dialogs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 6 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/extensions/media_galleries_dialog_views.h
diff --git a/chrome/browser/ui/views/extensions/media_galleries_dialog_views.h b/chrome/browser/ui/views/extensions/media_galleries_dialog_views.h
index 535adb3a01583463bc85e1c96cb53447ef1a2324..65a9c2a5c23e031fedbce75deff27267a7dcec14 100644
--- a/chrome/browser/ui/views/extensions/media_galleries_dialog_views.h
+++ b/chrome/browser/ui/views/extensions/media_galleries_dialog_views.h
@@ -65,20 +65,20 @@ class MediaGalleriesDialogViews : public MediaGalleriesDialog,
FRIEND_TEST_ALL_PREFIXES(MediaGalleriesDialogTest, UpdateAdds);
FRIEND_TEST_ALL_PREFIXES(MediaGalleriesDialogTest, ForgetDeletes);
- typedef std::map<GalleryDialogId, MediaGalleryCheckboxView*> CheckboxMap;
+ typedef std::map<MediaGalleryPrefId, MediaGalleryCheckboxView*> CheckboxMap;
void InitChildViews();
// Adds a checkbox or updates an existing checkbox. Returns true if a new one
// was added.
bool AddOrUpdateGallery(
- const MediaGalleriesDialogController::GalleryPermission& gallery,
+ const MediaGalleriesDialogController::Entry& gallery,
views::View* container,
int trailing_vertical_space);
void ShowContextMenu(const gfx::Point& point,
ui::MenuSourceType source_type,
- GalleryDialogId id);
+ MediaGalleryPrefId id);
// Whether |controller_| has a valid WebContents or not.
// In unit tests, it may not.
@@ -92,9 +92,9 @@ class MediaGalleriesDialogViews : public MediaGalleriesDialog,
// A map from gallery ID to views::Checkbox view.
CheckboxMap checkbox_map_;
- // Pointer to the button to add a new gallery. Owned by parent in
- // the dialog views tree.
- views::LabelButton* add_gallery_button_;
+ // Pointer to the controller specific auxiliary button, NULL otherwise.
+ // Owned by parent in the dialog views tree.
+ views::LabelButton* auxiliary_button_;
// This tracks whether the confirm button can be clicked. It starts as false
// if no checkboxes are ticked. After there is any interaction, or some

Powered by Google App Engine
This is Rietveld 408576698