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

Side by Side Diff: chrome/browser/cocoa/content_exceptions_window_controller_unittest.mm

Issue 2963006: Remove the cookie prompt from the settings UI and migrate prefs from ask to block. (Closed)
Patch Set: updates Created 10 years, 5 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #import "chrome/browser/cocoa/content_exceptions_window_controller.h" 5 #import "chrome/browser/cocoa/content_exceptions_window_controller.h"
6 6
7 #import <Cocoa/Cocoa.h> 7 #import <Cocoa/Cocoa.h>
8 8
9 #import "base/scoped_nsobject.h" 9 #import "base/scoped_nsobject.h"
10 #include "base/ref_counted.h" 10 #include "base/ref_counted.h"
(...skipping 202 matching lines...) Expand 10 before | Expand all | Expand 10 after
213 213
214 HostContentSettingsMap::SettingsForOneType settings; 214 HostContentSettingsMap::SettingsForOneType settings;
215 settingsMap_->GetSettingsForOneType(CONTENT_SETTINGS_TYPE_PLUGINS, 215 settingsMap_->GetSettingsForOneType(CONTENT_SETTINGS_TYPE_PLUGINS,
216 &settings); 216 &settings);
217 EXPECT_EQ(1u, settings.size()); 217 EXPECT_EQ(1u, settings.size());
218 } 218 }
219 219
220 TEST_F(ContentExceptionsWindowControllerTest, AddExistingDoesNotOverwrite) { 220 TEST_F(ContentExceptionsWindowControllerTest, AddExistingDoesNotOverwrite) {
221 settingsMap_->SetContentSetting(HostContentSettingsMap::Pattern("myhost"), 221 settingsMap_->SetContentSetting(HostContentSettingsMap::Pattern("myhost"),
222 CONTENT_SETTINGS_TYPE_COOKIES, 222 CONTENT_SETTINGS_TYPE_COOKIES,
223 CONTENT_SETTING_ASK); 223 CONTENT_SETTING_SESSION_ONLY);
224 224
225 ContentExceptionsWindowController* controller = 225 ContentExceptionsWindowController* controller =
226 GetController(CONTENT_SETTINGS_TYPE_COOKIES); 226 GetController(CONTENT_SETTINGS_TYPE_COOKIES);
227 227
228 ClickAdd(controller); 228 ClickAdd(controller);
229 EnterText(@"myhost\n"); 229 EnterText(@"myhost\n");
230 230
231 EXPECT_FALSE([controller editingNewException]); 231 EXPECT_FALSE([controller editingNewException]);
232 [controller close]; 232 [controller close];
233 233
234 HostContentSettingsMap::SettingsForOneType settings; 234 HostContentSettingsMap::SettingsForOneType settings;
235 settingsMap_->GetSettingsForOneType(CONTENT_SETTINGS_TYPE_COOKIES, 235 settingsMap_->GetSettingsForOneType(CONTENT_SETTINGS_TYPE_COOKIES,
236 &settings); 236 &settings);
237 EXPECT_EQ(1u, settings.size()); 237 EXPECT_EQ(1u, settings.size());
238 EXPECT_EQ(CONTENT_SETTING_ASK, settings[0].second); 238 EXPECT_EQ(CONTENT_SETTING_SESSION_ONLY, settings[0].second);
239 } 239 }
240 240
241 241
242 } // namespace 242 } // namespace
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698