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

Unified Diff: chrome/browser/ui/gtk/extensions/media_galleries_dialog_gtk_unittest.cc

Issue 12095074: Media Galleries: Keep media gallery permission dialogs in sync with the gallery (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: fix nit Created 7 years, 10 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/gtk/extensions/media_galleries_dialog_gtk_unittest.cc
===================================================================
--- chrome/browser/ui/gtk/extensions/media_galleries_dialog_gtk_unittest.cc (revision 181443)
+++ chrome/browser/ui/gtk/extensions/media_galleries_dialog_gtk_unittest.cc (working copy)
@@ -111,4 +111,27 @@
EXPECT_EQ(2U, dialog.checkbox_map_.size());
}
+TEST_F(MediaGalleriesDialogTest, ForgetDeletes) {
+ NiceMock<MediaGalleriesDialogControllerMock> controller;
+
+ MediaGalleriesDialogController::KnownGalleryPermissions permissions;
+ EXPECT_CALL(controller, permissions()).
+ WillRepeatedly(ReturnRef(permissions));
+
+ MediaGalleriesDialogGtk dialog(&controller);
+
+ EXPECT_TRUE(dialog.checkbox_map_.empty());
+
+ MediaGalleryPrefInfo gallery1 = MakePrefInfoForTesting(1);
+ dialog.UpdateGallery(&gallery1, true);
+ EXPECT_EQ(1U, dialog.checkbox_map_.size());
+
+ MediaGalleryPrefInfo gallery2 = MakePrefInfoForTesting(2);
+ dialog.UpdateGallery(&gallery2, true);
+ EXPECT_EQ(2U, dialog.checkbox_map_.size());
+
+ dialog.ForgetGallery(&gallery2);
+ EXPECT_EQ(1U, dialog.checkbox_map_.size());
+}
+
} // namespace chrome

Powered by Google App Engine
This is Rietveld 408576698