Chromium Code Reviews
Help | Chromium Project | Sign in
(1329)

Issue 9562029: GTK: Draw the correct omnibox state with new tab-to-traverse behavior. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 years, 1 month ago by Elliot Glaysher
Modified:
2 years, 1 month ago
Reviewers:
Peter Kasting
CC:
chromium-reviews_chromium.org, James Su
Visibility:
Public.

Description

GTK: Draw the correct omnibox state with new tab-to-traverse behavior.

This does everything but animate the trailing ">" icon from the right
side of the omnibox to the left when entering tab-to-search mode. The
TTS arrow indicator is now properly GTK themed, as well.

(Doing the animation is lower priority. It would require more
extensive refactoring that I don't have time for as I have code yellow work.)

BUG=116292
TEST=all new tab-to-search behavior matches views sans animation.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=124552

Patch Set 1 #

Total comments: 3

Patch Set 2 : pkasting nits #

Unified diffs Side-by-side diffs Delta from patch set Stats (+65 lines, -10 lines) Lint Patch
M chrome/browser/ui/gtk/omnibox/omnibox_popup_view_gtk.h View 1 1 chunk +9 lines, -1 line 0 comments 0 errors Download
M chrome/browser/ui/gtk/omnibox/omnibox_popup_view_gtk.cc View 1 8 chunks +51 lines, -8 lines 0 comments 0 errors Download
M chrome/browser/ui/gtk/theme_service_gtk.cc View 3 chunks +5 lines, -1 line 0 comments 0 errors Download
Commit:

Messages

Total messages: 4
Elliot Glaysher
2 years, 1 month ago #1
Peter Kasting
LGTM. Refactoring would really take longer than a day? https://chromiumcodereview.appspot.com/9562029/diff/1/chrome/browser/ui/gtk/omnibox/omnibox_popup_view_gtk.cc File chrome/browser/ui/gtk/omnibox/omnibox_popup_view_gtk.cc (right): https://chromiumcodereview.appspot.com/9562029/diff/1/chrome/browser/ui/gtk/omnibox/omnibox_popup_view_gtk.cc#newcode491 chrome/browser/ui/gtk/omnibox/omnibox_popup_view_gtk.cc:491: ...
2 years, 1 month ago #2
I haz the power (commit-bot)
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/erg@chromium.org/9562029/5001
2 years, 1 month ago #3
I haz the power (commit-bot)
2 years, 1 month ago #4
Change committed as 124552
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1275:d14800f88434