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

Side by Side Diff: chrome/browser/ui/views/browser_action_view.cc

Issue 10827271: Replace views::Event with ui::Event. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 4 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 "chrome/browser/ui/views/browser_action_view.h" 5 #include "chrome/browser/ui/views/browser_action_view.h"
6 6
7 #include "base/utf_string_conversions.h" 7 #include "base/utf_string_conversions.h"
8 #include "chrome/browser/extensions/api/commands/command_service.h" 8 #include "chrome/browser/extensions/api/commands/command_service.h"
9 #include "chrome/browser/extensions/api/commands/command_service_factory.h" 9 #include "chrome/browser/extensions/api/commands/command_service_factory.h"
10 #include "chrome/browser/extensions/extension_context_menu_model.h" 10 #include "chrome/browser/extensions/extension_context_menu_model.h"
(...skipping 187 matching lines...) Expand 10 before | Expand all | Expand 10 after
198 // actions buttons, which can be hidden inside the overflow area. 198 // actions buttons, which can be hidden inside the overflow area.
199 return true; 199 return true;
200 } 200 }
201 201
202 void BrowserActionButton::GetAccessibleState(ui::AccessibleViewState* state) { 202 void BrowserActionButton::GetAccessibleState(ui::AccessibleViewState* state) {
203 views::MenuButton::GetAccessibleState(state); 203 views::MenuButton::GetAccessibleState(state);
204 state->role = ui::AccessibilityTypes::ROLE_PUSHBUTTON; 204 state->role = ui::AccessibilityTypes::ROLE_PUSHBUTTON;
205 } 205 }
206 206
207 void BrowserActionButton::ButtonPressed(views::Button* sender, 207 void BrowserActionButton::ButtonPressed(views::Button* sender,
208 const views::Event& event) { 208 const ui::Event& event) {
209 delegate_->OnBrowserActionExecuted(this); 209 delegate_->OnBrowserActionExecuted(this);
210 } 210 }
211 211
212 void BrowserActionButton::ShowContextMenuForView(View* source, 212 void BrowserActionButton::ShowContextMenuForView(View* source,
213 const gfx::Point& point) { 213 const gfx::Point& point) {
214 if (!extension()->ShowConfigureContextMenus()) 214 if (!extension()->ShowConfigureContextMenus())
215 return; 215 return;
216 216
217 SetButtonPushed(); 217 SetButtonPushed();
218 218
(...skipping 226 matching lines...) Expand 10 before | Expand all | Expand 10 after
445 445
446 extensions::Command browser_action_command; 446 extensions::Command browser_action_command;
447 if (!only_if_active || !command_service->GetBrowserActionCommand( 447 if (!only_if_active || !command_service->GetBrowserActionCommand(
448 extension_->id(), 448 extension_->id(),
449 extensions::CommandService::ACTIVE_ONLY, 449 extensions::CommandService::ACTIVE_ONLY,
450 &browser_action_command, 450 &browser_action_command,
451 NULL)) { 451 NULL)) {
452 GetFocusManager()->UnregisterAccelerator(*keybinding_.get(), this); 452 GetFocusManager()->UnregisterAccelerator(*keybinding_.get(), this);
453 } 453 }
454 } 454 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/views/browser_action_view.h ('k') | chrome/browser/ui/views/chrome_to_mobile_bubble_view.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698