First run bubble polish:

- improve spacing and and sizing, both internally and externally (using windows as the guide)
- make legible for dark themes
- make show delay longer because it was too short on my machine (at least when using a Debug build).

Also explicitly set bookmark bubble labels black.

BUG=16782
TEST=none

Review URL: http://codereview.chromium.org/149664

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20762 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 6 insertions, 3 deletions

diff --git a/chrome/browser/gtk/location_bar_view_gtk.cc b/chrome/browser/gtk/location_bar_view_gtk.cc
index 38c2901..211f997 100644
--- a/chrome/browser/gtk/location_bar_view_gtk.cc
+++ b/chrome/browser/gtk/location_bar_view_gtk.cc
@@ -38,9 +38,11 @@ const int kBottomMargin = 1;
 const int kBorderThickness = 1;
 
 // Left margin of first run bubble.
-const int kFirstRunBubbleLeftMargin = 20;
+const int kFirstRunBubbleLeftMargin = 8;
+// Extra vertical spacing for first run bubble.
+const int kFirstRunBubbleTopMargin = 1;
 // Task delay (in milliseconds) to show first run bubble.
-const int kFirstRunBubbleTaskDelay = 30;
+const int kFirstRunBubbleTaskDelay = 200;
 
 // Left and right padding/margin.
 // no icon/text  : 4px url_text 4px
@@ -511,7 +513,8 @@ void LocationBarViewGtk::ShowFirstRunBubbleInternal(bool use_OEM_bubble) {
   // The bubble needs to be just below the Omnibox and slightly to the right
   // of star button, so shift x and y co-ordinates.
   x += widget()->allocation.x + kFirstRunBubbleLeftMargin;
-  y += widget()->allocation.y + widget()->allocation.height;
+  y += widget()->allocation.y + widget()->allocation.height +
+       kFirstRunBubbleTopMargin;
 
   FirstRunBubble::Show(profile_,
       GTK_WINDOW(gtk_widget_get_toplevel(widget())),