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

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

Issue 9479008: Re-factor location bar/toolbar code to get rid of the browser dependency. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: call right overloaded method Created 8 years, 9 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
« no previous file with comments | « chrome/browser/command_updater.cc ('k') | chrome/browser/sync/sync_global_error_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2006-2008 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 "base/compiler_specific.h"
5 #include "chrome/browser/command_updater.h" 6 #include "chrome/browser/command_updater.h"
6 #include "testing/gtest/include/gtest/gtest.h" 7 #include "testing/gtest/include/gtest/gtest.h"
7 8
8 class TestingCommandHandlerMock 9 class TestingCommandHandlerMock
9 : public CommandUpdater::CommandUpdaterDelegate { 10 : public CommandUpdater::CommandUpdaterDelegate {
10 public: 11 public:
11 virtual void ExecuteCommand(int id) { 12 virtual void ExecuteCommandWithDisposition(int id,
13 WindowOpenDisposition) OVERRIDE {
12 EXPECT_EQ(1, id); 14 EXPECT_EQ(1, id);
13 } 15 }
14 }; 16 };
15 17
16 class CommandUpdaterTest : public testing::Test { 18 class CommandUpdaterTest : public testing::Test {
17 }; 19 };
18 20
19 class TestingCommandObserverMock : public CommandUpdater::CommandObserver { 21 class TestingCommandObserverMock : public CommandUpdater::CommandObserver {
20 public: 22 public:
21 TestingCommandObserverMock() : enabled_(true) {} 23 TestingCommandObserverMock() : enabled_(true) {}
(...skipping 76 matching lines...) Expand 10 before | Expand all | Expand 10 after
98 100
99 // Remove one observer and update the command. Check the states, which 101 // Remove one observer and update the command. Check the states, which
100 // should be different. 102 // should be different.
101 command_updater.RemoveCommandObserver(&observer_remove); 103 command_updater.RemoveCommandObserver(&observer_remove);
102 command_updater.UpdateCommandEnabled(1, false); 104 command_updater.UpdateCommandEnabled(1, false);
103 command_updater.UpdateCommandEnabled(2, false); 105 command_updater.UpdateCommandEnabled(2, false);
104 command_updater.UpdateCommandEnabled(3, false); 106 command_updater.UpdateCommandEnabled(3, false);
105 EXPECT_TRUE(observer_remove.enabled()); 107 EXPECT_TRUE(observer_remove.enabled());
106 EXPECT_FALSE(observer_keep.enabled()); 108 EXPECT_FALSE(observer_keep.enabled());
107 } 109 }
OLDNEW
« no previous file with comments | « chrome/browser/command_updater.cc ('k') | chrome/browser/sync/sync_global_error_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698