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

Side by Side Diff: chrome/browser/ui/views/accessibility/accessibility_event_router_views_unittest.cc

Issue 393943006: MacViews: Remove GetMenu() from MenuRunner. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix unit test compile Created 6 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
« no previous file with comments | « no previous file | chrome/browser/ui/views/renderer_context_menu/render_view_context_menu_views.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 <string> 5 #include <string>
6 6
7 #include "base/message_loop/message_loop.h" 7 #include "base/message_loop/message_loop.h"
8 #include "base/strings/string_util.h" 8 #include "base/strings/string_util.h"
9 #include "base/strings/utf_string_conversions.h" 9 #include "base/strings/utf_string_conversions.h"
10 #include "chrome/browser/accessibility/accessibility_extension_api.h" 10 #include "chrome/browser/accessibility/accessibility_extension_api.h"
11 #include "chrome/browser/accessibility/accessibility_extension_api_constants.h" 11 #include "chrome/browser/accessibility/accessibility_extension_api_constants.h"
12 #include "chrome/browser/ui/views/accessibility/accessibility_event_router_views .h" 12 #include "chrome/browser/ui/views/accessibility/accessibility_event_router_views .h"
13 #include "chrome/test/base/testing_profile.h" 13 #include "chrome/test/base/testing_profile.h"
14 #include "testing/gtest/include/gtest/gtest.h" 14 #include "testing/gtest/include/gtest/gtest.h"
15 #include "ui/accessibility/ax_enums.h" 15 #include "ui/accessibility/ax_enums.h"
16 #include "ui/accessibility/ax_view_state.h" 16 #include "ui/accessibility/ax_view_state.h"
17 #include "ui/base/models/simple_menu_model.h" 17 #include "ui/base/models/simple_menu_model.h"
18 #include "ui/views/controls/button/label_button.h" 18 #include "ui/views/controls/button/label_button.h"
19 #include "ui/views/controls/label.h" 19 #include "ui/views/controls/label.h"
20 #include "ui/views/controls/menu/menu_item_view.h" 20 #include "ui/views/controls/menu/menu_item_view.h"
21 #include "ui/views/controls/menu/menu_model_adapter.h"
21 #include "ui/views/controls/menu/menu_runner.h" 22 #include "ui/views/controls/menu/menu_runner.h"
22 #include "ui/views/controls/menu/submenu_view.h" 23 #include "ui/views/controls/menu/submenu_view.h"
23 #include "ui/views/layout/grid_layout.h" 24 #include "ui/views/layout/grid_layout.h"
24 #include "ui/views/test/test_views_delegate.h" 25 #include "ui/views/test/test_views_delegate.h"
25 #include "ui/views/widget/native_widget.h" 26 #include "ui/views/widget/native_widget.h"
26 #include "ui/views/widget/root_view.h" 27 #include "ui/views/widget/root_view.h"
27 #include "ui/views/widget/widget.h" 28 #include "ui/views/widget/widget.h"
28 #include "ui/views/widget/widget_delegate.h" 29 #include "ui/views/widget/widget_delegate.h"
29 30
30 #if defined(OS_WIN) 31 #if defined(OS_WIN)
(...skipping 408 matching lines...) Expand 10 before | Expand all | Expand 10 after
439 440
440 views::MenuItemView* BuildMenu() { 441 views::MenuItemView* BuildMenu() {
441 menu_model_.reset(new ui::SimpleMenuModel(this)); 442 menu_model_.reset(new ui::SimpleMenuModel(this));
442 menu_model_->AddItem(IDC_MENU_ITEM_1, ASCIIToUTF16("Item 1")); 443 menu_model_->AddItem(IDC_MENU_ITEM_1, ASCIIToUTF16("Item 1"));
443 menu_model_->AddItem(IDC_MENU_ITEM_2, ASCIIToUTF16("Item 2")); 444 menu_model_->AddItem(IDC_MENU_ITEM_2, ASCIIToUTF16("Item 2"));
444 menu_model_->AddSeparator(ui::NORMAL_SEPARATOR); 445 menu_model_->AddSeparator(ui::NORMAL_SEPARATOR);
445 menu_model_->AddItem(IDC_MENU_INVISIBLE, ASCIIToUTF16("Invisible")); 446 menu_model_->AddItem(IDC_MENU_INVISIBLE, ASCIIToUTF16("Invisible"));
446 menu_model_->AddSeparator(ui::NORMAL_SEPARATOR); 447 menu_model_->AddSeparator(ui::NORMAL_SEPARATOR);
447 menu_model_->AddItem(IDC_MENU_ITEM_3, ASCIIToUTF16("Item 3")); 448 menu_model_->AddItem(IDC_MENU_ITEM_3, ASCIIToUTF16("Item 3"));
448 449
449 menu_runner_.reset(new views::MenuRunner(menu_model_.get(), 0)); 450 menu_adapter_.reset(new views::MenuModelAdapter(menu_model_.get()));
450 return menu_runner_->GetMenu(); 451 views::MenuItemView* menu_view = menu_adapter_->CreateMenu();
452 menu_runner_.reset(new views::MenuRunner(menu_view, 0));
453 return menu_view;
451 } 454 }
452 455
453 virtual bool IsCommandIdChecked(int command_id) const OVERRIDE { 456 virtual bool IsCommandIdChecked(int command_id) const OVERRIDE {
454 return false; 457 return false;
455 } 458 }
456 459
457 virtual bool IsCommandIdEnabled(int command_id) const OVERRIDE { 460 virtual bool IsCommandIdEnabled(int command_id) const OVERRIDE {
458 return true; 461 return true;
459 } 462 }
460 463
461 virtual bool IsCommandIdVisible(int command_id) const OVERRIDE { 464 virtual bool IsCommandIdVisible(int command_id) const OVERRIDE {
462 return command_id != IDC_MENU_INVISIBLE; 465 return command_id != IDC_MENU_INVISIBLE;
463 } 466 }
464 467
465 virtual bool GetAcceleratorForCommandId( 468 virtual bool GetAcceleratorForCommandId(
466 int command_id, 469 int command_id,
467 ui::Accelerator* accelerator) OVERRIDE { 470 ui::Accelerator* accelerator) OVERRIDE {
468 return false; 471 return false;
469 } 472 }
470 473
471 virtual void ExecuteCommand(int command_id, int event_flags) OVERRIDE { 474 virtual void ExecuteCommand(int command_id, int event_flags) OVERRIDE {
472 } 475 }
473 476
474 private: 477 private:
475 scoped_ptr<ui::SimpleMenuModel> menu_model_; 478 scoped_ptr<ui::SimpleMenuModel> menu_model_;
479 scoped_ptr<views::MenuModelAdapter> menu_adapter_;
476 scoped_ptr<views::MenuRunner> menu_runner_; 480 scoped_ptr<views::MenuRunner> menu_runner_;
477 481
478 DISALLOW_COPY_AND_ASSIGN(SimpleMenuDelegate); 482 DISALLOW_COPY_AND_ASSIGN(SimpleMenuDelegate);
479 }; 483 };
480 484
481 } // namespace 485 } // namespace
482 486
483 TEST_F(AccessibilityEventRouterViewsTest, MenuIndexAndCountForInvisibleMenu) { 487 TEST_F(AccessibilityEventRouterViewsTest, MenuIndexAndCountForInvisibleMenu) {
484 SimpleMenuDelegate menu_delegate; 488 SimpleMenuDelegate menu_delegate;
485 views::MenuItemView* menu = menu_delegate.BuildMenu(); 489 views::MenuItemView* menu = menu_delegate.BuildMenu();
(...skipping 16 matching lines...) Expand all
502 506
503 AccessibilityEventRouterViews::RecursiveGetMenuItemIndexAndCount( 507 AccessibilityEventRouterViews::RecursiveGetMenuItemIndexAndCount(
504 menu_container, 508 menu_container,
505 menu->GetMenuItemByID(kTestCases[i].command_id), 509 menu->GetMenuItemByID(kTestCases[i].command_id),
506 &index, 510 &index,
507 &count); 511 &count);
508 EXPECT_EQ(kTestCases[i].expected_index, index) << "Case " << i; 512 EXPECT_EQ(kTestCases[i].expected_index, index) << "Case " << i;
509 EXPECT_EQ(kTestCases[i].expected_count, count) << "Case " << i; 513 EXPECT_EQ(kTestCases[i].expected_count, count) << "Case " << i;
510 } 514 }
511 } 515 }
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/ui/views/renderer_context_menu/render_view_context_menu_views.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698