Better keyboard and mouse handling in Omnibox:

chrome/browser/autocomplete/autocomplete_edit_view_gtk.cc

 // Copyright (c) 2009 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/autocomplete/autocomplete_edit_view_gtk.h"
 
 #include <gtk/gtk.h>
+#include <gdk/gdkkeysyms.h>

[Evan Martin, 2009/03/11 18:25:32]

alphabetize?

 
 #include "base/logging.h"
 #include "base/string_util.h"
 #include "chrome/browser/autocomplete/autocomplete_edit.h"
 #include "chrome/browser/autocomplete/autocomplete_popup_model.h"
 #include "chrome/browser/autocomplete/autocomplete_popup_view_gtk.h"
 #include "chrome/browser/tab_contents/tab_contents.h"
 #include "chrome/browser/toolbar_model.h"
 #include "chrome/common/notification_service.h"
 #include "googleurl/src/gurl.h"
@@ skipping 61 matching lines @@
       NULL, "foreground", kInsecureSchemeColor, NULL);
 
   // NOTE: This code used to connect to "changed", however this was fired too
   // often and during bad times (our own buffer changes?).  It works out much
   // better to listen to end-user-action, which should be fired whenever the
   // user makes some sort of change to the buffer.
   g_signal_connect(text_buffer_, "begin-user-action",
                    G_CALLBACK(&HandleBeginUserActionThunk), this);
   g_signal_connect(text_buffer_, "end-user-action",
                    G_CALLBACK(&HandleEndUserActionThunk), this);
+  // We connect to key press and release for special handling of the enter key.
+  g_signal_connect(text_view_, "key-press-event",
+                   G_CALLBACK(&HandleKeyPressThunk), this);
+  g_signal_connect(text_view_, "key-release-event",
+                   G_CALLBACK(&HandleKeyReleaseThunk), this);
   g_signal_connect(text_view_, "size-request",
                    G_CALLBACK(&HandleViewSizeRequest), this);
   g_signal_connect(text_view_, "button-press-event",
                    G_CALLBACK(&HandleViewButtonPressThunk), this);
   g_signal_connect(text_view_, "focus-in-event",
                    G_CALLBACK(&HandleViewFocusInThunk), this);
   g_signal_connect(text_view_, "focus-out-event",
                    G_CALLBACK(&HandleViewFocusOutThunk), this);
   // NOTE: The GtkTextView documentation asks you not to connect to this
   // signal, but it is very convenient and clean for catching up/down.
