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

Issue 155971: Linux: Select omnibox text on button release instead of press. (Closed)

Created:
11 years, 5 months ago by use derat at chromium.org
Modified:
9 years, 7 months ago
CC:
chromium-reviews_googlegroups.com, Peter Kasting, tony
Visibility:
Public.

Description

Linux: Select omnibox text on button release instead of press. We also don't select all of the text if the user dragged to highlight some text before releasing the button. Tested by repeating the scenarios from: http://codereview.chromium.org/151006 http://codereview.chromium.org/159185 plus checking that the following cases behave as expected: 1. Type some text into the omnibox. Click in Webkit to give it the focus. Click once in the omnibox. All of its text should be highlighted, and middle-clicking in another window should show that the primary selection was updated. 2. Click in the omnibox to clear its highlighted text and click in Webkit again. Click in the omnibox again, but this time drag to highlight some text before letting go of the button. The highlighted text should be preserved on button release, and the primary selection should be updated. 3. Repeat step 2. Click a few times in the omnibox and confirm that we don't select all of its text (that should only happen at the end of the initial click that focuses the omnibox).

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+62 lines, -26 lines) Patch
M chrome/browser/autocomplete/autocomplete_edit_view_gtk.h View 2 chunks +21 lines, -0 lines 0 comments Download
M chrome/browser/autocomplete/autocomplete_edit_view_gtk.cc View 4 chunks +41 lines, -26 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
use derat at chromium.org
11 years, 5 months ago (2009-07-23 00:51:07 UTC) #1
Evan Stade
tested it a little bit locally, seems to work, so lgtm
11 years, 5 months ago (2009-07-23 01:01:50 UTC) #2
Dean McNamee
Came out less complex than I expected. I'm ok with it.
11 years, 5 months ago (2009-07-23 14:42:52 UTC) #3
use derat at chromium.org
On Wed, Jul 22, 2009 at 9:44 PM, Brett Wilson<brettw@chromium.org> wrote: > On Wed, Jul ...
11 years, 5 months ago (2009-07-23 16:42:40 UTC) #4
brettw
11 years, 5 months ago (2009-07-23 16:49:35 UTC) #5
On Thu, Jul 23, 2009 at 9:42 AM, Daniel Erat<derat@google.com> wrote:
> On Wed, Jul 22, 2009 at 9:44 PM, Brett Wilson<brettw@chromium.org> wrote:
>> On Wed, Jul 22, 2009 at 2:53 PM, <estade@chromium.org> wrote:
>>> lgtm, committed r21329
>>
>> It's not clear. Did you address Peter and my concerns?
>
> About selecting-all on mouse-up instead of -down? =A0Doing so is
> unrelated to this change, but yes,
> http://codereview.chromium.org/155971 (LGTM-ed and going in today)
> fixes this for Linux.

Thanks!

Brett

Powered by Google App Engine
This is Rietveld 408576698