Adjusted action box menu position relative to omnibox and action box menu button.

chrome/browser/ui/views/location_bar/location_bar_view.cc

 // Copyright (c) 2012 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/ui/views/location_bar/location_bar_view.h"
 
 #include <algorithm>
 #include <map>
 
 #include "base/command_line.h"
@@ skipping 293 matching lines @@
   AddChildView(web_intents_button_view_);
 
   if (browser_defaults::bookmarks_enabled && (mode_ == NORMAL)) {
     // Note: condition above means that the star icon is hidden in popups and in
     // the app launcher.
     star_view_ = new StarView(command_updater_);
     AddChildView(star_view_);
     star_view_->SetVisible(true);
   }
   if (extensions::switch_utils::IsActionBoxEnabled() && browser_) {
-    action_box_button_view_ = new ActionBoxButtonView(browser_);
+    gfx::Point menu_offset(

[Peter Kasting, 2012/09/14 22:50:53]

Nit: I would probably inline this into the next statement but it's not too
important

[yefimt, 2012/09/14 23:00:56]

I always felt that N one line expressions are more readable then one N line
expression :)

On 2012/09/14 22:50:53, Peter Kasting wrote:

+        (mode_ == NORMAL) ? kNormalHorizontalEdgeThickness : 0,
+        kVerticalEdgeThickness);
+    action_box_button_view_ = new ActionBoxButtonView(browser_, menu_offset);
     AddChildView(action_box_button_view_);
+
     if (star_view_)
       star_view_->SetVisible(false);
   }
 
   registrar_.Add(this,
                  chrome::NOTIFICATION_EXTENSION_LOCATION_BAR_UPDATED,
                  content::Source<Profile>(profile_));
 
   // Initialize the location entry. We do this to avoid a black flash which is
   // visible when the location entry has just been initialized.
@@ skipping 1225 matching lines @@
 }
 
 void LocationBarView::CleanupFadeAnimation() {
   // Since we're no longer animating we don't need our layer.
   SetPaintToLayer(false);
   // Bubble labels don't need a transparent background anymore.
   ev_bubble_view_->SetLabelBackgroundColor(SK_ColorWHITE);
   selected_keyword_view_->SetLabelBackgroundColor(SK_ColorWHITE);
 }
 #endif  // USE_AURA