Add an BubbleBorder to BrowserAction popups and fix positioning of the

chrome/browser/views/browser_actions_container.cc

 // Copyright (c) 2006-2009 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/views/browser_actions_container.h"
 
 #include "app/gfx/canvas.h"
 #include "app/resource_bundle.h"
 #include "base/stl_util-inl.h"
 #include "chrome/browser/extensions/extension_browser_event_router.h"
@@ skipping 314 matching lines @@
     bool same_showing = popup_ && button == popup_button_;
 
     // Always hide the current popup, even if it's not the same.
     // Only one popup should be visible at a time.
     HidePopup();
 
     if (same_showing)
       return;
 
     gfx::Point origin;
-    View::ConvertPointToWidget(button, &origin);
-    gfx::Rect rect = bounds();
+    View::ConvertPointToScreen(button, &origin);
+    gfx::Rect rect = button->bounds();
     rect.set_x(origin.x());
     rect.set_y(origin.y());
     popup_ = ExtensionPopup::Show(browser_action.popup_url(),
                                   toolbar_->browser(),
                                   rect,
                                   browser_action.popup_height());
     popup_->set_delegate(this);
     popup_button_ = button;
     popup_button_->PopupDidShow();
     return;
@@ skipping 142 matching lines @@
   // text was too large.
   rect.fLeft += kPadding;
   rect.fRight -= kPadding;
   canvas->clipRect(rect);
   canvas->drawText(text.c_str(), text.size(),
                    rect.fLeft + (rect.width() - text_width) / 2,
                    rect.fTop + kTextSize + 1,
                    text_paint);
   canvas->restore();
 }