Add Delete Support to New Autofill UI

chrome/browser/ui/gtk/autofill/autofill_popup_view_gtk.cc

Index: chrome/browser/ui/gtk/autofill/autofill_popup_view_gtk.cc
diff --git a/chrome/browser/ui/gtk/autofill/autofill_popup_view_gtk.cc b/chrome/browser/ui/gtk/autofill/autofill_popup_view_gtk.cc
index ed1f84a20708654b80bdc23e00aa4d833440537e..795dd8e12c56ebdc659851b743379fedac2f1ed3 100644
--- a/chrome/browser/ui/gtk/autofill/autofill_popup_view_gtk.cc
+++ b/chrome/browser/ui/gtk/autofill/autofill_popup_view_gtk.cc
@@ -238,7 +238,7 @@ bool AutofillPopupViewGtk::HandleKeyPressEvent(GdkEventKey* event) {
       return true;
     case GDK_Delete:
     case GDK_KP_Delete:
-      return (event->state == GDK_SHIFT_MASK) && RemoveSelectedLine();
+      return (event->state & GDK_SHIFT_MASK) && RemoveSelectedLine();

[Ilya Sherman, 2012/04/17 08:31:20]

Does this mean that ctrl-shift-delete and other such shortcuts will also delete
the item?  I'd prefer that shift-delete and nothing else be the keyboard
shortcut for deleting.

[csharp, 2012/04/18 15:35:58]

Fixed. There seem to be other modifiers on by default, I grabbed a fix the other
gtk seems to use.

[Ilya Sherman, 2012/04/18 18:12:51]

Ah, that's tricky.  Nice job finding how to do that :)

     case GDK_Return:
     case GDK_KP_Enter:
       return AcceptSelectedLine();