Linux GTK: Support keyboard navigation in avatar menu.

chrome/browser/ui/gtk/avatar_menu_item_gtk.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/gtk/avatar_menu_item_gtk.h"
 
+#include <gdk/gdkkeysyms.h>
+
 #include "base/bind.h"
 #include "base/message_loop.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/ui/gtk/gtk_chrome_link_button.h"
 #include "chrome/browser/ui/gtk/gtk_theme_service.h"
 #include "chrome/browser/ui/gtk/gtk_util.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "content/public/browser/notification_source.h"
 #include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
@@ skipping 58 matching lines @@
   // try to destroy this AvatarMenuItemGtk.
   // This is not OK to do this from the signal handler, so we'll
   // defer it.
   MessageLoop::current()->PostTask(
       FROM_HERE,
       base::Bind(&AvatarMenuItemGtk::OpenProfile,
                  weak_factory_.GetWeakPtr()));
   return FALSE;
 }
 
+gboolean AvatarMenuItemGtk::OnProfileKeyPress(GtkWidget* widget,
+                                              GdkEventKey* event) {
+  // delegate_->EditProfile() will close the avatar bubble which in turn
+  // try to destroy this AvatarMenuItemGtk.
+  // This is not OK to do this from the signal handler, so we'll
+  // defer it.
+  if (event->keyval == GDK_Return ||
+      event->keyval == GDK_ISO_Enter ||
+      event->keyval == GDK_KP_Enter) {
+    if (item_.active)
+      MessageLoop::current()->PostTask(
+          FROM_HERE,
+          base::Bind(&AvatarMenuItemGtk::EditProfile,
+                     weak_factory_.GetWeakPtr()));
+    else
+      MessageLoop::current()->PostTask(
+          FROM_HERE,
+          base::Bind(&AvatarMenuItemGtk::OpenProfile,
+                     weak_factory_.GetWeakPtr()));
+  }
+
+  return FALSE;
+}
+
+void AvatarMenuItemGtk::ShowStatusLabel() {
+  gtk_widget_show(status_label_);
+  gtk_widget_hide(link_alignment_);
+}
+
+void AvatarMenuItemGtk::ShowEditLink() {
+  gtk_widget_hide(status_label_);
+  gtk_widget_show(link_alignment_);
+}
+
+gboolean AvatarMenuItemGtk::OnProfileFocusIn(GtkWidget* widget,
+                                             GdkEventFocus* event) {
+  gtk_widget_modify_bg(widget, GTK_STATE_NORMAL, &highlighted_color_);
+  if (item_.active)
+    ShowEditLink();
+
+  return FALSE;
+}
+
+gboolean AvatarMenuItemGtk::OnProfileFocusOut(GtkWidget* widget,
+                                              GdkEventFocus* event) {
+  gtk_widget_modify_bg(widget, GTK_STATE_NORMAL, unhighlighted_color_);
+  if (item_.active)
+    ShowStatusLabel();
+
+  return FALSE;
+}
+
 gboolean AvatarMenuItemGtk::OnProfileEnter(GtkWidget* widget,
                                            GdkEventCrossing* event) {
   if (event->detail == GDK_NOTIFY_INFERIOR)
     return FALSE;
 
   gtk_widget_modify_bg(widget, GTK_STATE_NORMAL, &highlighted_color_);
-  if (item_.active) {
-    gtk_widget_hide(status_label_);
-    gtk_widget_show(link_alignment_);
-  }
+  if (item_.active)
+    ShowEditLink();
 
   return FALSE;
 }
 
 gboolean AvatarMenuItemGtk::OnProfileLeave(GtkWidget* widget,
                                            GdkEventCrossing* event) {
   if (event->detail == GDK_NOTIFY_INFERIOR)
     return FALSE;
 
   gtk_widget_modify_bg(widget, GTK_STATE_NORMAL, unhighlighted_color_);
-  if (item_.active) {
-    gtk_widget_show(status_label_);
-    gtk_widget_hide(link_alignment_);
-  }
+  if (item_.active)
+    ShowStatusLabel();
 
   return FALSE;
 }
 
 void AvatarMenuItemGtk::EditProfile() {
   delegate_->EditProfile(item_index_);
 }
 
 void AvatarMenuItemGtk::Observe(int type,
                                 const content::NotificationSource& source,
@@ skipping 34 matching lines @@
 
 void AvatarMenuItemGtk::Init(GtkThemeService* theme_service) {
   widget_.Own(gtk_event_box_new());
 
   g_signal_connect(widget_.get(), "button-press-event",
       G_CALLBACK(OnProfileClickThunk), this);
   g_signal_connect(widget_.get(), "enter-notify-event",
       G_CALLBACK(OnProfileEnterThunk), this);
   g_signal_connect(widget_.get(), "leave-notify-event",
       G_CALLBACK(OnProfileLeaveThunk), this);
+  g_signal_connect(widget_.get(), "focus-in-event",
+      G_CALLBACK(OnProfileFocusInThunk), this);
+  g_signal_connect(widget_.get(), "focus-out-event",
+      G_CALLBACK(OnProfileFocusOutThunk), this);
+  g_signal_connect(widget_.get(), "key-press-event",
+      G_CALLBACK(OnProfileKeyPressThunk), this);
 
   GtkWidget* item_hbox = gtk_hbox_new(FALSE, ui::kControlSpacing);
   GdkPixbuf* avatar_pixbuf = NULL;
 
   // If this profile is active then draw a check mark on the bottom right
   // of the profile icon.
   if (item_.active) {
     const SkBitmap* avatar_image = item_.icon.ToSkBitmap();
     gfx::CanvasSkia canvas(avatar_image->width(),
                            avatar_image->height(),
@@ skipping 66 matching lines @@
 
     GtkSizeGroup* size_group = gtk_size_group_new(GTK_SIZE_GROUP_BOTH);
     gtk_size_group_add_widget(size_group, status_label_);
     gtk_size_group_add_widget(size_group, link_alignment_);
     g_object_unref(size_group);
   }
 
   gtk_box_pack_start(GTK_BOX(item_hbox), item_vbox, TRUE, TRUE, 0);
   gtk_container_add(GTK_CONTAINER(widget_.get()), item_hbox);
 }