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

chrome/browser/ui/views/action_box_menu.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/action_box_menu.h"
 
 #include "chrome/browser/ui/toolbar/action_box_menu_model.h"
 #include "chrome/browser/ui/views/browser_action_view.h"
 #include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
@@ skipping 22 matching lines @@
 ActionBoxMenu::~ActionBoxMenu() {
 }
 
 void ActionBoxMenu::Init() {
   DCHECK(!root_);
   root_ = new views::MenuItemView(this);
   root_->set_has_icons(true);
   PopulateMenu();
 }
 
-void ActionBoxMenu::RunMenu(views::MenuButton* menu_button) {
-  gfx::Point screen_location;
+void ActionBoxMenu::RunMenu(views::MenuButton* menu_button,
+                            gfx::Point menu_offset) {
+  gfx::Point screen_location(menu_offset);

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

Nit: Omit this line, you're passing by value precisely so that you can modify
|menu_offset| directly

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

Done.

   views::View::ConvertPointToScreen(menu_button, &screen_location);
   menu_runner_.reset(new views::MenuRunner(root_));
   // Ignore the result since we don't need to handle a deleted menu specially.
   ignore_result(
       menu_runner_->RunMenuAt(menu_button->GetWidget(),
                               menu_button,
                               gfx::Rect(screen_location, menu_button->size()),
                               views::MenuItemView::TOPRIGHT,
                               views::MenuRunner::HAS_MNEMONICS));
 }
@@ skipping 83 matching lines @@
         const extensions::Extension* extension =
             model_->GetExtensionAt(model_index);
         BrowserActionView* view = new BrowserActionView(extension,
             browser_, this);
         // |menu_item| will own the |view| from now on.
         menu_item->SetIconView(view);
       }
     }
   }
 }