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

Side by Side Diff: chrome/browser/ui/gtk/browser_actions_toolbar_gtk.cc

Issue 9965049: ui/base: Clean up our AcceleratorGtk class. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 8 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/gtk/browser_actions_toolbar_gtk.h" 5 #include "chrome/browser/ui/gtk/browser_actions_toolbar_gtk.h"
6 6
7 #include <gtk/gtk.h>
8
7 #include <algorithm> 9 #include <algorithm>
8 #include <gtk/gtk.h>
9 #include <vector> 10 #include <vector>
10 11
11 #include "base/bind.h" 12 #include "base/bind.h"
12 #include "base/i18n/rtl.h" 13 #include "base/i18n/rtl.h"
13 #include "base/message_loop.h" 14 #include "base/message_loop.h"
14 #include "base/utf_string_conversions.h" 15 #include "base/utf_string_conversions.h"
15 #include "chrome/browser/extensions/extension_browser_event_router.h" 16 #include "chrome/browser/extensions/extension_browser_event_router.h"
16 #include "chrome/browser/extensions/extension_context_menu_model.h" 17 #include "chrome/browser/extensions/extension_context_menu_model.h"
17 #include "chrome/browser/extensions/extension_service.h" 18 #include "chrome/browser/extensions/extension_service.h"
18 #include "chrome/browser/extensions/image_loading_tracker.h" 19 #include "chrome/browser/extensions/image_loading_tracker.h"
19 #include "chrome/browser/profiles/profile.h" 20 #include "chrome/browser/profiles/profile.h"
20 #include "chrome/browser/sessions/restore_tab_helper.h" 21 #include "chrome/browser/sessions/restore_tab_helper.h"
21 #include "chrome/browser/ui/browser.h" 22 #include "chrome/browser/ui/browser.h"
22 #include "chrome/browser/ui/gtk/browser_window_gtk.h" 23 #include "chrome/browser/ui/gtk/browser_window_gtk.h"
23 #include "chrome/browser/ui/gtk/extensions/extension_popup_gtk.h" 24 #include "chrome/browser/ui/gtk/extensions/extension_popup_gtk.h"
24 #include "chrome/browser/ui/gtk/gtk_chrome_button.h" 25 #include "chrome/browser/ui/gtk/gtk_chrome_button.h"
25 #include "chrome/browser/ui/gtk/gtk_chrome_shrinkable_hbox.h" 26 #include "chrome/browser/ui/gtk/gtk_chrome_shrinkable_hbox.h"
26 #include "chrome/browser/ui/gtk/gtk_util.h" 27 #include "chrome/browser/ui/gtk/gtk_util.h"
27 #include "chrome/browser/ui/gtk/hover_controller_gtk.h" 28 #include "chrome/browser/ui/gtk/hover_controller_gtk.h"
28 #include "chrome/browser/ui/gtk/menu_gtk.h" 29 #include "chrome/browser/ui/gtk/menu_gtk.h"
29 #include "chrome/browser/ui/gtk/theme_service_gtk.h" 30 #include "chrome/browser/ui/gtk/theme_service_gtk.h"
30 #include "chrome/browser/ui/gtk/view_id_util.h" 31 #include "chrome/browser/ui/gtk/view_id_util.h"
31 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h" 32 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"
32 #include "chrome/common/chrome_notification_types.h" 33 #include "chrome/common/chrome_notification_types.h"
33 #include "chrome/common/extensions/extension.h" 34 #include "chrome/common/extensions/extension.h"
35 #include "chrome/common/extensions/extension_action.h"
34 #include "chrome/common/extensions/extension_manifest_constants.h" 36 #include "chrome/common/extensions/extension_manifest_constants.h"
35 #include "chrome/common/extensions/extension_action.h"
36 #include "chrome/common/extensions/extension_resource.h" 37 #include "chrome/common/extensions/extension_resource.h"
37 #include "content/public/browser/notification_details.h" 38 #include "content/public/browser/notification_details.h"
38 #include "content/public/browser/notification_source.h" 39 #include "content/public/browser/notification_source.h"
39 #include "grit/theme_resources.h" 40 #include "grit/theme_resources.h"
40 #include "grit/theme_resources_standard.h" 41 #include "grit/theme_resources_standard.h"
41 #include "grit/ui_resources.h" 42 #include "grit/ui_resources.h"
42 #include "ui/base/accelerators/accelerator_gtk.h" 43 #include "ui/base/accelerators/accelerator_gtk.h"
43 #include "ui/base/gtk/gtk_compat.h" 44 #include "ui/base/gtk/gtk_compat.h"
44 #include "ui/base/resource/resource_bundle.h" 45 #include "ui/base/resource/resource_bundle.h"
45 #include "ui/gfx/canvas_skia_paint.h" 46 #include "ui/gfx/canvas_skia_paint.h"
(...skipping 345 matching lines...) Expand 10 before | Expand all | Expand 10 after
391 commands[i].accelerator().IsAltDown())); 392 commands[i].accelerator().IsAltDown()));
392 393
393 gfx::NativeWindow window = 394 gfx::NativeWindow window =
394 toolbar_->browser()->window()->GetNativeHandle(); 395 toolbar_->browser()->window()->GetNativeHandle();
395 accel_group_ = gtk_accel_group_new(); 396 accel_group_ = gtk_accel_group_new();
396 gtk_window_add_accel_group(window, accel_group_); 397 gtk_window_add_accel_group(window, accel_group_);
397 398
398 gtk_accel_group_connect( 399 gtk_accel_group_connect(
399 accel_group_, 400 accel_group_,
400 keybinding_.get()->GetGdkKeyCode(), 401 keybinding_.get()->GetGdkKeyCode(),
401 keybinding_.get()->gdk_modifier_type(), 402 keybinding_.get()->GetGdkModifierType(),
402 GtkAccelFlags(0), 403 GtkAccelFlags(0),
403 g_cclosure_new(G_CALLBACK(OnGtkAccelerator), this, NULL)); 404 g_cclosure_new(G_CALLBACK(OnGtkAccelerator), this, NULL));
404 405
405 // Since we've added an accelerator, we'll need to unregister it before 406 // Since we've added an accelerator, we'll need to unregister it before
406 // the window is closed, so we listen for the window being closed. 407 // the window is closed, so we listen for the window being closed.
407 registrar_.Add(this, 408 registrar_.Add(this,
408 chrome::NOTIFICATION_WINDOW_CLOSED, 409 chrome::NOTIFICATION_WINDOW_CLOSED,
409 content::Source<GtkWindow>(window)); 410 content::Source<GtkWindow>(window));
410 break; 411 break;
411 } 412 }
(...skipping 672 matching lines...) Expand 10 before | Expand all | Expand 10 after
1084 1085
1085 menu->PopupAsContext(gfx::Point(event->x_root, event->y_root), 1086 menu->PopupAsContext(gfx::Point(event->x_root, event->y_root),
1086 event->time); 1087 event->time);
1087 return TRUE; 1088 return TRUE;
1088 } 1089 }
1089 1090
1090 void BrowserActionsToolbarGtk::OnButtonShowOrHide(GtkWidget* sender) { 1091 void BrowserActionsToolbarGtk::OnButtonShowOrHide(GtkWidget* sender) {
1091 if (!resize_animation_.is_animating()) 1092 if (!resize_animation_.is_animating())
1092 UpdateChevronVisibility(); 1093 UpdateChevronVisibility();
1093 } 1094 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698