@@ skipping 187 matching lines @@
   gdk_window_get_origin(text_view_->window, x, y);
   *y += text_view_->allocation.height;
   *width = text_view_->allocation.width;
 }
 
 void AutocompleteEditViewGtk::HandleBeginUserAction() {
   OnBeforePossibleChange();
 }
 
 void AutocompleteEditViewGtk::HandleEndUserAction() {
-  bool had_newline = false;
+  // Eat any newline / paragraphs that might have come in, for example in a
+  // copy and paste.  We want to make sure our widget stays single line.
+  for (;;) {
+    GtkTextIter cur;
+    gtk_text_buffer_get_start_iter(text_buffer_, &cur);
 
-  // TODO(deanm): obviously super inefficient.
-  for(;;) {
-    GtkTextIter cur, end;
-    gtk_text_buffer_get_bounds(text_buffer_, &cur, &end);
+    // If there is a line ending, this should put is right before the newline

[Evan Martin, 2009/03/11 18:25:32]

typo: s/is/it/

+    // or carriage return / newline (or Unicode) sequence.  If not, we're done.
+    if (gtk_text_iter_forward_to_line_end(&cur) == FALSE)
+      break;
 
-    while (!gtk_text_iter_equal(&cur, &end)) {
-      if (gtk_text_iter_ends_line(&cur)) {
-        had_newline = true;
-        GtkTextIter next = cur;
-        gtk_text_iter_forward_char(&next);
-        gtk_text_buffer_delete(text_buffer_, &cur, &next);
+    // Stepping to the next cursor position should put us on the other side of
+    // the newline / paragraph / etc sequence, and then delete this range.
+    GtkTextIter next_line = cur;
+    gtk_text_iter_forward_cursor_position(&next_line);
+    gtk_text_buffer_delete(text_buffer_, &cur, &next_line);
 
-        // We've invalidated our iterators, gotta start again.
-        break;
-      }
-
-      gtk_text_iter_forward_char(&cur);
-    }
-
-    // We've exhausted the whole input and there is now only 1 line, good.
-    if (gtk_text_iter_equal(&cur, &end))
-      break;
+    // We've invalidated our iterators, gotta start again.
   }
 
   OnAfterPossibleChange();
+}
 
-  if (had_newline)
-    model_->AcceptInput(CURRENT_TAB, false);
+gboolean AutocompleteEditViewGtk::HandleKeyPress(GtkWidget* widget,
+                                                 GdkEventKey* event) {
+  // This is very similar to the special casing of the return key in the
+  // GtkTextView key_press default handler.  TODO(deanm): We do however omit
+  // some IME related code, this might become a problem if an IME wants to
+  // handle enter.  We can get at the im_context and do it ourselves if needed.
+  if (event->keyval == GDK_Return ||
+      event->keyval == GDK_ISO_Enter ||
+      event->keyval == GDK_KP_Enter) {
+    bool alt_held = (event->state & GDK_MOD1_MASK);
+    model_->AcceptInput(alt_held ? NEW_FOREGROUND_TAB : CURRENT_TAB, false);
+    return TRUE;  // Don't propagate into GtkTextView.
+  }
+  return FALSE;  // Propagate into GtkTextView.
+}
+
+gboolean AutocompleteEditViewGtk::HandleKeyRelease(GtkWidget* widget,
+                                                   GdkEventKey* event) {
+  // We ate the press, might as well eat the release.
+  if (event->keyval == GDK_Return ||

[Evan Stade, 2009/03/11 20:56:29]

perhaps factor this out into a helper function GdkKeyvalIsReturn(int keyval)

+      event->keyval == GDK_ISO_Enter ||
+      event->keyval == GDK_KP_Enter) {
+    return TRUE;  // Don't propagate into GtkTextView.
+  }
+  return FALSE;  // Propagate into GtkTextView.
 }
 
 // static
 void AutocompleteEditViewGtk::HandleViewSizeRequest(GtkWidget* view,
                                                     GtkRequisition* req,
                                                     gpointer unused) {
   // Don't force a minimum size, allow our embedder to size us better.
   req->height = req->width = 1;
 }
 
@@ skipping 31 matching lines @@
 gboolean AutocompleteEditViewGtk::HandleViewFocusIn() {
   model_->OnSetFocus(false);
   // TODO(deanm): Some keyword hit business, etc here.
   
   return FALSE;  // Continue propagation.
 }
 
 gboolean AutocompleteEditViewGtk::HandleViewFocusOut() {
   // Close the popup.
   ClosePopup();
-
   // Tell the model to reset itself.
   model_->OnKillFocus();
-
-  // TODO(deanm): This probably isn't right, and doesn't match Windows.  We
-  // don't really want to match Windows though, because it probably feels
-  // wrong on Linux.  Firefox doesn't have great behavior here also, imo.
-  // Deselect any selection and make sure the input is at the beginning.
-  GtkTextIter start, end;
-  gtk_text_buffer_get_bounds(text_buffer_, &start, &end);
-  gtk_text_buffer_place_cursor(text_buffer_, &start);
   return FALSE;  // Pass the event on to the GtkTextView.
 }
 
 void AutocompleteEditViewGtk::HandleViewMoveCursor(
     GtkMovementStep step,
     gint count,
     gboolean extendion_selection) {
   // Handle up / down cursor movement on our own.
   if (step == GTK_MOVEMENT_DISPLAY_LINES) {
     model_->OnUpOrDownKeyPressed(count);
@@ skipping 80 matching lines @@
                                 &start, &end);
     }
     gtk_widget_modify_base(text_view_, GTK_STATE_NORMAL, background);
   }
 }
 
 void AutocompleteEditViewGtk::TextChanged() {
   EmphasizeURLComponents();
   controller_->OnChanged();
 }