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

Side by Side Diff: chrome/browser/ui/views/toolbar/toolbar_view_interactive_uitest.cc

Issue 1669793002: mac_views_browser: fixed interactive_ui_tests compilation. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Review fixes Created 4 years, 10 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/location.h" 5 #include "base/location.h"
6 #include "base/run_loop.h" 6 #include "base/run_loop.h"
7 #include "base/single_thread_task_runner.h" 7 #include "base/single_thread_task_runner.h"
8 #include "build/build_config.h" 8 #include "build/build_config.h"
9 #include "chrome/browser/extensions/extension_browsertest.h" 9 #include "chrome/browser/extensions/extension_browsertest.h"
10 #include "chrome/browser/ui/toolbar/toolbar_actions_bar.h" 10 #include "chrome/browser/ui/toolbar/toolbar_actions_bar.h"
(...skipping 144 matching lines...) Expand 10 before | Expand all | Expand 10 after
155 ASSERT_TRUE(LoadExtension(test_data_dir_.AppendASCII("api_test") 155 ASSERT_TRUE(LoadExtension(test_data_dir_.AppendASCII("api_test")
156 .AppendASCII("browser_action") 156 .AppendASCII("browser_action")
157 .AppendASCII("basics"))); 157 .AppendASCII("basics")));
158 base::RunLoop().RunUntilIdle(); // Ensure the extension is fully loaded. 158 base::RunLoop().RunUntilIdle(); // Ensure the extension is fully loaded.
159 159
160 ASSERT_EQ(1u, browser_actions()->VisibleBrowserActions()); 160 ASSERT_EQ(1u, browser_actions()->VisibleBrowserActions());
161 161
162 ToolbarActionView* view = browser_actions()->GetToolbarActionViewAt(0); 162 ToolbarActionView* view = browser_actions()->GetToolbarActionViewAt(0);
163 ASSERT_TRUE(view); 163 ASSERT_TRUE(view);
164 164
165 gfx::Point browser_action_view_loc = test::GetCenterInScreenCoordinates(view); 165 gfx::Point browser_action_view_loc =
166 gfx::Point app_button_loc = 166 ui_test_utils::GetCenterInScreenCoordinates(view);
167 test::GetCenterInScreenCoordinates(toolbar_view()->app_menu_button()); 167 gfx::Point app_button_loc = ui_test_utils::GetCenterInScreenCoordinates(
168 toolbar_view()->app_menu_button());
168 169
169 // Perform a drag and drop from the browser action view to the app button, 170 // Perform a drag and drop from the browser action view to the app button,
170 // which should open the app menu. 171 // which should open the app menu.
171 DoDragAndDrop(browser_action_view_loc, app_button_loc); 172 DoDragAndDrop(browser_action_view_loc, app_button_loc);
172 } 173 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698