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

Side by Side Diff: chrome/browser/chromeos/setting_level_bubble_browsertest.cc

Issue 8909002: views: Convert IsEnabled() to just enabled() since it's just a simple accessor. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years 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) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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/message_loop.h" 5 #include "base/message_loop.h"
6 #include "chrome/browser/chromeos/setting_level_bubble.h" 6 #include "chrome/browser/chromeos/setting_level_bubble.h"
7 #include "chrome/browser/chromeos/setting_level_bubble_view.h" 7 #include "chrome/browser/chromeos/setting_level_bubble_view.h"
8 #include "chrome/test/base/in_process_browser_test.h" 8 #include "chrome/test/base/in_process_browser_test.h"
9 #include "third_party/skia/include/core/SkBitmap.h" 9 #include "third_party/skia/include/core/SkBitmap.h"
10 #include "ui/views/controls/progress_bar.h" 10 #include "ui/views/controls/progress_bar.h"
(...skipping 28 matching lines...) Expand all
39 EXPECT_EQ(NULL, bubble.view_); 39 EXPECT_EQ(NULL, bubble.view_);
40 EXPECT_EQ(&up_icon_, bubble.increase_icon_); 40 EXPECT_EQ(&up_icon_, bubble.increase_icon_);
41 EXPECT_EQ(&down_icon_, bubble.decrease_icon_); 41 EXPECT_EQ(&down_icon_, bubble.decrease_icon_);
42 EXPECT_EQ(&mute_icon_, bubble.disabled_icon_); 42 EXPECT_EQ(&mute_icon_, bubble.disabled_icon_);
43 43
44 // Update setting: 44 // Update setting:
45 // Old target is 50, new target is 70, set enable = false. 45 // Old target is 50, new target is 70, set enable = false.
46 bubble.ShowBubble(70, false); 46 bubble.ShowBubble(70, false);
47 EXPECT_TRUE(bubble.view_->GetWidget()->IsVisible()); 47 EXPECT_TRUE(bubble.view_->GetWidget()->IsVisible());
48 EXPECT_EQ(&mute_icon_, bubble.view_->icon_); 48 EXPECT_EQ(&mute_icon_, bubble.view_->icon_);
49 EXPECT_FALSE(bubble.view_->progress_bar_->IsEnabled()); 49 EXPECT_FALSE(bubble.view_->progress_bar_->enabled());
50 50
51 // Old target is 70, new target is 30, set enable = true. 51 // Old target is 70, new target is 30, set enable = true.
52 bubble.ShowBubble(30, true); 52 bubble.ShowBubble(30, true);
53 EXPECT_EQ(&down_icon_, bubble.view_->icon_); 53 EXPECT_EQ(&down_icon_, bubble.view_->icon_);
54 EXPECT_TRUE(bubble.view_->progress_bar_->IsEnabled()); 54 EXPECT_TRUE(bubble.view_->progress_bar_->enabled());
55 55
56 // Old target is 30, new target is 40, set enable = true. 56 // Old target is 30, new target is 40, set enable = true.
57 bubble.ShowBubble(30, true); 57 bubble.ShowBubble(30, true);
58 EXPECT_EQ(&up_icon_, bubble.view_->icon_); 58 EXPECT_EQ(&up_icon_, bubble.view_->icon_);
59 EXPECT_TRUE(bubble.view_->progress_bar_->IsEnabled()); 59 EXPECT_TRUE(bubble.view_->progress_bar_->enabled());
60 60
61 // Old target is 30, new target is 0, set enable = true. 61 // Old target is 30, new target is 0, set enable = true.
62 bubble.ShowBubble(0, true); 62 bubble.ShowBubble(0, true);
63 EXPECT_EQ(&mute_icon_, bubble.view_->icon_); 63 EXPECT_EQ(&mute_icon_, bubble.view_->icon_);
64 EXPECT_TRUE(bubble.view_->progress_bar_->IsEnabled()); 64 EXPECT_TRUE(bubble.view_->progress_bar_->enabled());
65 bubble.HideBubble(); 65 bubble.HideBubble();
66 MessageLoop::current()->RunAllPending(); 66 MessageLoop::current()->RunAllPending();
67 } 67 }
68 68
69 IN_PROC_BROWSER_TEST_F(SettingLevelBubbleTest, ShowBubble) { 69 IN_PROC_BROWSER_TEST_F(SettingLevelBubbleTest, ShowBubble) {
70 // Create setting at 30 percent, enabled. 70 // Create setting at 30 percent, enabled.
71 SettingLevelBubble bubble(&up_icon_, 71 SettingLevelBubble bubble(&up_icon_,
72 &down_icon_, 72 &down_icon_,
73 &mute_icon_); 73 &mute_icon_);
74 bubble.UpdateWithoutShowingBubble(30, false); 74 bubble.UpdateWithoutShowingBubble(30, false);
75 EXPECT_EQ(NULL, bubble.view_); 75 EXPECT_EQ(NULL, bubble.view_);
76 EXPECT_EQ(30, bubble.current_percent_); 76 EXPECT_EQ(30, bubble.current_percent_);
77 77
78 // Show bubble at 40 percent, enabled. 78 // Show bubble at 40 percent, enabled.
79 bubble.ShowBubble(40, true); 79 bubble.ShowBubble(40, true);
80 EXPECT_TRUE(bubble.view_ != NULL); 80 EXPECT_TRUE(bubble.view_ != NULL);
81 EXPECT_TRUE(bubble.view_->GetWidget()->IsVisible()); 81 EXPECT_TRUE(bubble.view_->GetWidget()->IsVisible());
82 82
83 // Update to 0 percent and close. 83 // Update to 0 percent and close.
84 bubble.UpdateWithoutShowingBubble(0, true); 84 bubble.UpdateWithoutShowingBubble(0, true);
85 bubble.OnWidgetClosing(bubble.view_->GetWidget()); 85 bubble.OnWidgetClosing(bubble.view_->GetWidget());
86 EXPECT_EQ(0, bubble.current_percent_); 86 EXPECT_EQ(0, bubble.current_percent_);
87 } 87 }
88 88
89 } // namespace chromeos 89 } // namespace chromeos
OLDNEW
« no previous file with comments | « chrome/browser/chromeos/options/wifi_config_view.cc ('k') | chrome/browser/ui/views/download/download_item_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698