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

Side by Side Diff: ash/tooltips/tooltip_controller_unittest.cc

Issue 2618143002: cros: Remove some OS platform ifdefs from ash (Closed)
Patch Set: rebase Created 3 years, 11 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
« no previous file with comments | « ash/test/test_shell_delegate.cc ('k') | ash/touch/touch_observer_hud.h » ('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) 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 "ash/shell.h" 5 #include "ash/shell.h"
6 #include "ash/test/ash_test_base.h" 6 #include "ash/test/ash_test_base.h"
7 #include "base/strings/utf_string_conversions.h" 7 #include "base/strings/utf_string_conversions.h"
8 #include "ui/aura/env.h" 8 #include "ui/aura/env.h"
9 #include "ui/aura/window.h" 9 #include "ui/aura/window.h"
10 #include "ui/aura/window_event_dispatcher.h" 10 #include "ui/aura/window_event_dispatcher.h"
(...skipping 138 matching lines...) Expand 10 before | Expand all | Expand 10 after
149 // Show tooltip on second display. 149 // Show tooltip on second display.
150 ui::test::EventGenerator generator(root_windows[1]); 150 ui::test::EventGenerator generator(root_windows[1]);
151 generator.MoveMouseRelativeTo(widget2->GetNativeView(), 151 generator.MoveMouseRelativeTo(widget2->GetNativeView(),
152 view2->bounds().CenterPoint()); 152 view2->bounds().CenterPoint());
153 helper_->FireTooltipTimer(); 153 helper_->FireTooltipTimer();
154 EXPECT_TRUE(helper_->IsTooltipVisible()); 154 EXPECT_TRUE(helper_->IsTooltipVisible());
155 155
156 // Get rid of secondary display. This destroy's the tooltip's aura window. If 156 // Get rid of secondary display. This destroy's the tooltip's aura window. If
157 // we have handled this case, we will not crash in the following statement. 157 // we have handled this case, we will not crash in the following statement.
158 UpdateDisplay("1000x600"); 158 UpdateDisplay("1000x600");
159 #if !defined(OS_WIN)
160 // TODO(cpu): Detangle the window destruction notification. Currently
161 // the TooltipController::OnWindowDestroyed is not being called then the
162 // display is torn down so the tooltip is is still there.
163 EXPECT_FALSE(helper_->IsTooltipVisible()); 159 EXPECT_FALSE(helper_->IsTooltipVisible());
164 #endif
165 EXPECT_EQ(widget2->GetNativeView()->GetRootWindow(), root_windows[0]); 160 EXPECT_EQ(widget2->GetNativeView()->GetRootWindow(), root_windows[0]);
166 161
167 // The tooltip should create a new aura window for itself, so we should still 162 // The tooltip should create a new aura window for itself, so we should still
168 // be able to show tooltips on the primary display. 163 // be able to show tooltips on the primary display.
169 ui::test::EventGenerator generator1(root_windows[0]); 164 ui::test::EventGenerator generator1(root_windows[0]);
170 generator1.MoveMouseRelativeTo(widget1->GetNativeView(), 165 generator1.MoveMouseRelativeTo(widget1->GetNativeView(),
171 view1->bounds().CenterPoint()); 166 view1->bounds().CenterPoint());
172 helper_->FireTooltipTimer(); 167 helper_->FireTooltipTimer();
173 EXPECT_TRUE(helper_->IsTooltipVisible()); 168 EXPECT_TRUE(helper_->IsTooltipVisible());
174 } 169 }
175 170
176 } // namespace test 171 } // namespace test
177 } // namespace ash 172 } // namespace ash
OLDNEW
« no previous file with comments | « ash/test/test_shell_delegate.cc ('k') | ash/touch/touch_observer_hud.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698