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

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

Issue 2858032: Display content settings applying to the current otr session only. (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 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
50 public: 50 public:
51 virtual void SetUp() { 51 virtual void SetUp() {
52 CocoaTest::SetUp(); 52 CocoaTest::SetUp();
53 TestingProfile* profile = browser_helper_.profile(); 53 TestingProfile* profile = browser_helper_.profile();
54 settingsMap_ = new HostContentSettingsMap(profile); 54 settingsMap_ = new HostContentSettingsMap(profile);
55 } 55 }
56 56
57 ContentExceptionsWindowController* GetController(ContentSettingsType type) { 57 ContentExceptionsWindowController* GetController(ContentSettingsType type) {
58 id controller = [ContentExceptionsWindowController 58 id controller = [ContentExceptionsWindowController
59 controllerForType:type 59 controllerForType:type
60 settingsMap:settingsMap_.get()]; 60 settingsMap:settingsMap_.get()
61 otrSettingsMap:NULL];
61 [controller showWindow:nil]; 62 [controller showWindow:nil];
62 return controller; 63 return controller;
63 } 64 }
64 65
65 void ClickAdd(ContentExceptionsWindowController* controller) { 66 void ClickAdd(ContentExceptionsWindowController* controller) {
66 [controller addException:nil]; 67 [controller addException:nil];
67 ProcessEvents(); 68 ProcessEvents();
68 } 69 }
69 70
70 void ClickRemove(ContentExceptionsWindowController* controller) { 71 void ClickRemove(ContentExceptionsWindowController* controller) {
(...skipping 18 matching lines...) Expand all
89 90
90 protected: 91 protected:
91 BrowserTestHelper browser_helper_; 92 BrowserTestHelper browser_helper_;
92 scoped_refptr<HostContentSettingsMap> settingsMap_; 93 scoped_refptr<HostContentSettingsMap> settingsMap_;
93 }; 94 };
94 95
95 TEST_F(ContentExceptionsWindowControllerTest, Construction) { 96 TEST_F(ContentExceptionsWindowControllerTest, Construction) {
96 ContentExceptionsWindowController* controller = 97 ContentExceptionsWindowController* controller =
97 [ContentExceptionsWindowController 98 [ContentExceptionsWindowController
98 controllerForType:CONTENT_SETTINGS_TYPE_PLUGINS 99 controllerForType:CONTENT_SETTINGS_TYPE_PLUGINS
99 settingsMap:settingsMap_.get()]; 100 settingsMap:settingsMap_.get()
101 otrSettingsMap:NULL];
100 [controller showWindow:nil]; 102 [controller showWindow:nil];
101 [controller close]; // Should autorelease. 103 [controller close]; // Should autorelease.
102 } 104 }
103 105
104 // Regression test for http://crbug.com/37137 106 // Regression test for http://crbug.com/37137
105 TEST_F(ContentExceptionsWindowControllerTest, AddRemove) { 107 TEST_F(ContentExceptionsWindowControllerTest, AddRemove) {
106 ContentExceptionsWindowController* controller = 108 ContentExceptionsWindowController* controller =
107 GetController(CONTENT_SETTINGS_TYPE_PLUGINS); 109 GetController(CONTENT_SETTINGS_TYPE_PLUGINS);
108 110
109 HostContentSettingsMap::SettingsForOneType settings; 111 HostContentSettingsMap::SettingsForOneType settings;
(...skipping 121 matching lines...) Expand 10 before | Expand all | Expand 10 after
231 233
232 HostContentSettingsMap::SettingsForOneType settings; 234 HostContentSettingsMap::SettingsForOneType settings;
233 settingsMap_->GetSettingsForOneType(CONTENT_SETTINGS_TYPE_COOKIES, 235 settingsMap_->GetSettingsForOneType(CONTENT_SETTINGS_TYPE_COOKIES,
234 &settings); 236 &settings);
235 EXPECT_EQ(1u, settings.size()); 237 EXPECT_EQ(1u, settings.size());
236 EXPECT_EQ(CONTENT_SETTING_ASK, settings[0].second); 238 EXPECT_EQ(CONTENT_SETTING_ASK, settings[0].second);
237 } 239 }
238 240
239 241
240 } // namespace 242 } // namespace
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698