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

Side by Side Diff: chrome/browser/extensions/extension_context_menu_browsertest.cc

Issue 10702029: Move tab functions off Browser into browser_tabstrip and browser_tabrestore. (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 "base/utf_string_conversions.h" 5 #include "base/utf_string_conversions.h"
6 #include "chrome/app/chrome_command_ids.h" 6 #include "chrome/app/chrome_command_ids.h"
7 #include "chrome/browser/extensions/extension_browsertest.h" 7 #include "chrome/browser/extensions/extension_browsertest.h"
8 #include "chrome/browser/extensions/extension_context_menu_model.h" 8 #include "chrome/browser/extensions/extension_context_menu_model.h"
9 #include "chrome/browser/extensions/extension_service.h" 9 #include "chrome/browser/extensions/extension_service.h"
10 #include "chrome/browser/extensions/extension_system.h" 10 #include "chrome/browser/extensions/extension_system.h"
11 #include "chrome/browser/extensions/extension_test_message_listener.h" 11 #include "chrome/browser/extensions/extension_test_message_listener.h"
12 #include "chrome/browser/extensions/lazy_background_page_test_util.h" 12 #include "chrome/browser/extensions/lazy_background_page_test_util.h"
13 #include "chrome/browser/extensions/test_management_policy.h" 13 #include "chrome/browser/extensions/test_management_policy.h"
14 #include "chrome/browser/profiles/profile.h" 14 #include "chrome/browser/profiles/profile.h"
15 #include "chrome/browser/tab_contents/render_view_context_menu.h" 15 #include "chrome/browser/tab_contents/render_view_context_menu.h"
16 #include "chrome/browser/ui/browser.h" 16 #include "chrome/browser/ui/browser.h"
17 #include "chrome/browser/ui/browser_finder.h" 17 #include "chrome/browser/ui/browser_finder.h"
18 #include "chrome/browser/ui/browser_tabstrip.h"
18 #include "chrome/common/chrome_switches.h" 19 #include "chrome/common/chrome_switches.h"
19 #include "chrome/test/base/ui_test_utils.h" 20 #include "chrome/test/base/ui_test_utils.h"
20 #include "content/public/common/context_menu_params.h" 21 #include "content/public/common/context_menu_params.h"
21 #include "net/base/mock_host_resolver.h" 22 #include "net/base/mock_host_resolver.h"
22 #include "third_party/WebKit/Source/WebKit/chromium/public/WebContextMenuData.h" 23 #include "third_party/WebKit/Source/WebKit/chromium/public/WebContextMenuData.h"
23 #include "ui/base/models/menu_model.h" 24 #include "ui/base/models/menu_model.h"
24 25
25 using WebKit::WebContextMenuData; 26 using WebKit::WebContextMenuData;
26 using content::WebContents; 27 using content::WebContents;
27 using ui::MenuModel; 28 using ui::MenuModel;
(...skipping 108 matching lines...) Expand 10 before | Expand all | Expand 10 after
136 std::string subdirectory) { 137 std::string subdirectory) {
137 FilePath extension_dir = 138 FilePath extension_dir =
138 test_data_dir_.AppendASCII("context_menus").AppendASCII(subdirectory); 139 test_data_dir_.AppendASCII("context_menus").AppendASCII(subdirectory);
139 return LoadExtensionIncognito(extension_dir); 140 return LoadExtensionIncognito(extension_dir);
140 } 141 }
141 142
142 TestRenderViewContextMenu* CreateMenu(Browser* browser, 143 TestRenderViewContextMenu* CreateMenu(Browser* browser,
143 const GURL& page_url, 144 const GURL& page_url,
144 const GURL& link_url, 145 const GURL& link_url,
145 const GURL& frame_url) { 146 const GURL& frame_url) {
146 WebContents* web_contents = browser->GetActiveWebContents(); 147 WebContents* web_contents = chrome::GetActiveWebContents(browser);
147 WebContextMenuData data; 148 WebContextMenuData data;
148 content::ContextMenuParams params(data); 149 content::ContextMenuParams params(data);
149 params.page_url = page_url; 150 params.page_url = page_url;
150 params.link_url = link_url; 151 params.link_url = link_url;
151 params.frame_url = frame_url; 152 params.frame_url = frame_url;
152 TestRenderViewContextMenu* menu = 153 TestRenderViewContextMenu* menu =
153 new TestRenderViewContextMenu(web_contents, params); 154 new TestRenderViewContextMenu(web_contents, params);
154 menu->Init(); 155 menu->Init();
155 return menu; 156 return menu;
156 } 157 }
(...skipping 428 matching lines...) Expand 10 before | Expand all | Expand 10 after
585 EXPECT_FALSE(menu->IsCommandIdChecked(command_id)); 586 EXPECT_FALSE(menu->IsCommandIdChecked(command_id));
586 587
587 // Executing the checkbox also fires the onClicked event. 588 // Executing the checkbox also fires the onClicked event.
588 ExtensionTestMessageListener listener("onClicked fired for checkbox1", false); 589 ExtensionTestMessageListener listener("onClicked fired for checkbox1", false);
589 menu->ExecuteCommand(command_id); 590 menu->ExecuteCommand(command_id);
590 checkbox_checked.WaitUntilClosed(); 591 checkbox_checked.WaitUntilClosed();
591 592
592 EXPECT_TRUE(menu->IsCommandIdChecked(command_id)); 593 EXPECT_TRUE(menu->IsCommandIdChecked(command_id));
593 ASSERT_TRUE(listener.WaitUntilSatisfied()); 594 ASSERT_TRUE(listener.WaitUntilSatisfied());
594 } 595 }
OLDNEW
« no previous file with comments | « chrome/browser/extensions/extension_browser_event_router.cc ('k') | chrome/browser/extensions/extension_context_menu_model.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698