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

Unified Diff: chrome/browser/ui/website_settings/website_settings_unittest.cc

Issue 2702923002: Page Info: Hide default permissions with a value of Ask if the default is Ask. (Closed)
Patch Set: Fix existing tests. Created 3 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/website_settings/website_settings_unittest.cc
diff --git a/chrome/browser/ui/website_settings/website_settings_unittest.cc b/chrome/browser/ui/website_settings/website_settings_unittest.cc
index 89119f6ab1ee626ea8ed53aa32ac583b0ee4154d..925b13b800d844f1b8b177a9da31ac5f2e68bd4a 100644
--- a/chrome/browser/ui/website_settings/website_settings_unittest.cc
+++ b/chrome/browser/ui/website_settings/website_settings_unittest.cc
@@ -73,6 +73,8 @@ class MockWebsiteSettingsUI : public WebsiteSettingsUI {
virtual ~MockWebsiteSettingsUI() {}
MOCK_METHOD1(SetCookieInfo, void(const CookieInfoList& cookie_info_list));
MOCK_METHOD0(SetPermissionInfoStub, void());
+ MOCK_METHOD1(UpdatePermissionButton,
+ void(VisiblePermissions visible_permissions));
MOCK_METHOD1(SetIdentityInfo, void(const IdentityInfo& identity_info));
void SetPermissionInfo(
@@ -130,6 +132,7 @@ class WebsiteSettingsTest : public ChromeRenderViewHostTestHarness {
EXPECT_CALL(*mock_ui, SetPermissionInfoStub());
EXPECT_CALL(*mock_ui, SetIdentityInfo(_));
EXPECT_CALL(*mock_ui, SetCookieInfo(_));
+ EXPECT_CALL(*mock_ui, UpdatePermissionButton(_));
}
void SetURL(const std::string& url) { url_ = GURL(url); }
@@ -631,6 +634,7 @@ TEST_F(WebsiteSettingsTest, ShowInfoBar) {
EXPECT_CALL(*mock_ui(), SetCookieInfo(_));
EXPECT_CALL(*mock_ui(), SetPermissionInfoStub()).Times(2);
+ EXPECT_CALL(*mock_ui(), UpdatePermissionButton(_)).Times(2);
EXPECT_EQ(0u, infobar_service()->infobar_count());
website_settings()->OnSitePermissionChanged(

Powered by Google App Engine
This is Rietveld 408576698