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

Unified Diff: chrome/browser/ui/content_settings/content_setting_bubble_model_unittest.cc

Issue 7484072: Migrate geolocation settings to host content settings map. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: " Created 9 years, 4 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/content_settings/content_setting_bubble_model_unittest.cc
diff --git a/chrome/browser/ui/content_settings/content_setting_bubble_model_unittest.cc b/chrome/browser/ui/content_settings/content_setting_bubble_model_unittest.cc
index 99ee7db2323c4f318d536e89f1df7a4e16cba224..ef8faf7fd7cb849d0e0ea63bbfcd8119f2c901c6 100644
--- a/chrome/browser/ui/content_settings/content_setting_bubble_model_unittest.cc
+++ b/chrome/browser/ui/content_settings/content_setting_bubble_model_unittest.cc
@@ -7,7 +7,6 @@
#include "chrome/browser/content_settings/host_content_settings_map.h"
#include "chrome/browser/content_settings/tab_specific_content_settings.h"
#include "chrome/browser/profiles/profile.h"
-#include "chrome/browser/geolocation/geolocation_content_settings_map.h"
#include "chrome/browser/ui/content_settings/content_setting_bubble_model.h"
#include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"
#include "chrome/browser/ui/tab_contents/test_tab_contents_wrapper.h"
@@ -184,9 +183,14 @@ TEST_F(ContentSettingBubbleModelTest, Geolocation) {
CheckGeolocationBubble(1, false, true);
// Add it to the content map, should now have a clear link.
- GeolocationContentSettingsMap* setting_map =
- profile_->GetGeolocationContentSettingsMap();
- setting_map->SetContentSetting(frame1_url, page_url, CONTENT_SETTING_ALLOW);
+ HostContentSettingsMap* setting_map =
+ profile_->GetHostContentSettingsMap();
+ setting_map->SetContentSetting(
+ ContentSettingsPattern::FromURLNoWildcard(frame1_url),
+ ContentSettingsPattern::FromURLNoWildcard(page_url),
+ CONTENT_SETTINGS_TYPE_GEOLOCATION,
+ std::string(),
+ CONTENT_SETTING_ALLOW);
CheckGeolocationBubble(1, true, false);
// Change the default to allow: no message needed.

Powered by Google App Engine
This is Rietveld 408576698