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

Side by Side Diff: chrome/browser/managed_mode_unittest.cc

Issue 10680013: Remove unused fields found by clang's new Wunused-private-fields. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #include "chrome/browser/managed_mode.h" 5 #include "chrome/browser/managed_mode.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after
69 } 69 }
70 70
71 MOCK_METHOD0(Close, void()); 71 MOCK_METHOD0(Close, void());
72 }; 72 };
73 73
74 class BrowserFixture { 74 class BrowserFixture {
75 public: 75 public:
76 BrowserFixture(FakeManagedMode* managed_mode, 76 BrowserFixture(FakeManagedMode* managed_mode,
77 TestingProfile* profile) 77 TestingProfile* profile)
78 : browser_(Browser::TYPE_TABBED, profile), 78 : browser_(Browser::TYPE_TABBED, profile),
79 window_(&browser_), 79 window_(&browser_) {
80 managed_mode_(managed_mode) {
81 browser_.SetWindowForTesting(&window_); 80 browser_.SetWindowForTesting(&window_);
82 } 81 }
83 82
84 ~BrowserFixture() { 83 ~BrowserFixture() {
85 } 84 }
86 85
87 MockBrowserWindow* window() { 86 MockBrowserWindow* window() {
88 return &window_; 87 return &window_;
89 } 88 }
90 89
91 private: 90 private:
92 Browser browser_; 91 Browser browser_;
93 StrictMock<MockBrowserWindow> window_; 92 StrictMock<MockBrowserWindow> window_;
94 FakeManagedMode* managed_mode_;
95 }; 93 };
96 94
97 class MockCallback : public base::RefCountedThreadSafe<MockCallback> { 95 class MockCallback : public base::RefCountedThreadSafe<MockCallback> {
98 public: 96 public:
99 explicit MockCallback(FakeManagedMode* managed_mode) 97 explicit MockCallback(FakeManagedMode* managed_mode)
100 : managed_mode_(managed_mode) { 98 : managed_mode_(managed_mode) {
101 } 99 }
102 100
103 void CheckManagedMode(bool success) { 101 void CheckManagedMode(bool success) {
104 EXPECT_EQ(success, managed_mode_->IsInManagedModeImpl()); 102 EXPECT_EQ(success, managed_mode_->IsInManagedModeImpl());
(...skipping 137 matching lines...) Expand 10 before | Expand all | Expand 10 after
242 EXPECT_TRUE(managed_mode_.UserMayModifySettings(NULL, NULL)); 240 EXPECT_TRUE(managed_mode_.UserMayModifySettings(NULL, NULL));
243 241
244 managed_mode_.SetInManagedMode(&managed_mode_profile_); 242 managed_mode_.SetInManagedMode(&managed_mode_profile_);
245 EXPECT_FALSE(managed_mode_.UserMayLoad(NULL, NULL)); 243 EXPECT_FALSE(managed_mode_.UserMayLoad(NULL, NULL));
246 EXPECT_FALSE(managed_mode_.UserMayModifySettings(NULL, NULL)); 244 EXPECT_FALSE(managed_mode_.UserMayModifySettings(NULL, NULL));
247 245
248 #ifndef NDEBUG 246 #ifndef NDEBUG
249 EXPECT_FALSE(managed_mode_.GetDebugPolicyProviderName().empty()); 247 EXPECT_FALSE(managed_mode_.GetDebugPolicyProviderName().empty());
250 #endif 248 #endif
251 } 249 }
OLDNEW
« no previous file with comments | « chrome/browser/locale_tests_browsertest.cc ('k') | chrome/browser/mock_browsing_data_database_helper.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698